[ScreenOrientation] Hide ScreenOrientationProvider inside WebContentsImpl.

content/browser/web_contents/web_contents_impl.h

Index: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index 4ece254c6d035529a25871133e49fcaa9dbe8342..2ae1f97af68db937d0c4ed6044472212142b1ce6 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -209,7 +209,11 @@ class CONTENT_EXPORT WebContentsImpl
 
   WebContentsView* GetView() const;
 
-  ScreenOrientationProvider* GetScreenOrientationProvider() const;
+  void OnScreenOrientationChange();
+
+  ScreenOrientationProvider* screen_orientation_provider_for_testing() const {

[kinuko, 2017/02/09 05:57:38]

This name sounds like it's returning mock/testing-only screen orientation
provider or something (as lower_case() inline accessors usually correspond to
lower_case_ variables).  GetScreenOrientationProviderForTesting() would be
clearer?

[leonhsl(Using Gerrit), 2017/02/09 08:32:06]

Got it and done.

+    return screen_orientation_provider_.get();
+  }
 
   bool should_normally_be_visible() { return should_normally_be_visible_; }