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

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 10666010: Don't pass the connector check policy flag along. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: anon namespace Created 8 years, 6 months 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/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 0babd72bd8f15412f384e5a7aa372e19eb33ae46..10722548e0d0f79ee03490823b3efc3bccf39c27 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1509,13 +1509,22 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
return chrome::RESULT_CODE_PACK_EXTENSION_ERROR;
}
+ bool pass_command_line = true;
+
#if !defined(OS_MACOSX)
// In environments other than Mac OS X we support import of settings
// from other browsers. In case this process is a short-lived "import"
// process that another browser runs just to import the settings, we
// don't want to be checking for another browser process, by design.
- if (!HasImportSwitch(parsed_command_line())) {
+ pass_command_line = !HasImportSwitch(parsed_command_line());
#endif
+
+ // If we're being launched just to check the connector policy, we are
+ // short-lived and don't want to be passing that switch off.
+ pass_command_line = pass_command_line && !parsed_command_line().HasSwitch(
+ switches::kCheckCloudPrintConnectorPolicy);
+
+ if (pass_command_line) {
// When another process is running, use that process instead of starting a
// new one. NotifyOtherProcess will currently give the other process up to
// 20 seconds to respond. Note that this needs to be done before we attempt
@@ -1532,7 +1541,7 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
printf("%s\n", base::SysWideToNativeMB(UTF16ToWide(
l10n_util::GetStringUTF16(IDS_USED_EXISTING_BROWSER))).c_str());
#endif
- return content::RESULT_CODE_NORMAL_EXIT;
+ return chrome::RESULT_CODE_NORMAL_EXIT_PROCESS_NOTIFIED;
case ProcessSingleton::PROFILE_IN_USE:
return chrome::RESULT_CODE_PROFILE_IN_USE;
@@ -1548,9 +1557,7 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
default:
NOTREACHED();
}
-#if !defined(OS_MACOSX) // closing brace for if
}
-#endif
#if defined(USE_X11)
SetBrowserX11ErrorHandlers();

Powered by Google App Engine
This is Rietveld 408576698