Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(871)

Unified Diff: content/browser/web_contents/web_contents_impl.h

Issue 2391883006: Mojo-ify implementation of screen orientation locking/unlocking. (Closed)
Patch Set: Moved enum mojo definition to content/common due to dependency issue Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 e57408e5957da0e7f89969da009f5ba8c26c6600..1e6bafe51af6386e326432def2337b16dd1ad1d6 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -36,6 +36,7 @@
#include "content/public/browser/color_chooser.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
+#include "content/public/browser/screen_orientation_provider.h"
#include "content/public/browser/web_contents.h"
#include "content/public/browser/web_contents_observer.h"
#include "content/public/common/page_importance_signals.h"
@@ -73,7 +74,6 @@ class RenderViewHostDelegateView;
class RenderWidgetHostImpl;
class RenderWidgetHostInputEventRouter;
class SavePackage;
-class ScreenOrientationDispatcherHost;
class SiteInstance;
class TestWebContents;
class TextInputManager;
@@ -209,10 +209,6 @@ class CONTENT_EXPORT WebContentsImpl
WebContentsView* GetView() const;
- ScreenOrientationDispatcherHost* screen_orientation_dispatcher_host() {
- return screen_orientation_dispatcher_host_.get();
- }
-
bool should_normally_be_visible() { return should_normally_be_visible_; }
// Indicate if the window has been occluded, and pass this to the views, only
@@ -491,6 +487,7 @@ class CONTENT_EXPORT WebContentsImpl
int browser_plugin_instance_id) override;
device::GeolocationServiceContext* GetGeolocationServiceContext() override;
device::WakeLockServiceContext* GetWakeLockServiceContext() override;
+ ScreenOrientationProvider* GetScreenOrientationProvider() override;
void EnterFullscreenMode(const GURL& origin) override;
void ExitFullscreenMode(bool will_cause_resize) override;
bool ShouldRouteMessageEvent(
@@ -1408,8 +1405,7 @@ class CONTENT_EXPORT WebContentsImpl
std::unique_ptr<device::WakeLockServiceContext> wake_lock_service_context_;
- std::unique_ptr<ScreenOrientationDispatcherHost>
- screen_orientation_dispatcher_host_;
+ std::unique_ptr<ScreenOrientationProvider> screen_orientation_provider_;
std::unique_ptr<ManifestManagerHost> manifest_manager_host_;

Powered by Google App Engine
This is Rietveld 408576698