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..fc1d3c941f0fd7303aa7bdbdcc9259541b165075 100644
--- a/android_webview/browser/test/rendering_test.cc
+++ b/android_webview/browser/test/rendering_test.cc
@@ -16,6 +16,30 @@
 
 namespace android_webview {
 
+namespace {
+// BrowserViewRenderer subclass used for enabling tests to observe
+// OnParentDrawConstraintsUpdated.
+class BrowserViewRendererTest : public BrowserViewRenderer {

[boliu, 2016/04/21 18:14:11]

TestFoo, not FooTest. FooTest means it's the test itself.

[Tobias Sargeant, 2016/04/22 11:42:22]

Done.

+ 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() override {
+    BrowserViewRenderer::OnParentDrawConstraintsUpdated();
+    rendering_test_->OnParentDrawConstraintsUpdated();
+  }
+
+ private:
+  RenderingTest* rendering_test_;

[boliu, 2016/04/21 18:14:11]

* const

[Tobias Sargeant, 2016/04/22 11:42:22]

Done.

+};
+}
+
 RenderingTest::RenderingTest() : message_loop_(new base::MessageLoop) {
   ui_task_runner_ = base::ThreadTaskRunnerHandle::Get();
 }
@@ -30,9 +54,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());

[boliu, 2016/04/21 18:14:11]

hmm, this totally gives us the test ability to put in a fake consumer

[Tobias Sargeant, 2016/04/22 11:42:22]

Acknowledged.

   InitializeCompositor();
   Attach();
 }
@@ -114,10 +139,6 @@ void RenderingTest::PostInvalidate() {
     window_->PostInvalidate();
 }
 
-void RenderingTest::OnParentDrawConstraintsUpdated() {
-  browser_view_renderer_->OnParentDrawConstraintsUpdated();
-}
-
 void RenderingTest::DetachFunctorFromView() {
 }