Adding compositor callbacks to RenderWidgetHostViewAndroid

content/public/browser/android/compositor.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_
 #define CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_
 
 #include "base/callback.h"
 #include "cc/resources/ui_resource_bitmap.h"
 #include "cc/resources/ui_resource_client.h"
 #include "content/common/content_export.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/rect.h"
 #include "ui/gfx/size.h"
 
 #include "third_party/WebKit/public/platform/WebGraphicsContext3D.h"
 
 namespace cc {
 class Layer;
 }
 
 namespace gfx {
 class JavaBitmap;
 }
 
+namespace ui {
+class WindowAndroid;
+}
+
 namespace content {
 class CompositorClient;
 
 // An interface to the browser-side compositor.
 class CONTENT_EXPORT Compositor {
  public:
   virtual ~Compositor() {}
 
   // Performs the global initialization needed before any compositor
   // instance can be used. This should be called only once.
   static void Initialize();
 
   // Creates and returns a compositor instance.
-  static Compositor* Create(CompositorClient* client);
+  static Compositor* Create(CompositorClient* client,
+                            ui::WindowAndroid* window_android);

[no sievers, 2013/11/14 22:01:01]

nit: |root_window|? Can you also use the common typedef 'NativeWindow' from
native_widget_types.h here and everywhere else in content/ just so that it's
more obvious what it is?

Can you also document that this needs to outlive the compositor?

[powei, 2013/11/15 02:18:26]

Done.

 
   // Attaches the layer tree.
   virtual void SetRootLayer(scoped_refptr<cc::Layer> root) = 0;
 
   // Set the scale factor from DIP to pixel.
   virtual void setDeviceScaleFactor(float factor) = 0;
 
   // Set the output surface bounds.
   virtual void SetWindowBounds(const gfx::Size& size) = 0;
 
@@ skipping 49 matching lines @@
   virtual bool CopyTextureToBitmap(blink::WebGLId texture_id,
                                    const gfx::Rect& src_rect,
                                    gfx::JavaBitmap& bitmap) = 0;
  protected:
   Compositor() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_ANDROID_COMPOSITOR_H_