Add Ability to pass in allowed extensions to GPU contexts.

Questions:

1) Is WebGraphicsContext3D only used for WebGL? Currently
this patch makes that assumption.

2) I started by adding const char* allowed_extensions to
a bunch of functions. Then, at the IPC level added
GPUInitParams. Should I use GPUInitParams everywhere?

3) Is the path for gpu_init_params.h ok?

TEST=unit tests, manually tested WebGL
BUG=none

Review URL: http://codereview.chromium.org/3775014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63026 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 16 insertions, 12 deletions

diff --git a/gpu/command_buffer/service/gpu_processor.h b/gpu/command_buffer/service/gpu_processor.h
index 417f4e0..79d9e73 100644
--- a/gpu/command_buffer/service/gpu_processor.h
+++ b/gpu/command_buffer/service/gpu_processor.h
@@ -18,7 +18,6 @@
 #include "gpu/command_buffer/common/command_buffer.h"
 #include "gpu/command_buffer/service/cmd_buffer_engine.h"
 #include "gpu/command_buffer/service/cmd_parser.h"
-#include "gpu/command_buffer/service/context_group.h"
 #include "gpu/command_buffer/service/gles2_cmd_decoder.h"
 
 #if defined(OS_MACOSX)
@@ -30,12 +29,16 @@ class GLContext;
 }
 
 namespace gpu {
+namespace gles2 {
+class ContextGroup;
+}
 
 // This class processes commands in a command buffer. It is event driven and
 // posts tasks to the current message loop to do additional work.
 class GPUProcessor : public CommandBufferEngine {
  public:
-  explicit GPUProcessor(CommandBuffer* command_buffer);
+  // If a group is not passed in one will be created.
+  GPUProcessor(CommandBuffer* command_buffer, gles2::ContextGroup* group);
 
   // This constructor is for unit tests.
   GPUProcessor(CommandBuffer* command_buffer,
@@ -48,17 +51,11 @@ class GPUProcessor : public CommandBufferEngine {
   // Perform platform specific and common initialization.
   bool Initialize(gfx::PluginWindowHandle hwnd,
                   const gfx::Size& size,
+                  const char* allowed_extensions,
                   const std::vector<int32>& attribs,
                   GPUProcessor* parent,
                   uint32 parent_texture_id);
 
-  // Perform common initialization. Takes ownership of GLContext.
-  bool InitializeCommon(gfx::GLContext* context,
-                        const gfx::Size& size,
-                        const std::vector<int32>& attribs,
-                        gles2::GLES2Decoder* parent_decoder,
-                        uint32 parent_texture_id);
-
   void Destroy();
   void DestroyCommon();
 
@@ -96,6 +93,16 @@ class GPUProcessor : public CommandBufferEngine {
   // Get the GLES2Decoder associated with this processor.
   gles2::GLES2Decoder* decoder() const { return decoder_.get(); }
 
+ protected:
+  // Perform common initialization. Takes ownership of GLContext.
+  bool InitializeCommon(gfx::GLContext* context,
+                        const gfx::Size& size,
+                        const char* allowed_extensions,
+                        const std::vector<int32>& attribs,
+                        gles2::GLES2Decoder* parent_decoder,
+                        uint32 parent_texture_id);
+
+
  private:
   // Called via a callback just before we are supposed to call the
   // user's swap buffers callback.
@@ -108,9 +115,6 @@ class GPUProcessor : public CommandBufferEngine {
 
   int commands_per_update_;
 
-  // TODO(gman): Group needs to be passed in so it can be shared by
-  //    multiple GPUProcessors.
-  gles2::ContextGroup group_;
   scoped_ptr<gles2::GLES2Decoder> decoder_;
   scoped_ptr<CommandParser> parser_;