From 869e4de32ddd3f0f53d31cd319b2692c1ed422b0 Mon Sep 17 00:00:00 2001 From: "jorlow@chromium.org" Date: Sun, 18 Apr 2010 08:32:31 +0000 Subject: Revert 44885 - Select and close previous theme info bars when resetting to default theme via the options dialog. As a nice side effect if one tab is showing a theme info bar and the user switches to a different theme in another tab, this change closes the previous theme info bars as well. BUG=35607 TEST=Install a theme, and with the theme info bar visible open Preferences dialog and click 'reset to default theme' and check the info bar. (full steps given in the above bug entry) Landing http://codereview.chromium.org/1622017/show for Satish. Review URL: http://codereview.chromium.org/1600030 TBR=jorlow@chromium.org Review URL: http://codereview.chromium.org/1599038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44886 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/automation/automation_provider.cc | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) (limited to 'chrome/browser/automation/automation_provider.cc') diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc index ccba52b..023b337 100644 --- a/chrome/browser/automation/automation_provider.cc +++ b/chrome/browser/automation/automation_provider.cc @@ -567,7 +567,6 @@ void AutomationProvider::OnMessageReceived(const IPC::Message& message) { IPC_MESSAGE_HANDLER_DELAY_REPLY(AutomationMsg_WaitForPopupMenuToOpen, WaitForPopupMenuToOpen) #endif - IPC_MESSAGE_HANDLER(AutomationMsg_ResetToDefaultTheme, ResetToDefaultTheme) IPC_END_MESSAGE_MAP() } @@ -2794,7 +2793,7 @@ void AutomationProvider::WaitForExtensionTestResult( } void AutomationProvider::InstallExtensionAndGetHandle( - const FilePath& crx_path, bool with_ui, IPC::Message* reply_message) { + const FilePath& crx_path, IPC::Message* reply_message) { ExtensionsService* service = profile_->GetExtensionsService(); ExtensionProcessManager* manager = profile_->GetExtensionProcessManager(); if (service && manager) { @@ -2805,12 +2804,10 @@ void AutomationProvider::InstallExtensionAndGetHandle( AutomationMsg_InstallExtensionAndGetHandle::ID, reply_message); - ExtensionInstallUI* client = - (with_ui ? new ExtensionInstallUI(profile_) : NULL); scoped_refptr installer( new CrxInstaller(service->install_directory(), service, - client)); + NULL)); // silent install, no UI installer->set_allow_privilege_increase(true); installer->InstallCrx(crx_path); } else { @@ -3010,7 +3007,3 @@ void AutomationProvider::WaitForPopupMenuToOpen(IPC::Message* reply_message) { NOTIMPLEMENTED(); } #endif // !defined(TOOLKIT_VIEWS) - -void AutomationProvider::ResetToDefaultTheme() { - profile_->ClearTheme(); -} -- cgit v1.1