Have a common implementation of cc::OutputSurface

cc/test/fake_output_surface.h

Index: cc/test/fake_output_surface.h
diff --git a/cc/test/fake_output_surface.h b/cc/test/fake_output_surface.h
index a2dd67cdbff7bfa08475dc794668cc8089cda5ac..3541f0b631dee7a2820430a40b26298ba3d71dcd 100644
--- a/cc/test/fake_output_surface.h
+++ b/cc/test/fake_output_surface.h
@@ -5,8 +5,6 @@
 #ifndef CC_TEST_FAKE_OUTPUT_SURFACE_H_
 #define CC_TEST_FAKE_OUTPUT_SURFACE_H_
 
-#include "base/logging.h"
-#include "base/memory/scoped_ptr.h"
 #include "cc/compositor_frame.h"
 #include "cc/output_surface.h"
 #include "cc/test/fake_software_output_device.h"
@@ -19,12 +17,12 @@ class FakeOutputSurface : public OutputSurface {
  public:
   virtual ~FakeOutputSurface();
 
-  static inline scoped_ptr<FakeOutputSurface> Create3d(
+  static scoped_ptr<FakeOutputSurface> Create3d(
       scoped_ptr<WebKit::WebGraphicsContext3D> context3d) {
     return make_scoped_ptr(new FakeOutputSurface(context3d.Pass(), false));
   }
 
-  static inline scoped_ptr<FakeOutputSurface> Create3d() {
+  static scoped_ptr<FakeOutputSurface> Create3d() {
     scoped_ptr<WebKit::WebGraphicsContext3D> context3d =
         FakeWebGraphicsContext3D::Create(
             WebKit::WebGraphicsContext3D::Attributes())
@@ -32,18 +30,18 @@ class FakeOutputSurface : public OutputSurface {
     return make_scoped_ptr(new FakeOutputSurface(context3d.Pass(), false));
   }
 
-  static inline scoped_ptr<FakeOutputSurface> CreateSoftware(
+  static scoped_ptr<FakeOutputSurface> CreateSoftware(
       scoped_ptr<SoftwareOutputDevice> software_device) {
     return make_scoped_ptr(
         new FakeOutputSurface(software_device.Pass(), false));
   }
 
-  static inline scoped_ptr<FakeOutputSurface> CreateDelegating3d(
+  static scoped_ptr<FakeOutputSurface> CreateDelegating3d(
       scoped_ptr<WebKit::WebGraphicsContext3D> context3d) {
     return make_scoped_ptr(new FakeOutputSurface(context3d.Pass(), true));
   }
 
-  static inline scoped_ptr<FakeOutputSurface> CreateDelegating3d() {
+  static scoped_ptr<FakeOutputSurface> CreateDelegating3d() {
     scoped_ptr<WebKit::WebGraphicsContext3D> context3d =
         FakeWebGraphicsContext3D::Create(
             WebKit::WebGraphicsContext3D::Attributes())
@@ -51,45 +49,34 @@ class FakeOutputSurface : public OutputSurface {
     return make_scoped_ptr(new FakeOutputSurface(context3d.Pass(), true));
   }
 
-  static inline scoped_ptr<FakeOutputSurface> CreateDelegatingSoftware(
+  static scoped_ptr<FakeOutputSurface> CreateDelegatingSoftware(
       scoped_ptr<SoftwareOutputDevice> software_device) {
     return make_scoped_ptr(
         new FakeOutputSurface(software_device.Pass(), true));
   }
 
-  virtual bool BindToClient(OutputSurfaceClient* client) OVERRIDE;
-
-  virtual const struct Capabilities& Capabilities() const OVERRIDE;
-
-  virtual WebKit::WebGraphicsContext3D* Context3D() const OVERRIDE;
-  virtual SoftwareOutputDevice* SoftwareDevice() const OVERRIDE;
-
   virtual void SendFrameToParentCompositor(CompositorFrame*) OVERRIDE;
 
   CompositorFrame& last_sent_frame() { return last_sent_frame_; }
   size_t num_sent_frames() { return num_sent_frames_; }
 
 private:
-  explicit FakeOutputSurface(
-      scoped_ptr<WebKit::WebGraphicsContext3D> context3d, bool has_parent);
-  explicit FakeOutputSurface(
-      scoped_ptr<SoftwareOutputDevice> software_device, bool has_parent);
-
-  scoped_ptr<WebKit::WebGraphicsContext3D> context3d_;
-  scoped_ptr<SoftwareOutputDevice> software_device_;
-  struct Capabilities capabilities_;
-  OutputSurfaceClient* client_;
+  FakeOutputSurface(scoped_ptr<WebKit::WebGraphicsContext3D> context3d,

[danakj, 2013/01/24 02:03:06]

nit: line wrap at the ( please

+                    bool has_parent);
+
+  FakeOutputSurface(scoped_ptr<SoftwareOutputDevice> software_device,

[danakj, 2013/01/24 02:03:06]

same

+                    bool has_parent);
+
   CompositorFrame last_sent_frame_;
   size_t num_sent_frames_;
 };
 
-static inline scoped_ptr<cc::OutputSurface> createFakeOutputSurface()
-{
-    return FakeOutputSurface::Create3d(
-        FakeWebGraphicsContext3D::Create(
-            WebKit::WebGraphicsContext3D::Attributes())
-        .PassAs<WebKit::WebGraphicsContext3D>())
-        .PassAs<cc::OutputSurface>();
+static inline scoped_ptr<cc::OutputSurface> createFakeOutputSurface() {
+  return FakeOutputSurface::Create3d(
+      FakeWebGraphicsContext3D::Create(
+          WebKit::WebGraphicsContext3D::Attributes())
+      .PassAs<WebKit::WebGraphicsContext3D>())
+      .PassAs<cc::OutputSurface>();
 }
 
 }  // namespace cc