Hook up BeginFrameSource to SurfaceFactoryClient via SurfaceManager

android_webview/browser/hardware_renderer.cc

Index: android_webview/browser/hardware_renderer.cc
diff --git a/android_webview/browser/hardware_renderer.cc b/android_webview/browser/hardware_renderer.cc
index 5688b1ca0580aa3a5553129eea75c72f64171580..5bf913fb1b6e39bfbfb7faa3d98ba0723b2a3afd 100644
--- a/android_webview/browser/hardware_renderer.cc
+++ b/android_webview/browser/hardware_renderer.cc
@@ -53,6 +53,8 @@ HardwareRenderer::HardwareRenderer(SharedRendererState* state)
 
   surface_manager_.reset(new cc::SurfaceManager);
   surface_id_allocator_.reset(new cc::SurfaceIdAllocator(1));
+  surface_manager_->RegisterSurfaceFactoryClient(
+      surface_id_allocator_.id_namespace(), this);
   display_.reset(new cc::Display(this, surface_manager_.get(), nullptr, nullptr,
                                  settings));
 }
@@ -66,6 +68,8 @@ HardwareRenderer::~HardwareRenderer() {
     surface_factory_->Destroy(child_id_);
   display_.reset();
   surface_factory_.reset();
+  surface_manager_->RegisterSurfaceFactoryClient(
+      surface_id_allocator_.id_namespace(), nullptr);
 
   // Reset draw constraints.
   shared_renderer_state_->PostExternalDrawConstraintsToChildCompositorOnRT(
@@ -222,7 +226,6 @@ void HardwareRenderer::ReturnResources(
 }
 
 void HardwareRenderer::SetBeginFrameSource(
-    cc::SurfaceId surface_id,
     cc::BeginFrameSource* begin_frame_source) {
   // TODO(tansell): Hook this up.
 }