Make the reload button respond to middle-clicks like back/forward/home already do.
This also makes GTK and Mac reset the location bar in the correct cases with various types of clicks on back/forward.
BUG=none
TEST=Middle-click reload. It should reload the page in a new background tab.
Review URL: http://codereview.chromium.org/2779011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49350 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc
index dd1a07b..98f05e7c 100644
--- a/chrome/browser/automation/automation_provider.cc
+++ b/chrome/browser/automation/automation_provider.cc
@@ -709,7 +709,7 @@
Browser* browser = FindAndActivateTab(tab);
if (browser && browser->command_updater()->IsCommandEnabled(IDC_RELOAD)) {
AddNavigationStatusListener(tab, reply_message, 1, false);
- browser->Reload();
+ browser->Reload(CURRENT_TAB);
return;
}
}
@@ -2079,11 +2079,11 @@
void AutomationProvider::OmniboxAcceptInput(Browser* browser,
DictionaryValue* args,
IPC::Message* reply_message) {
- NavigationController& tab =
- browser->GetOrCloneNavigationControllerForDisposition(CURRENT_TAB);
+ NavigationController& controller =
+ browser->GetSelectedTabContents()->controller();
// Setup observer to wait until the selected item loads.
NotificationObserver* observer =
- new OmniboxAcceptNotificationObserver(&tab, this, reply_message);
+ new OmniboxAcceptNotificationObserver(&controller, this, reply_message);
notification_observer_list_.AddObserver(observer);
browser->window()->GetLocationBar()->AcceptInput();
