Refactor BrowserViewRenderer-RenderThreadManager relationship.

android_webview/browser/test/rendering_test.cc

Index: android_webview/browser/test/rendering_test.cc
diff --git a/android_webview/browser/test/rendering_test.cc b/android_webview/browser/test/rendering_test.cc
index 0149400214eac752ccbb3c632682f3a2c04a0e82..93aeabb94f00858d79343763aba08dbd0e14add2 100644
--- a/android_webview/browser/test/rendering_test.cc
+++ b/android_webview/browser/test/rendering_test.cc
@@ -16,6 +16,32 @@
 
 namespace android_webview {
 
+namespace {
+// BrowserViewRenderer subclass used for enabling tests to observe
+// OnParentDrawConstraintsUpdated.
+class BrowserViewRendererTest : public BrowserViewRenderer {
+ public:
+  BrowserViewRendererTest(
+      RenderingTest* rendering_test,
+      const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner,
+      bool disable_page_visibility)
+      : BrowserViewRenderer(rendering_test,
+                            ui_task_runner,
+                            disable_page_visibility),
+        rendering_test_(rendering_test) {}
+
+  void OnParentDrawConstraintsUpdated(
+      CompositorFrameConsumer* compositor_frame_consumer) override {
+    BrowserViewRenderer::OnParentDrawConstraintsUpdated(
+        compositor_frame_consumer);
+    rendering_test_->OnParentDrawConstraintsUpdated();
+  }
+
+ private:
+  RenderingTest* rendering_test_;
+};
+}
+
 RenderingTest::RenderingTest() : message_loop_(new base::MessageLoop) {
   ui_task_runner_ = base::ThreadTaskRunnerHandle::Get();
 }
@@ -30,9 +56,10 @@ void RenderingTest::SetUpTestHarness() {
   DCHECK(!render_thread_manager_.get());
   render_thread_manager_.reset(
       new RenderThreadManager(this, base::ThreadTaskRunnerHandle::Get()));
-  browser_view_renderer_.reset(new BrowserViewRenderer(
+  browser_view_renderer_.reset(new BrowserViewRendererTest(
       this, base::ThreadTaskRunnerHandle::Get(), false));
-  browser_view_renderer_->SetRenderThreadManager(render_thread_manager_.get());
+  browser_view_renderer_->SetCompositorFrameConsumer(
+      render_thread_manager_.get());
   InitializeCompositor();
   Attach();
 }
@@ -114,10 +141,6 @@ void RenderingTest::PostInvalidate() {
     window_->PostInvalidate();
 }
 
-void RenderingTest::OnParentDrawConstraintsUpdated() {
-  browser_view_renderer_->OnParentDrawConstraintsUpdated();
-}
-
 void RenderingTest::DetachFunctorFromView() {
 }