cc: Make separate interface for BeginFrame ipc from OutputSurface

content/browser/android/in_process/synchronous_compositor_factory_impl.cc

Index: content/browser/android/in_process/synchronous_compositor_factory_impl.cc
diff --git a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
index ef0c3ed557a495a2aa596b32f7ca2e999bebbaaf..c1f51811782c7ac7a9636f944de6ac89bebb3fc0 100644
--- a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
+++ b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
@@ -5,6 +5,8 @@
 #include "content/browser/android/in_process/synchronous_compositor_factory_impl.h"
 
 #include "base/observer_list.h"
+#include "cc/scheduler/begin_frame_source.h"
+#include "content/browser/android/in_process/synchronous_compositor_impl.h"
 #include "content/browser/android/in_process/synchronous_compositor_output_surface.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/renderer/gpu/frame_swap_message_queue.h"
@@ -189,6 +191,13 @@ SynchronousCompositorFactoryImpl::GetInputHandlerManagerClient() {
   return synchronous_input_event_filter();
 }
 
+scoped_ptr<cc::BeginFrameSource>
+SynchronousCompositorFactoryImpl::GetExternalBeginFrameSource(int routing_id) {
+  SynchronousCompositorImpl* compositor =
+      SynchronousCompositorImpl::FromRoutingID(routing_id);
+  return compositor->GetExternalBeginFrameSource();

[boliu, 2014/10/18 00:49:38]

This is not thread safe.

Maybe follow the OutputSurface pattern, have the factory create FrameSource, and
then FrameSource can try to find the SynchronousCompositorImpl once it's bound
to the compositor thread?

[simonhong, 2014/10/23 01:03:08]

Done.

+}
+
 scoped_refptr<ContextProviderWebContext>
 SynchronousCompositorFactoryImpl::CreateOffscreenContextProvider(
     const blink::WebGraphicsContext3D::Attributes& attributes,