Replace non-copyable macro with a helper class.

This class provides a simpler scheme for blocking default copy
and assignment operators. It also reduces the amount of code
needed since it's inherited to child classes. This also fixes
the conflict between our macro and the same-named macro in
Chromium code.

BUG=angleproject:956

Change-Id: If0dc72aa3f63fbc7b8fa34907418821c64c39e2f
Reviewed-on: https://chromium-review.googlesource.com/263257
Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
diff --git a/src/libANGLE/renderer/FramebufferImpl.h b/src/libANGLE/renderer/FramebufferImpl.h
index 8e296b8..64b01e8 100644
--- a/src/libANGLE/renderer/FramebufferImpl.h
+++ b/src/libANGLE/renderer/FramebufferImpl.h
@@ -25,7 +25,7 @@
 namespace rx
 {
 
-class FramebufferImpl
+class FramebufferImpl : angle::NonCopyable
 {
   public:
     explicit FramebufferImpl(const gl::Framebuffer::Data &data) : mData(data) { }
@@ -61,9 +61,6 @@
 
   protected:
     const gl::Framebuffer::Data &mData;
-
-  private:
-    DISALLOW_COPY_AND_ASSIGN(FramebufferImpl);
 };
 
 }