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

Unified Diff: chrome/browser/ui/fullscreen_controller.cc

Issue 10173030: User Gesture plumbed from Renderer to Browser's FullscreenController::RequestToLockMouse. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: yzshen nits fixed. Created 8 years, 8 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
« no previous file with comments | « chrome/browser/ui/fullscreen_controller.h ('k') | content/browser/renderer_host/render_view_host_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/fullscreen_controller.cc
diff --git a/chrome/browser/ui/fullscreen_controller.cc b/chrome/browser/ui/fullscreen_controller.cc
index f0e3514b5fca94014353dde3307c42de4d0925e6..80aeaf04abedaba4ab90c82a7bd52b8499ba7fe2 100644
--- a/chrome/browser/ui/fullscreen_controller.cc
+++ b/chrome/browser/ui/fullscreen_controller.cc
@@ -59,7 +59,11 @@ bool FullscreenController::IsMouseLockedOrPending() const {
return mouse_lock_state_ != MOUSELOCK_NOT_REQUESTED;
}
-void FullscreenController::RequestToLockMouse(WebContents* tab) {
+void FullscreenController::RequestToLockMouse(WebContents* tab,
+ bool /* user_gesture */) {
+ // TODO(scheib) user_gesture required for Mouse Lock in Windowed Mode.
+ // See http://crbug.com/107013, which will land in multiple patches.
+
// Mouse Lock is only permitted when browser is in tab fullscreen.
if (!IsFullscreenForTabOrPending(tab)) {
tab->GotResponseToLockMouseRequest(false);
« no previous file with comments | « chrome/browser/ui/fullscreen_controller.h ('k') | content/browser/renderer_host/render_view_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698