[PointerLock] Move "silent mouse lock" logic from renderer to RWHI

content/browser/browser_plugin/browser_plugin_guest.cc

Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index 8f26ef0690081c2eabc1add1f64474f37874be78..d853a4babcc698d7a223c8b3d855790f98bb7af9 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -935,7 +935,6 @@ void BrowserPluginGuest::OnExtendSelectionAndDelete(
 }
 
 void BrowserPluginGuest::OnLockMouse(bool user_gesture,
-                                     bool last_unlocked_by_target,
                                      bool privileged) {
   if (pending_lock_request_) {
     // Immediately reject the lock because only one pointerLock may be active
@@ -946,9 +945,12 @@ void BrowserPluginGuest::OnLockMouse(bool user_gesture,
 
   pending_lock_request_ = true;
 
+  RenderWidgetHostImpl* owner = GetOwnerRenderWidgetHost();
+  const bool is_last_unlocked_by_target =

[jochen (gone - plz use gerrit), 2017/06/06 14:59:47]

why const? that looks unusual..

[chongz, 2017/06/06 17:11:38]

Done. Hmmm I though we prefer using 'const', no...?

+      owner ? owner->is_last_unlocked_by_target() : false;
+
   delegate_->RequestPointerLockPermission(
-      user_gesture,
-      last_unlocked_by_target,
+      user_gesture, is_last_unlocked_by_target,
       base::Bind(&BrowserPluginGuest::PointerLockPermissionResponse,
                  weak_ptr_factory_.GetWeakPtr()));
 }