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..c53e2468360066dd31a642585d62091f2e361100 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,9 @@ void ImageManager::RegisterGpuMemoryBuffer(int32 id,
 }
 
 void ImageManager::UnregisterGpuMemoryBuffer(int32 id) {
+  gfx::GLImage* image = LookupImage(id);
+  DCHECK(image);

[reveman, 2014/06/03 13:57:39]

What's preventing a malicous renderer from using an invalid id and causing this
to crash?

[sohanjg, 2014/06/03 15:09:20]

Done.

+  image->Destroy(true);
   RemoveImage(id);
 }