gpu: Add YCbCr 420v extension.

ui/gl/scoped_binders.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 "ui/gl/scoped_binders.h"
 #include "ui/gl/gl_bindings.h"
 #include "ui/gl/gl_context.h"
 #include "ui/gl/gl_state_restorer.h"
 
 namespace gfx {
@@ skipping 29 matching lines @@
     switch (target) {
       case GL_TEXTURE_2D:
         target_getter = GL_TEXTURE_BINDING_2D;
         break;
       case GL_TEXTURE_CUBE_MAP:
         target_getter = GL_TEXTURE_BINDING_CUBE_MAP;
         break;
       case GL_TEXTURE_EXTERNAL_OES:
         target_getter = GL_TEXTURE_BINDING_EXTERNAL_OES;
         break;
+      case GL_TEXTURE_RECTANGLE_ARB:
+        target_getter = GL_TEXTURE_BINDING_RECTANGLE_ARB;
+        break;
       default:
-        NOTIMPLEMENTED() << "Target not part of OpenGL ES 2.0 spec.";
+        NOTIMPLEMENTED() << " Target not part of OpenGL ES 2.0 spec.";

[reveman, 2015/10/27 19:31:54]

why the space here? also I'm not sure this is accurate anymore. RECTANGLE is not
part of the ES 2.0 spec. we should update the comment so it's not confusing.

[Daniele Castagna, 2015/10/29 20:09:05]

Fixed the string.
The added space is because NOTIMPLEMENTED prints out "NOT IMPLEMENTED" and
concatenating a string without spaces ended up printing
"[...]IMPLEMENTEDTarget[...]"

     }
     glGetIntegerv(target_getter, &old_id_);
   }
   glBindTexture(target_, id);
 }
 
 ScopedTextureBinder::~ScopedTextureBinder() {
   if (state_restorer_) {
     DCHECK(!!GLContext::GetCurrent());
     DCHECK_EQ(state_restorer_, GLContext::GetCurrent()->GetGLStateRestorer());
     state_restorer_->RestoreActiveTextureUnitBinding(target_);
   } else {
     glBindTexture(target_, old_id_);
   }
 }
 
 }  // namespace gfx