Android Browser Compositor: Add ScheduleComposite() callback.

content/shell/android/shell_manager.cc

 // 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.
 
 #include "content/shell/android/shell_manager.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/message_loop.h"
 #include "content/shell/shell.h"
 #include "content/shell/shell_browser_context.h"
 #include "content/shell/shell_content_browser_client.h"
 #include "content/public/browser/android/compositor.h"
 #include "content/public/browser/android/draw_delegate.h"
 #include "content/public/browser/web_contents.h"
 #include "content/shell/shell.h"
 #include "googleurl/src/gurl.h"
 #include "jni/ShellManager_jni.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebLayer.h"
 #include "ui/gfx/size.h"
 
 #include <android/native_window_jni.h>
 
 using base::android::ScopedJavaLocalRef;
 using content::Compositor;
 using content::DrawDelegate;
 
 namespace {
 
+class CompositorClient : public Compositor::Client {
+ public:
+  virtual void ScheduleComposite() OVERRIDE;
+};
+
 struct GlobalState {
   base::android::ScopedJavaGlobalRef<jobject> j_obj;
+  CompositorClient client;
   scoped_ptr<content::Compositor> compositor;
   scoped_ptr<WebKit::WebLayer> root_layer;
 };
 
 base::LazyInstance<GlobalState> g_global_state = LAZY_INSTANCE_INITIALIZER;
 
 content::Compositor* GetCompositor() {
   return g_global_state.Get().compositor.get();
 }
 
-static void SurfacePresented(
-    const DrawDelegate::SurfacePresentedCallback& callback,
-    uint32 sync_point) {
-  callback.Run(sync_point);
+bool g_scheduled_composite = false;

[piman, 2012/10/13 18:04:21]

nit: should this be in the global state?

+void Composite() {
+  g_scheduled_composite = false;
+  if (GetCompositor()) {
+    GetCompositor()->Composite();
+  }
 }
 
-static void SurfaceUpdated(
-    uint64 texture,
-    content::RenderWidgetHostView* view,
-    const DrawDelegate::SurfacePresentedCallback& callback) {
-  GetCompositor()->OnSurfaceUpdated(base::Bind(
-      &SurfacePresented, callback));
+void CompositorClient::ScheduleComposite() {
+  if (!g_scheduled_composite) {
+    g_scheduled_composite = true;
+    MessageLoop::current()->PostTask(FROM_HERE, base::Bind(&Composite));
+  }
 }
 
 } // anonymous namespace
 
 namespace content {
 
 jobject CreateShellView() {
   JNIEnv* env = base::android::AttachCurrentThread();
   if (!GetCompositor()) {
     Compositor::Initialize();
-    g_global_state.Get().compositor.reset(Compositor::Create());
+    g_global_state.Get().compositor.reset(Compositor::Create(
+        &g_global_state.Get().client));
     DCHECK(!g_global_state.Get().root_layer.get());
     g_global_state.Get().root_layer.reset(WebKit::WebLayer::create());
   }
   return Java_ShellManager_createShell(
       env, g_global_state.Get().j_obj.obj()).Release();
 }
 
 // Register native methods
 bool RegisterShellManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 static void Init(JNIEnv* env, jclass clazz, jobject obj) {
   g_global_state.Get().j_obj.Reset(
       base::android::ScopedJavaLocalRef<jobject>(env, obj));
-  DrawDelegate::SurfaceUpdatedCallback cb = base::Bind(
-      &SurfaceUpdated);
-  DrawDelegate::GetInstance()->SetUpdateCallback(cb);
 }
 
 static void SurfaceCreated(
     JNIEnv* env, jclass clazz, jobject jsurface) {
   ANativeWindow* native_window = ANativeWindow_fromSurface(env, jsurface);
   if (native_window) {
     GetCompositor()->SetWindowSurface(native_window);
     ANativeWindow_release(native_window);
     GetCompositor()->SetRootLayer(g_global_state.Get().root_layer.get());
   }
@@ skipping 25 matching lines @@
 void ShellAttachLayer(WebKit::WebLayer* layer) {
   g_global_state.Get().root_layer->addChild(layer);
 }
 
 
 void ShellRemoveLayer(WebKit::WebLayer* layer) {
   layer->removeFromParent();
 }
 
 }  // namespace content