Exit mouse lock permision prompt on tab switch or close.

chrome/browser/ui/fullscreen_controller_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/compiler_specific.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/fullscreen_controller_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/url_constants.h"
+#include "content/public/test/test_navigation_observer.h"
 #if defined(OS_MACOSX)
 #include "base/mac/mac_util.h"
 #endif
 
 using content::WebContents;
 
 namespace {
 
 const FilePath::CharType* kSimpleFile = FILE_PATH_LITERAL("simple.html");
 
@@ skipping 306 matching lines @@
     // Test that tab fullscreen mode doesn't make presentation mode the default
     // on Lion.
     FullscreenNotificationObserver fullscreen_observer;
     browser()->ToggleFullscreenMode();
     fullscreen_observer.Wait();
     ASSERT_TRUE(browser()->window()->IsFullscreen());
     ASSERT_FALSE(browser()->window()->InPresentationMode());
   }
 }
 #endif
+
+IN_PROC_BROWSER_TEST_F(FullscreenControllerTest,
+                       PendingMouseLockExitsOnTabSwitch) {
+  AddTabAtIndexAndWait(0, GURL(chrome::kAboutBlankURL),
+                       content::PAGE_TRANSITION_TYPED);
+  WebContents* tab2 = browser()->GetActiveWebContents();
+  AddTabAtIndexAndWait(0, GURL(chrome::kAboutBlankURL),
+                       content::PAGE_TRANSITION_TYPED);
+  WebContents* tab1 = browser()->GetActiveWebContents();
+
+  // Request mouse lock. Bubble is displayed.
+  RequestToLockMouse(tab1, true, false);
+  ASSERT_TRUE(IsFullscreenBubbleDisplayed());
+
+  // Activate current tab. Mouse lock bubble remains.
+  browser()->ActivateTabAt(0, true);
+  ASSERT_TRUE(IsFullscreenBubbleDisplayed());
+
+  // Activate tab2. Mouse lock bubble clears.
+  {
+    MouseLockNotificationObserver mouse_lock_observer;
+    browser()->ActivateTabAt(1, true);
+    mouse_lock_observer.Wait();
+  }
+  ASSERT_FALSE(IsFullscreenBubbleDisplayed());
+
+  // Now, test that closing an unrelated tab does not disturb a request.
+
+  // Request mouse lock. Bubble is displayed.
+  RequestToLockMouse(tab2, true, false);
+  ASSERT_TRUE(IsFullscreenBubbleDisplayed());
+
+  // Close tab1. Mouse lock bubble remains.
+  browser()->CloseTabContents(tab1);
+  ASSERT_TRUE(IsFullscreenBubbleDisplayed());
+}
+
+IN_PROC_BROWSER_TEST_F(FullscreenControllerTest,

[yzshen1, 2012/06/22 22:18:04]

nit: it might be useful to also test another case: the tab with mouse lock
bubble is not the last tab in the window to close.

[scheib, 2012/06/22 22:27:16]

Done.

Acutlaly, there were already two tabs in this test. The new tab exists, and then
this test explicitly adds a BlankURL tab. However, adding a 3rd won't hurt, and
will ease readability in case anyone has the same concern.

+                       PendingMouseLockExitsOnTabClose) {
+  AddTabAtIndexAndWait(0, GURL(chrome::kAboutBlankURL),
+                       content::PAGE_TRANSITION_TYPED);
+
+  // Request mouse lock. Bubble is displayed.
+  RequestToLockMouse(browser()->GetActiveWebContents(), true, false);
+  ASSERT_TRUE(IsFullscreenBubbleDisplayed());
+
+  // Close tab. Bubble is cleared.
+  {
+    MouseLockNotificationObserver mouse_lock_observer;
+    browser()->CloseTab();
+    mouse_lock_observer.Wait();
+  }
+  ASSERT_FALSE(IsFullscreenBubbleDisplayed());
+}