Make MockScreenOrientationClient an internal detail of components/test_runner.

components/test_runner/test_runner.h

Index: components/test_runner/test_runner.h
diff --git a/components/test_runner/test_runner.h b/components/test_runner/test_runner.h
index b671cee215287d1b1665da684844b895d6bed672..8abc27b50b8e286305bba22dbcd3ac8ba6265147 100644
--- a/components/test_runner/test_runner.h
+++ b/components/test_runner/test_runner.h
@@ -41,6 +41,7 @@ class Arguments;
 namespace test_runner {
 
 class InvokeCallbackTask;
+class MockScreenOrientationClient;
 class TestInterfaces;
 class WebContentSettings;
 class WebTestDelegate;
@@ -81,6 +82,7 @@ class TestRunner : public WebTestRunner,
 
   // Methods used by WebTestProxyBase.
   bool shouldStayOnPageAfterHandlingBeforeUnload() const;
+  MockScreenOrientationClient* getMockScreenOrientationClient();
   bool shouldDumpSelectionRect() const;
   bool isPrinting() const;
   bool shouldDumpAsTextWithPixelResults();
@@ -812,6 +814,8 @@ class TestRunner : public WebTestRunner,
   } pointer_lock_planned_result_;
   bool use_mock_theme_;
 
+  scoped_ptr<MockScreenOrientationClient> mock_screen_orientation_client_;

[Łukasz Anforowicz, 2016/03/16 00:16:01]

TestRunner's lifetime is longer than the lifetime of WebTestProxy so it seems
fine to move ownership of this mock here.  This is also consistent with the plan
to replace WebTestProxyBase with exposing borrower WebFrame/View/WidgetClient
pointers out of TestRunner (see
https://docs.google.com/document/d/1Q-1uc_vCcMhPiH0NViAEVzJmSJT8Cs2YT1QDaVNM6...).

+
   base::WeakPtrFactory<TestRunner> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TestRunner);