Mouse lock and fullscreen exit upon reload.

chrome/browser/ui/browser.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 "chrome/browser/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 4862 matching lines @@
   window_pref.append(app_name);
   PrefService* prefs = profile->GetPrefs();
   if (!prefs->FindPreference(window_pref.c_str())) {
     prefs->RegisterDictionaryPref(window_pref.c_str(),
                                   PrefService::UNSYNCABLE_PREF);
   }
 }
 
 void Browser::ReloadInternal(WindowOpenDisposition disposition,
                              bool ignore_cache) {
+  if (fullscreen_controller_->HandleUserPressedReload())
+    return;

[yzshen1, 2012/06/18 16:22:05]

Why the following actions are not needed when fullscreen controller returns
true? Besides, I read the code and saw that the method always returns false:

- Do you plan to have the method return true in a future change?
- If you have such a plan, that will affect reload behavior. I don't think that
is intended, since in bug 130301, you wrote "Expected: Mouse is unlocked, page
is reloaded."

+
   // If we are showing an interstitial, treat this as an OpenURL.
   WebContents* current_tab = GetActiveWebContents();
   if (current_tab && current_tab->ShowingInterstitialPage()) {
     NavigationEntry* entry = current_tab->GetController().GetActiveEntry();
     DCHECK(entry);  // Should exist if interstitial is showing.
     OpenURL(OpenURLParams(
         entry->GetURL(), Referrer(), disposition,
         content::PAGE_TRANSITION_RELOAD, false));
     return;
   }
@@ skipping 337 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         content::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }