During image destroy, delete textures only if we have a GL context.

gpu/command_buffer/service/image_manager.cc

Index: gpu/command_buffer/service/image_manager.cc
diff --git a/gpu/command_buffer/service/image_manager.cc b/gpu/command_buffer/service/image_manager.cc
index a3ac4350c875fabb11f6e53375bfeef68d158e6f..e0394c361538936e2f64b9f563e9b751bd4e8f0b 100644
--- a/gpu/command_buffer/service/image_manager.cc
+++ b/gpu/command_buffer/service/image_manager.cc
@@ -15,6 +15,14 @@ ImageManager::ImageManager() : release_after_use_(false) {
 ImageManager::~ImageManager() {
 }
 
+void ImageManager::Destroy(bool have_context) {
+  for (GLImageMap::const_iterator iter = gl_images_.begin();
+       iter != gl_images_.end();
+       ++iter)
+    iter->second.get()->Destroy(have_context);
+  gl_images_.clear();
+}
+
 void ImageManager::RegisterGpuMemoryBuffer(int32 id,
                                            gfx::GpuMemoryBufferHandle buffer,
                                            size_t width,
@@ -44,6 +52,16 @@ void ImageManager::RegisterGpuMemoryBuffer(int32 id,
 }
 
 void ImageManager::UnregisterGpuMemoryBuffer(int32 id) {
+  if (id <= 0) {
+    DVLOG(0) << "Cannot Unregister GPU memory buffer with non-positive ID.";
+    return;
+  }
+  gfx::GLImage* image = LookupImage(id);
+  if (!image) {
+    DVLOG(0) << "Invalid GLImage corresponding to given ID";

[reveman, 2014/06/03 15:57:54]

nit: the GLImage is not invalid, the ID is

[sohanjg, 2014/06/04 06:39:02]

Done.

+    return;
+  }
+  image->Destroy(true);

[reveman, 2014/06/03 15:57:54]

nit: please make the use of blank lines consistent with RegisterGpuMemoryBuffer

[sohanjg, 2014/06/04 06:39:02]

Done.

   RemoveImage(id);
 }