Delay bookmarks load while the profile is loading.

chrome/browser/profiles/profile_manager.cc

Index: chrome/browser/profiles/profile_manager.cc
diff --git a/chrome/browser/profiles/profile_manager.cc b/chrome/browser/profiles/profile_manager.cc
index f95ab4f82ce480b203cc926220fbabb9f329ff37..8574faf71700495b9f82696d36f1369d48c32bc1 100644
--- a/chrome/browser/profiles/profile_manager.cc
+++ b/chrome/browser/profiles/profile_manager.cc
@@ -8,6 +8,7 @@
 
 #include "base/bind.h"
 #include "base/command_line.h"
+#include "base/deferred_sequenced_task_runner.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/metrics/histogram.h"
@@ -761,6 +762,9 @@ void ProfileManager::DoFinalInitForServices(Profile* profile,
   // initializing the managed flag if necessary).
   ManagedUserServiceFactory::GetForProfile(profile)->Init();
 #endif
+  // Start loading the bookmarks after the profile is loaded.
+  BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile);
+  model->GetBookmarkTaskRunner()->Start();

[Miranda Callahan, 2013/03/27 13:31:46]

It seems wrong to expose this inner lazy constructor and let the outside world
know about our inner workings; perhaps just give the BookmarkModel a StartLoad
method, or something with a better name, that calls GetBookmarkTaskRunner
privately to lazily construct the taskrunner? Then you could do a one-liner
here:

BookmarkModelFactory::GetForProfile(profile)->StartLoading();

[msarda, 2013/03/27 14:16:48]

Done.

 }
 
 void ProfileManager::DoFinalInitLogging(Profile* profile) {