Make MockScreenOrientationClient an internal detail of components/test_runner.

components/test_runner/test_runner.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_TEST_RUNNER_TEST_RUNNER_H_
 #define COMPONENTS_TEST_RUNNER_TEST_RUNNER_H_
 
 #include <stdint.h>
 
 #include <deque>
@@ skipping 23 matching lines @@
 }
 
 namespace gin {
 class ArrayBufferView;
 class Arguments;
 }
 
 namespace test_runner {
 
 class InvokeCallbackTask;
+class MockScreenOrientationClient;
 class TestInterfaces;
 class WebContentSettings;
 class WebTestDelegate;
 class WebTestProxyBase;
 
 class TestRunner : public WebTestRunner,
                    public base::SupportsWeakPtr<TestRunner> {
  public:
   explicit TestRunner(TestInterfaces*);
   virtual ~TestRunner();
@@ skipping 20 matching lines @@
   void GetAudioData(std::vector<unsigned char>* buffer_view) const override;
   const LayoutDumpFlags& GetLayoutDumpFlags() override;
   void ReplicateLayoutDumpFlagsChanges(
       const base::DictionaryValue& changed_values) override;
   bool HasCustomTextDump(std::string* custom_text_dump) const override;
   bool ShouldDumpBackForwardList() const override;
   blink::WebContentSettingsClient* GetWebContentSettings() const override;
 
   // Methods used by WebTestProxyBase.
   bool shouldStayOnPageAfterHandlingBeforeUnload() const;
+  MockScreenOrientationClient* getMockScreenOrientationClient();
   bool shouldDumpSelectionRect() const;
   bool isPrinting() const;
   bool shouldDumpAsTextWithPixelResults();
   bool shouldDumpAsCustomText() const;
   std:: string customDumpText() const;
   void ShowDevTools(const std::string& settings,
                     const std::string& frontend_url);
   void ClearDevToolsLocalStorage();
   void setShouldDumpAsText(bool);
   void setShouldDumpAsMarkup(bool);
@@ skipping 711 matching lines @@
   scoped_ptr<WebContentSettings> web_content_settings_;
 
   bool pointer_locked_;
   enum {
     PointerLockWillSucceed,
     PointerLockWillRespondAsync,
     PointerLockWillFailSync,
   } 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);
 };
 
 }  // namespace test_runner
 
 #endif  // COMPONENTS_TEST_RUNNER_TEST_RUNNER_H_