[ScreenOrientation] Merge ScreenOrientation into ScreenOrientationProvider.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 95a45927753fc3a64f770e520c67a380f60e3c38..365618f806c699ab79c8061009e8688f46b8fd19 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -73,7 +73,6 @@
 #include "content/browser/renderer_host/render_widget_host_input_event_router.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #include "content/browser/renderer_host/text_input_manager.h"
-#include "content/browser/screen_orientation/screen_orientation.h"
 #include "content/browser/site_instance_impl.h"
 #include "content/browser/web_contents/web_contents_view_child_frame.h"
 #include "content/browser/web_contents/web_contents_view_guest.h"
@@ -926,6 +925,11 @@ WebContentsView* WebContentsImpl::GetView() const {
   return view_.get();
 }
 
+ScreenOrientationProvider* WebContentsImpl::GetScreenOrientationProvider()
+    const {

[mlamouri (slow - plz ping), 2017/01/28 02:17:51]

curiosity (not the spaceship): is `git cl format` producing this wrapping? I
would expect something with the return type in its own line

[leonhsl(Using Gerrit), 2017/02/04 03:13:45]

Yeah this is produced by 'git cl format'. I changed to let the return type be in
one separate line but 'git cl format' will restore it again. So I think we'd
better follow 'git cl format' otherwise we may need to care about this every
time when we change here in future.

+  return screen_orientation_provider_.get();
+}
+
 SkColor WebContentsImpl::GetThemeColor() const {
   return theme_color_;
 }
@@ -1586,7 +1590,7 @@ void WebContentsImpl::Init(const WebContents::CreateParams& params) {
                  NOTIFICATION_RENDER_WIDGET_HOST_DESTROYED,
                  NotificationService::AllBrowserContextsAndSources());
 
-  screen_orientation_.reset(new ScreenOrientation(this));
+  screen_orientation_provider_.reset(new ScreenOrientationProvider(this));
 
   manifest_manager_host_.reset(new ManifestManagerHost(this));
 
@@ -2429,10 +2433,6 @@ device::WakeLockServiceContext* WebContentsImpl::GetWakeLockServiceContext() {
   return wake_lock_service_context_.get();
 }
 
-ScreenOrientationProvider* WebContentsImpl::GetScreenOrientationProvider() {
-  return screen_orientation_.get()->GetScreenOrientationProvider();
-}
-
 void WebContentsImpl::OnShowValidationMessage(
     RenderViewHostImpl* source,
     const gfx::Rect& anchor_in_root_view,