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

gpu/command_buffer/service/image_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "gpu/command_buffer/service/image_manager.h"
 
+#include "base/logging.h"
 #include "ui/gl/gl_image.h"
 
 namespace gpu {
 namespace gles2 {
 
 ImageManager::ImageManager() {
 }
 
 ImageManager::~ImageManager() {
 }
 
+void ImageManager::Destroy(bool have_context) {
+  for (GLImageMap::const_iterator iter = images_.begin(); iter != images_.end();
+       ++iter)
+    iter->second.get()->Destroy(have_context);
+  images_.clear();
+}
+
 void ImageManager::AddImage(gfx::GLImage* image, int32 service_id) {
+  // Ensure the service_id is not already in use.

[reveman, 2014/07/25 19:00:23]

no need for this comment.

[sohanjg, 2014/07/26 10:42:23]

Done.

+  DCHECK(images_.find(service_id) == images_.end());
   images_[service_id] = image;
 }
 
 void ImageManager::RemoveImage(int32 service_id) {
-  images_.erase(service_id);
+  if (service_id <= 0) {

[reveman, 2014/07/25 19:00:23]

this check should not be here.

[sohanjg, 2014/07/26 10:42:23]

Done.

+    LOG(ERROR) << "Cannot remove image with non-positive ID.";
+    return;
+  }
+  GLImageMap::iterator iter = images_.find(service_id);
+  if (iter == images_.end()) {

[reveman, 2014/07/25 19:00:23]

replace this with DCHECK(iter != images_.end());

[sohanjg, 2014/07/26 10:42:23]

Done.

+    LOG(ERROR) << "Invalid ID.";
+    return;
+  }
+  iter->second.get()->Destroy(true);
+  images_.erase(iter);
 }
 
 gfx::GLImage* ImageManager::LookupImage(int32 service_id) {
   GLImageMap::const_iterator iter = images_.find(service_id);
   if (iter != images_.end())
     return iter->second.get();
 
   return NULL;
 }
 
 }  // namespace gles2
 }  // namespace gpu