Delay bookmarks load while the profile is loading.

chrome/browser/profiles/profile_manager.cc

 // Copyright (c) 2012 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 <set>
 
 #include "chrome/browser/profiles/profile_manager.h"
 
 #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"
 #include "base/prefs/pref_registry_simple.h"
 #include "base/prefs/pref_service.h"
 #include "base/string_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
@@ skipping 733 matching lines @@
           extensions::ExtensionSystem::Get(profile)->extension_service());
     }
   }
 #endif
 #if defined(ENABLE_MANAGED_USERS)
   // Initialization needs to happen after extension system initialization (for
   // extension::ManagementPolicy) and InitProfileUserPrefs (for setting the
   // 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) {
   // Count number of extensions in this profile.
   int extension_count = -1;
 #if defined(ENABLE_EXTENSIONS)
   ExtensionService* extension_service = profile->GetExtensionService();
   if (extension_service)
     extension_count = extension_service->GetAppIds().size();
 #endif
@@ skipping 336 matching lines @@
 ProfileManager::ProfileInfo::ProfileInfo(
     Profile* profile,
     bool created)
     : profile(profile),
       created(created) {
 }
 
 ProfileManager::ProfileInfo::~ProfileInfo() {
   ProfileDestroyer::DestroyProfileWhenAppropriate(profile.release());
 }