Stop printing GL errors for disabling color tables on core profiles on nvidia/windows machines.

src/gpu/gl/GrGpuGL.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "GrGpuGL.h"
 #include "GrGLStencilBuffer.h"
@@ skipping 404 matching lines @@
     if (kDesktop_GrGLBinding == this->glBinding()) {
         // Desktop-only state that we never change
         if (!this->glCaps().isCoreProfile()) {
             GL_CALL(Disable(GR_GL_POINT_SMOOTH));
             GL_CALL(Disable(GR_GL_LINE_SMOOTH));
             GL_CALL(Disable(GR_GL_POLYGON_SMOOTH));
             GL_CALL(Disable(GR_GL_POLYGON_STIPPLE));
             GL_CALL(Disable(GR_GL_COLOR_LOGIC_OP));
             GL_CALL(Disable(GR_GL_INDEX_LOGIC_OP));
         }
-        if (this->glCaps().imagingSupport()) {
-            // This produces a GL error on the windows NVIDIA driver when using a core profile but
-            // I think that is a driver bug since GL_ARB_imaging is in the extension string.
+        // The windows NVIDIA driver has GL_ARB_imaging in the extension string when using a core
+        // profile. This seems like a bug since the core spec removes any mention of GL_ARB_imaging.
+        if (this->glCaps().imagingSupport() && !this->glCaps().isCoreProfile()) {
             GL_CALL(Disable(GR_GL_COLOR_TABLE));
         }
         GL_CALL(Disable(GR_GL_POLYGON_OFFSET_FILL));
         // Since ES doesn't support glPointSize at all we always use the VS to
         // set the point size
         GL_CALL(Enable(GR_GL_VERTEX_PROGRAM_POINT_SIZE));
 
         // We should set glPolygonMode(FRONT_AND_BACK,FILL) here, too. It isn't
         // currently part of our gl interface. There are probably others as
         // well.
@@ skipping 1909 matching lines @@
             this->setVertexArrayID(gpu, 0);
         }
         int attrCount = gpu->glCaps().maxVertexAttributes();
         if (fDefaultVertexArrayAttribState.count() != attrCount) {
             fDefaultVertexArrayAttribState.resize(attrCount);
         }
         attribState = &fDefaultVertexArrayAttribState;
     }
     return attribState;
 }