diff options
author | cpu@google.com <cpu@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-29 23:25:08 +0000 |
---|---|---|
committer | cpu@google.com <cpu@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-29 23:25:08 +0000 |
commit | 1d1a51bb6862ae5d0fe4147c5f5a78ff450607a2 (patch) | |
tree | 37d05eef3e28c5d6b09097e95580fa627d611e7e /chrome/browser/browser_main.cc | |
parent | 162dc56ca7a6e9092c4dbc2f1da5a2be2ce4a26c (diff) | |
download | chromium_src-1d1a51bb6862ae5d0fe4147c5f5a78ff450607a2.zip chromium_src-1d1a51bb6862ae5d0fe4147c5f5a78ff450607a2.tar.gz chromium_src-1d1a51bb6862ae5d0fe4147c5f5a78ff450607a2.tar.bz2 |
This CL fixes bugs with EULA
- The first run import will trigger the eula again : this requires changes in browser_main.cc
so the master prefs are not processed again by the importer process
- The launch of setup.exe to show the eula was using a path that only works on developer builds
this requires relocating a constant from setup_constants to util_constants
BUG=1468838
Review URL: http://codereview.chromium.org/19680
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8928 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_main.cc')
-rw-r--r-- | chrome/browser/browser_main.cc | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc index c09f99f..bc25e95 100644 --- a/chrome/browser/browser_main.cc +++ b/chrome/browser/browser_main.cc @@ -220,12 +220,17 @@ int BrowserMain(const MainFunctionParams& parameters) { PathService::Get(chrome::DIR_USER_DATA, &user_data_dir); MessageWindow message_window(user_data_dir); + bool is_first_run = FirstRun::IsChromeFirstRun() || + parsed_command_line.HasSwitch(switches::kFirstRun); + bool first_run_ui_bypass = false; + scoped_ptr<BrowserProcess> browser_process; if (parsed_command_line.HasSwitch(switches::kImport)) { // We use different BrowserProcess when importing so no GoogleURLTracker is // instantiated (as it makes a URLRequest and we don't have an IO thread, // see bug #1292702). browser_process.reset(new FirstRunBrowserProcess(parsed_command_line)); + is_first_run = false; } else { browser_process.reset(new BrowserProcessImpl(parsed_command_line)); } @@ -242,10 +247,6 @@ int BrowserMain(const MainFunctionParams& parameters) { PrefService* local_state = browser_process->local_state(); DCHECK(local_state); - bool is_first_run = FirstRun::IsChromeFirstRun() || - parsed_command_line.HasSwitch(switches::kFirstRun); - bool first_run_ui_bypass = false; - // Initialize ResourceBundle which handles files loaded from external // sources. This has to be done before uninstall code path and before prefs // are registered. |