Handle currently bound texture id in PowerVR workaround

gpu/command_buffer/service/texture_definition.cc

 // Copyright 2014 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/texture_definition.h"
 
 #include <list>
 
 #include "base/memory/linked_ptr.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 393 matching lines @@
   texture->usage_ = usage_;
 }
 
 void TextureDefinition::UpdateTexture(Texture* texture) const {
   GLuint old_service_id = 0u;
   if (image_buffer_.get() && g_avoid_egl_target_texture_reuse) {
     GLuint service_id = 0u;
     glGenTextures(1, &service_id);
     old_service_id = texture->service_id();
     texture->SetServiceId(service_id);
+
+    GLint bound_id = 0;
+    glGetIntegerv(target_, &bound_id);

[no sievers, 2015/04/29 21:04:43]

This doesn't work. It will generate a GL error I assume.

You can just DCHECK_EQ(GL_TEXTURE2D, target_); and then do:
glGetIntegerv(GL_TEXTURE_BINDING_2D, &bound_id);

[boliu, 2015/04/29 21:16:14]

Meh, missed the giant block doing the GL_TEXTURE2D -> GL_TEXTURE_BINDING_2D in
ScopedTextureBinder. Done

Surprised there isn't already a helper function to do that though..

+    if (bound_id == static_cast<GLint>(old_service_id)) {
+      glBindTexture(target_, service_id);
+    }
   }
 
   UpdateTextureInternal(texture);
 
   if (old_service_id) {
     glDeleteTextures(1, &old_service_id);
   }
 }
 
 bool TextureDefinition::Matches(const Texture* texture) const {
@@ skipping 16 matching lines @@
 
   return true;
 }
 
 bool TextureDefinition::SafeToRenderFrom() const {
   return level_info_.cleared;
 }
 
 }  // namespace gles2
 }  // namespace gpu