aura/texture transport: Keep ref to texture infos to avoid races with the UI

content/common/gpu/texture_image_transport_surface.h

Index: content/common/gpu/texture_image_transport_surface.h
diff --git a/content/common/gpu/texture_image_transport_surface.h b/content/common/gpu/texture_image_transport_surface.h
index 5fcd5d30b3d16d9a2703f67edcf02b98931f0f3f..701e9d6058da6b2ccb9d80f27ff82a14351f89ce 100644
--- a/content/common/gpu/texture_image_transport_surface.h
+++ b/content/common/gpu/texture_image_transport_surface.h
@@ -7,7 +7,6 @@
 #pragma once
 
 #include "base/basictypes.h"
-#include "base/memory/weak_ptr.h"
 #include "content/common/gpu/gpu_command_buffer_stub.h"
 #include "content/common/gpu/image_transport_surface.h"
 #include "gpu/command_buffer/service/texture_manager.h"
@@ -56,7 +55,8 @@ class TextureImageTransportSurface :
  private:
   // A texture backing the front/back buffer in the parent stub.
   struct Texture {
-    Texture() : client_id(0), sent_to_client(false) {}
+    Texture();
+    ~Texture();
 
     // The client-side id in the parent stub.
     uint32 client_id;
@@ -66,13 +66,15 @@ class TextureImageTransportSurface :
 
     // Whether or not that texture has been sent to the client yet.
     bool sent_to_client;
+
+    // The texture info in the parent stub.
+    scoped_refptr<gpu::gles2::TextureManager::TextureInfo> info;

[jonathan.backer, 2012/04/17 17:21:50]

TextureManager::Ref

[piman, 2012/04/17 17:37:24]

Done.

   };
 
   virtual ~TextureImageTransportSurface();
   void CreateBackTexture(const gfx::Size& size);
   void ReleaseBackTexture();
   void AttachBackTextureToFBO();
-  gpu::gles2::TextureManager::TextureInfo* GetParentInfo(uint32 client_id);
   int back() const { return 1 - front_; }
 
   // The framebuffer that represents this surface (service id). Allocated lazily
@@ -91,7 +93,7 @@ class TextureImageTransportSurface :
   bool stub_destroyed_;
 
   scoped_ptr<ImageTransportHelper> helper_;
-  base::WeakPtr<GpuCommandBufferStub> parent_stub_;
+  GpuCommandBufferStub* parent_stub_;

[jonathan.backer, 2012/04/17 17:21:50]

Here's my understanding: we've got a race on teardown. With a weak_ptr,
parent_stub_ is only NULLed in the destructor. However, parent_stub_ is really
invalid during its Destroy. By becoming a DestructionObserver of parent_stub_,
we can invalidate at the appropriate time.

 
   DISALLOW_COPY_AND_ASSIGN(TextureImageTransportSurface);
 };