Factor out cc::OutputSurface::InitializeAndSetContext3D

cc/output/output_surface.h

Index: cc/output/output_surface.h
diff --git a/cc/output/output_surface.h b/cc/output/output_surface.h
index 00877de68c8f8baa72c8113f87982225b8be2fa2..5961dd8f406aca3a25aca6341da7c09769aa15a3 100644
--- a/cc/output/output_surface.h
+++ b/cc/output/output_surface.h
@@ -6,8 +6,10 @@
 #define CC_OUTPUT_OUTPUT_SURFACE_H_
 
 #include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "cc/base/cc_export.h"
+#include "cc/output/context_provider.h"
 #include "cc/output/software_output_device.h"
 #include "third_party/WebKit/public/platform/WebGraphicsContext3D.h"
 
@@ -106,17 +108,21 @@ class CC_EXPORT OutputSurface {
   virtual void SetNeedsBeginFrame(bool enable) {}
 
  protected:
+  bool InitializeAndSetContext3D(

[joth, 2013/06/07 01:02:14]

document?
esp. is context_provider for offscreen (impl thread only?) context?

[danakj, 2013/06/07 14:26:09]

Agree, offscreen_context_provider is the name used for this variable elsewhere.

[boliu, 2013/06/07 16:03:44]

Done.

+      scoped_ptr<WebKit::WebGraphicsContext3D> context3d,
+      scoped_refptr<ContextProvider> context_provider);
+
   OutputSurfaceClient* client_;
   struct cc::OutputSurface::Capabilities capabilities_;
+  scoped_ptr<OutputSurfaceCallbacks> callbacks_;
   scoped_ptr<WebKit::WebGraphicsContext3D> context3d_;
   scoped_ptr<cc::SoftwareOutputDevice> software_device_;
   bool has_gl_discard_backbuffer_;
   gfx::Size surface_size_;
   float device_scale_factor_;
 
-  scoped_ptr<OutputSurfaceCallbacks> callbacks_;
-
  private:
+  void SetContext3D(scoped_ptr<WebKit::WebGraphicsContext3D> context3d);
   DISALLOW_COPY_AND_ASSIGN(OutputSurface);
 };