Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8027)

Unified Diff: chrome/installer/setup/setup_main.cc

Issue 345021: Continue to remove CHROME_FRAME_BUILD define from code that goes into chrome.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/installer/setup/setup_main.cc
===================================================================
--- chrome/installer/setup/setup_main.cc (revision 30960)
+++ chrome/installer/setup/setup_main.cc (working copy)
@@ -598,18 +598,18 @@
BrowserDistribution* dist = BrowserDistribution::GetDistribution();
-#if defined(CHROME_FRAME_BUILD)
- if (install_status == installer_util::UNINSTALL_REQUIRES_REBOOT) {
- ShowRebootDialog();
- } else if (parsed_command_line.HasSwitch(
- installer_util::switches::kUninstall)) {
- ::MessageBoxW(NULL,
- installer_util::GetLocalizedString(
- IDS_UNINSTALL_COMPLETE_BASE).c_str(),
- dist->GetApplicationName().c_str(),
- MB_OK);
+ if (InstallUtil::IsChromeFrameProcess()) {
+ if (install_status == installer_util::UNINSTALL_REQUIRES_REBOOT) {
+ ShowRebootDialog();
+ } else if (parsed_command_line.HasSwitch(
+ installer_util::switches::kUninstall)) {
+ ::MessageBoxW(NULL,
+ installer_util::GetLocalizedString(
+ IDS_UNINSTALL_COMPLETE_BASE).c_str(),
+ dist->GetApplicationName().c_str(),
+ MB_OK);
+ }
}
-#endif
if (install_status == installer_util::UNINSTALL_REQUIRES_REBOOT) {
install_status = installer_util::UNINSTALL_SUCCESSFUL;

Powered by Google App Engine
This is Rietveld 408576698