WebGL: Don't destroy mailbox textures in the destructor until they're released.

Source/core/html/canvas/WebGLRenderingContext.h

Index: Source/core/html/canvas/WebGLRenderingContext.h
diff --git a/Source/core/html/canvas/WebGLRenderingContext.h b/Source/core/html/canvas/WebGLRenderingContext.h
index 6e4494cc95f7fc29f1fa9feb785fc8075c61b8db..83babe3cca4f9497fe2a7320b136af64677f7fa7 100644
--- a/Source/core/html/canvas/WebGLRenderingContext.h
+++ b/Source/core/html/canvas/WebGLRenderingContext.h
@@ -318,7 +318,7 @@ public:
     void forceRestoreContext();
     void loseContextImpl(LostContextMode);
 
-    blink::WebGraphicsContext3D* webGraphicsContext3D() const { return m_context.get(); }
+    blink::WebGraphicsContext3D* webGraphicsContext3D() const { return m_context; }
     WebGLContextGroup* contextGroup() const { return m_contextGroup.get(); }
     virtual blink::WebLayer* platformLayer() const OVERRIDE;
     Extensions3DUtil* extensionsUtil();
@@ -385,7 +385,7 @@ public:
 
     WebGLRenderbuffer* ensureEmulatedStencilBuffer(GLenum target, WebGLRenderbuffer*);
 
-    OwnPtr<blink::WebGraphicsContext3D> m_context;
+    blink::WebGraphicsContext3D* m_context;

[Justin Novosad, 2014/02/18 18:07:33]

IMHO: the ownership model would be clearer and more robust if you got rid of
this member, and instead accessed the context through m_drawingBuffer.

     RefPtr<WebGLContextGroup> m_contextGroup;
 
     // Structure for rendering to a DrawingBuffer, instead of directly