Removed most calls to GraphicsContext3D from DrawingBuffer

Source/platform/graphics/gpu/DrawingBuffer.h

Index: Source/platform/graphics/gpu/DrawingBuffer.h
diff --git a/Source/platform/graphics/gpu/DrawingBuffer.h b/Source/platform/graphics/gpu/DrawingBuffer.h
index d6f880fb308bb277c37ce60de6181228ce5e343a..b1260a479072fe597f1691927752853632d01b35 100644
--- a/Source/platform/graphics/gpu/DrawingBuffer.h
+++ b/Source/platform/graphics/gpu/DrawingBuffer.h
@@ -38,6 +38,7 @@
 
 #include "public/platform/WebExternalTextureLayerClient.h"
 #include "public/platform/WebExternalTextureMailbox.h"
+#include "public/platform/WebGraphicsContext3D.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
@@ -76,7 +77,7 @@ public:
         Discard
     };
 
-    static PassRefPtr<DrawingBuffer> create(GraphicsContext3D*, const IntSize&, PreserveDrawingBuffer, PassRefPtr<ContextEvictionManager>);
+    static PassRefPtr<DrawingBuffer> create(blink::WebGraphicsContext3D*, const IntSize&, PreserveDrawingBuffer, PassRefPtr<ContextEvictionManager>);
 
     ~DrawingBuffer();
 
@@ -133,7 +134,7 @@ public:
         GLenum destType, GLint level, bool premultiplyAlpha, bool flipY);
 
 private:
-    DrawingBuffer(GraphicsContext3D*, const IntSize&, bool multisampleExtensionSupported,
+    DrawingBuffer(blink::WebGraphicsContext3D*, GraphicsContext3D*, const IntSize&, bool multisampleExtensionSupported,
                   bool packedDepthStencilExtensionSupported, PreserveDrawingBuffer, PassRefPtr<ContextEvictionManager>);
 
     void initialize(const IntSize&);
@@ -172,7 +173,8 @@ private:
     Platform3DObject m_framebufferBinding;
     GLenum m_activeTextureUnit;
 
-    RefPtr<GraphicsContext3D> m_context;
+    blink::WebGraphicsContext3D* m_context;

[bajones, 2014/01/08 03:13:08]

General design/style question: When using GraphicsContext3D we used RefPtr to
manage it, but that won't work for WebGraphicsContext3D, since it's actually
instantiated Chromium-side. In WebGLRenderingContext we're using OwnPtr to
manage the object lifetime, is there a preferred pattern for classes like this
one that the object is passed into?

+    RefPtr<GraphicsContext3D> m_contextSupport;
     IntSize m_size;
     bool m_multisampleExtensionSupported;
     bool m_packedDepthStencilExtensionSupported;
@@ -197,7 +199,7 @@ private:
     // True if commit() has been called since the last time markContentsChanged() had been called.
     bool m_contentsChangeCommitted;
 
-    GraphicsContext3D::Attributes m_attributes;
+    blink::WebGraphicsContext3D::Attributes m_attributes;
     unsigned m_internalColorFormat;
     unsigned m_colorFormat;
     unsigned m_internalRenderbufferFormat;