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

ui/gl/gl_image_io_surface.cc

Index: ui/gl/gl_image_io_surface.cc
diff --git a/ui/gl/gl_image_io_surface.cc b/ui/gl/gl_image_io_surface.cc
index e20c739cffad6ce232405e4cc460e7aeec078a39..4b27d5cbd248c18a1b8cdb31fb52e1b73fb13a60 100644
--- a/ui/gl/gl_image_io_surface.cc
+++ b/ui/gl/gl_image_io_surface.cc
@@ -15,7 +15,9 @@ GLImageIOSurface::GLImageIOSurface(gfx::Size size)
   CHECK(io_surface_support_);
 }
 
-GLImageIOSurface::~GLImageIOSurface() { Destroy(); }
+GLImageIOSurface::~GLImageIOSurface() {
+  DCHECK(!io_surface_);
+}
 
 bool GLImageIOSurface::Initialize(gfx::GpuMemoryBufferHandle buffer) {
   io_surface_.reset(io_surface_support_->IOSurfaceLookup(buffer.io_surface_id));
@@ -27,6 +29,11 @@ bool GLImageIOSurface::Initialize(gfx::GpuMemoryBufferHandle buffer) {
   return true;
 }
 
+void GLImageIOSurface::Destroy(bool have_context) {
+  DCHECK(io_surface_);

[reveman, 2014/05/31 05:05:06]

This DCHECK seems inconsistent with other implementations of the Destroy
function which seem to handle multiple calls to Destroy or Initialize not having
been called fine.

[sohanjg, 2014/06/02 13:59:42]

Done.

+  io_surface_.reset()
+}
+
 gfx::Size GLImageIOSurface::GetSize() { return size_; }
 
 bool GLImageIOSurface::BindTexImage(unsigned target) {