Do not initialize event routers in the Profile Import process.

chrome/browser/browser_main.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browser_main.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 1691 matching lines @@
     std::string username =
         parsed_command_line.GetSwitchValueASCII(switches::kLoginUser);
     VLOG(1) << "Relaunching browser for user: " << username;
     chromeos::UserManager::Get()->UserLoggedIn(username);
 
     // Redirects Chrome logging to the user data dir.
     logging::RedirectChromeLogging(parsed_command_line);
   }
 #endif
 
+  if (is_first_run) {
+    // Warn the ProfileManager that an import process will run, possibly
+    // locking the WebDataService directory of the next Profile created.
+    g_browser_process->profile_manager()->SetWillImport(true, NULL);
+  }
+
   Profile* profile = CreateProfile(parameters, user_data_dir,
                                    parsed_command_line);
   if (!profile)
     return content::RESULT_CODE_NORMAL_EXIT;
 
   // Post-profile init ---------------------------------------------------------
 
 #if defined(OS_CHROMEOS)
   // Handling the user cloud policy initialization for case 2 mentioned above.
   // We do this after the profile creation since we need the TokenService.
@@ skipping 63 matching lines @@
   }
 #endif
 
   // Show the First Run UI if this is the first time Chrome has been run on
   // this computer, or we're being compelled to do so by a command line flag.
   // Note that this be done _after_ the PrefService is initialized and all
   // preferences are registered, since some of the code that the importer
   // touches reads preferences.
   if (is_first_run) {
     if (!first_run_ui_bypass) {
       FirstRun::AutoImport(profile,

[Yoyo Zhou, 2011/08/25 16:34:03]

This returns after the import is finished (or it's done locking the WDS
directory), right?

[Miranda Callahan, 2011/08/26 08:31:32]

Yes it does; the rest of the startup process continues below.

                            master_prefs.homepage_defined,
                            master_prefs.do_import_items,
                            master_prefs.dont_import_items,
                            master_prefs.run_search_engine_experiment,
                            master_prefs.randomize_search_engine_experiment,
                            master_prefs.make_chrome_default,
                            &process_singleton);
 #if defined(OS_POSIX)
       // On Windows, the download is tagged with enable/disable stats so there
       // is no need for this code.
 
       // If stats reporting was turned on by the first run dialog then toggle
       // the pref.
       if (GoogleUpdateSettings::GetCollectStatsConsent())
         local_state->SetBoolean(prefs::kMetricsReportingEnabled, true);
 #endif  // OS_POSIX
     }  // if (!first_run_ui_bypass)
 
     Browser::SetNewHomePagePrefs(user_prefs);
-  }
+    g_browser_process->profile_manager()->SetWillImport(false, profile);

[Yoyo Zhou, 2011/08/25 16:34:03]

Maybe this function should have a different name -- at first glance it isn't
obvious that this notifies the profile manager that import is finished.

Perhaps
SetWillImport()
and
[On]ImportFinished(profile)
?
(I'm not sure about the naming conventions here.)

[Miranda Callahan, 2011/08/26 08:31:32]

Yeah, I wasn't really happy with this either. I'll use your suggestion.

+  }  // if (is_first_run)
 
   // Sets things up so that if we crash from this point on, a dialog will
   // popup asking the user to restart chrome. It is done this late to avoid
   // testing against a bunch of special cases that are taken care early on.
   PrepareRestartOnCrashEnviroment(parsed_command_line);
 
 #if defined(OS_WIN)
   // Registers Chrome with the Windows Restart Manager, which will restore the
   // Chrome session when the computer is restarted after a system update.
   // This could be run as late as WM_QUERYENDSESSION for system update reboots,
@@ skipping 313 matching lines @@
 #if defined(OS_CHROMEOS)
   // To be precise, logout (browser shutdown) is not yet done, but the
   // remaining work is negligible, hence we say LogoutDone here.
   chromeos::BootTimesLoader::Get()->AddLogoutTimeMarker("LogoutDone",
                                                         false);
   chromeos::BootTimesLoader::Get()->WriteLogoutTimes();
 #endif
   TRACE_EVENT_END_ETW("BrowserMain", 0, 0);
   return result_code;
 }