kWebGraphicsContext3DPerferredGLExtensions -> kWebGLPreferredGLExtensions

content/renderer/webgraphicscontext3d_command_buffer_impl.cc

Index: content/renderer/webgraphicscontext3d_command_buffer_impl.cc
diff --git a/content/renderer/webgraphicscontext3d_command_buffer_impl.cc b/content/renderer/webgraphicscontext3d_command_buffer_impl.cc
index ac1b62de47a8449d41f7dde998b73bf8573db2a7..0fad22b3b28e1559e56c53a198b4a0aa10e31655 100644
--- a/content/renderer/webgraphicscontext3d_command_buffer_impl.cc
+++ b/content/renderer/webgraphicscontext3d_command_buffer_impl.cc
@@ -47,7 +47,10 @@ WebGraphicsContext3DCommandBufferImpl::
   delete context_;
 }
 
-static const char* kWebGraphicsContext3DPerferredGLExtensions =
+// This string should only be passed for WebGL contexts. Nothing ELSE!!!
+// Compositor contexts, Canvas2D contexts, Pepper Contexts, nor any other use of
+// a context should not pass this string.
+static const char* kWebGLPreferredGLExtensions =
     "GL_OES_packed_depth_stencil "
     "GL_OES_depth24 "
     "GL_CHROMIUM_webglsl";
@@ -108,7 +111,7 @@ bool WebGraphicsContext3DCommandBufferImpl::initialize(
         host,
         renderview->compositing_surface(),
         renderview->routing_id(),
-        kWebGraphicsContext3DPerferredGLExtensions,
+        kWebGLPreferredGLExtensions,

[jamesr, 2011/05/24 22:05:11]

i think this codepath (render_directly_to_web_view) is only used for compositor
contexts, so this seems to contradict the comment above. should we be passing a
different string in?

         attribs,
         active_url);
     if (context_) {
@@ -138,7 +141,7 @@ bool WebGraphicsContext3DCommandBufferImpl::initialize(
         host,
         parent_context,
         gfx::Size(1, 1),
-        kWebGraphicsContext3DPerferredGLExtensions,
+        kWebGLPreferredGLExtensions,

[jamesr, 2011/05/24 22:05:11]

this is used for webgl, canvas 2d, and probably other stuff too :(

         attribs,
         active_url);
     web_view_ = NULL;