Only set MouseLockDispatcher unlocked_by_target_ after mouse lock confirmed.

content/renderer/mouse_lock_dispatcher.cc

Index: content/renderer/mouse_lock_dispatcher.cc
diff --git a/content/renderer/mouse_lock_dispatcher.cc b/content/renderer/mouse_lock_dispatcher.cc
index dd2326aba77661ac879acc93c799346e23fc8db1..d1d5edbf1860fa2693f557a93ee493ffaddf98ea 100644
--- a/content/renderer/mouse_lock_dispatcher.cc
+++ b/content/renderer/mouse_lock_dispatcher.cc
@@ -44,7 +44,14 @@ bool MouseLockDispatcher::LockMouse(LockTarget* target) {
 void MouseLockDispatcher::UnlockMouse(LockTarget* target) {
   if (target && target == target_ && !pending_unlock_request_) {
     pending_unlock_request_ = true;
-    unlocked_by_target_ = true;
+
+    // When a target application voluntarily unlocks the mouse we permit

[yzshen1, 2012/06/04 21:40:30]

[I don't know the exact answer, but would like to make sure that you have
considered it.]
According to the code, we are assuming that a MouseLockDispatcher object will
only be used by one "application": we don't distinguish target that sets
|unlocked_by_target_| and the one that uses it. Is that intended? Can we
guarantee that?

[scheib, 2012/06/04 23:25:40]

I considered this when designing the first patch. A web page is composing code
and plugins that may call this API. Before and after this new feature (silent
acceptance) code on a page is able to fight with other code on the page after
the user has accepted mouse lock the first time. The difference is that
previously it would fight only on user gestures, and the user would see exit
instructions.

I don't think we can put protections in place to fix poorly composed pages with
multiple sources trying to use mouse lock. It is similar to fullscreen, or
audio, or other apis: a poorly composed page can result in a poor experience.

+    // relocking the mouse silently and with no user gesture requirement.
+    // Check that the lock request is not currently pending and not yet
+    // accepted by the browser process before setting |unlocked_by_target_|.
+    if (!pending_lock_request_)
+      unlocked_by_target_ = true;
+
     Send(new ViewHostMsg_UnlockMouse(routing_id()));
   }
 }