[Android] Add workaround to unbind gpu memory buffer only on NVIDIA

ui/gl/gl_image.h

Index: ui/gl/gl_image.h
diff --git a/ui/gl/gl_image.h b/ui/gl/gl_image.h
index 859966c6a7f00d804c9bd1d7def8632a27c3d689..b5b28755cc100db7a521aa5c1a8d363f7a403cb4 100644
--- a/ui/gl/gl_image.h
+++ b/ui/gl/gl_image.h
@@ -37,7 +37,9 @@ class GL_EXPORT GLImage : public base::RefCounted<GLImage> {
   virtual void WillUseTexImage();
 
   // Called after the texture has been used for drawing.
-  virtual void DidUseTexImage();
+  // If |release_tex_image| is false, then skip releasing image from texture
+  // currently bound to GL_TEXTURE_2D.
+  virtual void DidUseTexImage(bool release_tex_image);

[reveman, 2013/11/12 16:39:42]

Bind/release as part of Will/DidUseTexImage is an implementation detail. This
API should not expose that.

How about we add a "virtual void SetUnbindImageOnDidUse()" to the GLImage class
with a comment that makes it clear that this is for an Android work-around?
Consistent with GLContext::SetUnbindFboOnMakeCurrent.

[boliu, 2013/11/12 16:47:47]

Ahh, that's why this felt so weird. Yes, SetUnbindImageOnDidUse sounds more
reaonsable.

One more question, do you care what the default for workaround should be for
non-android platforms?

 
   // Create a GL image for a window.
   static scoped_refptr<GLImage> CreateGLImage(gfx::PluginWindowHandle window);