Move synchronous compositor into content/browser

content/browser/android/in_process/synchronous_compositor_output_surface.h

Index: content/browser/android/in_process/synchronous_compositor_output_surface.h
diff --git a/content/renderer/android/synchronous_compositor_output_surface.h b/content/browser/android/in_process/synchronous_compositor_output_surface.h
similarity index 75%
rename from content/renderer/android/synchronous_compositor_output_surface.h
rename to content/browser/android/in_process/synchronous_compositor_output_surface.h
index c52eca9b09da0c961c1256e6e9aa190623c57487..66cd227955aa6cbd6bafe9bbc7a6449a8cfb417a 100644
--- a/content/renderer/android/synchronous_compositor_output_surface.h
+++ b/content/browser/android/in_process/synchronous_compositor_output_surface.h
@@ -2,48 +2,46 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CONTENT_RENDERER_ANDROID_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_
-#define CONTENT_RENDERER_ANDROID_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_
+#ifndef CONTENT_BROWSER_ANDROID_IN_PROCESS_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_
+#define CONTENT_BROWSER_ANDROID_IN_PROCESS_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "cc/output/output_surface.h"
-#include "content/public/renderer/android/synchronous_compositor.h"
+#include "content/public/browser/android/synchronous_compositor.h"
 
 namespace content {
 
 class SynchronousCompositorClient;
-class SynchronousCompositorOutputSurfaceDelegate;
+class SynchronousCompositorOutputSurface;
 class WebGraphicsContext3DCommandBufferImpl;
 
 class SynchronousCompositorOutputSurfaceDelegate {
  public:
+   virtual void DidBindOutputSurface(
+      SynchronousCompositorOutputSurface* output_surface) = 0;
+  virtual void DidDestroySynchronousOutputSurface(
+      SynchronousCompositorOutputSurface* output_surface) = 0;
   virtual void SetContinuousInvalidate(bool enable) = 0;
-  virtual void DidCreateSynchronousOutputSurface() = 0;
-  virtual void DidDestroySynchronousOutputSurface() = 0;
 
  protected:
   SynchronousCompositorOutputSurfaceDelegate() {}
   virtual ~SynchronousCompositorOutputSurfaceDelegate() {}
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(SynchronousCompositorOutputSurfaceDelegate);
 };
 
 // Specialization of the output surface that adapts it to implement the
 // content::SynchronousCompositor public API. This class effects an "inversion
 // of control" - enabling drawing to be  orchestrated by the embedding
 // layer, instead of driven by the compositor internals - hence it holds two
-// 'client' pointers (|client_| in the OutputSurface baseclass and |delegate_|)
-// which represent the consumers of the two roles in plays.
+// 'client' pointers (|client_| in the OutputSurface baseclass and
+// GetDelegate()) which represent the consumers of the two roles in plays.
 // This class can be created only on the main thread, but then becomes pinned
 // to a fixed thread when BindToClient is called.
 class SynchronousCompositorOutputSurface
     : NON_EXPORTED_BASE(public cc::OutputSurface) {
  public:
-  explicit SynchronousCompositorOutputSurface(
-      SynchronousCompositorOutputSurfaceDelegate* delegate);
+  explicit SynchronousCompositorOutputSurface(int routing_id);
   virtual ~SynchronousCompositorOutputSurface();
 
   // OutputSurface.
@@ -69,8 +67,9 @@ class SynchronousCompositorOutputSurface
   void UpdateCompositorClientSettings();
   void NotifyCompositorSettingsChanged();
   bool CalledOnValidThread() const;
+  SynchronousCompositorOutputSurfaceDelegate* GetDelegate();

[mkosiba (inactive), 2013/05/30 11:05:29]

uber-nit: might be fine to call this delegate(). I think the style guide says
you can use this naming convention for simple&cheap getters (which this one is)
and I don't think this naming convention is limited to inline functions only.

[joth, 2013/05/30 19:18:05]

I don't like hacker_case() methods calling CamelCase() methods internally. Any
this does do a map lookup, not not entirely as cheap as returning a member
(which hacker_case() is intended for).

 
-  SynchronousCompositorOutputSurfaceDelegate* delegate_;
+  int routing_id_;
   bool needs_begin_frame_;
   bool did_swap_buffer_;
 
@@ -82,4 +81,4 @@ class SynchronousCompositorOutputSurface
 
 }  // namespace content
 
-#endif  // CONTENT_RENDERER_ANDROID_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_
+#endif  // CONTENT_BROWSER_ANDROID_IN_PROCESS_SYNCHRONOUS_COMPOSITOR_OUTPUT_SURFACE_H_