Delay bookmarks load while the profile is loading.

chrome/browser/bookmarks/bookmark_model.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 "chrome/browser/bookmarks/bookmark_model.h"
 
 #include <algorithm>
 #include <functional>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/deferred_sequenced_task_runner.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/sequenced_task_runner.h"
 #include "base/string_util.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "chrome/browser/bookmarks/bookmark_expanded_state_tracker.h"
 #include "chrome/browser/bookmarks/bookmark_index.h"
 #include "chrome/browser/bookmarks/bookmark_model_observer.h"
 #include "chrome/browser/bookmarks/bookmark_storage.h"
 #include "chrome/browser/bookmarks/bookmark_utils.h"
@@ skipping 221 matching lines @@
 
   expanded_state_tracker_.reset(new BookmarkExpandedStateTracker(
       profile_, this));
 
   // Listen for changes to favicons so that we can update the favicon of the
   // node appropriately.
   registrar_.Add(this, chrome::NOTIFICATION_FAVICON_CHANGED,
                  content::Source<Profile>(profile_));
 
   // Load the bookmarks. BookmarkStorage notifies us when done.
-  store_ = new BookmarkStorage(profile_, this, profile_->GetIOTaskRunner());
+  store_ = new BookmarkStorage(profile_, this, GetBookmarkTaskRunner());
   store_->LoadBookmarks(CreateLoadDetails());
 }
 
+void BookmarkModel::StartLoading() {
+  GetBookmarkTaskRunner()->Start();
+}
+
 bool BookmarkModel::IsLoaded() const {
   return loaded_;
 }
 
 const BookmarkNode* BookmarkModel::GetParentForNewNodes() {
   std::vector<const BookmarkNode*> nodes =
       bookmark_utils::GetMostRecentlyModifiedFolders(this, 1);
   DCHECK(!nodes.empty());  // This list is always padded with default folders.
   return nodes[0];
 }
@@ skipping 687 matching lines @@
   BookmarkPermanentNode* bb_node =
       CreatePermanentNode(BookmarkNode::BOOKMARK_BAR);
   BookmarkPermanentNode* other_node =
       CreatePermanentNode(BookmarkNode::OTHER_NODE);
   BookmarkPermanentNode* mobile_node =
       CreatePermanentNode(BookmarkNode::MOBILE);
   return new BookmarkLoadDetails(bb_node, other_node, mobile_node,
                                  new BookmarkIndex(profile_),
                                  next_node_id_);
 }
+
+scoped_refptr<base::DeferredSequencedTaskRunner>
+    BookmarkModel::GetBookmarkTaskRunner() {
+  if (!bookmark_task_runner_) {
+    bookmark_task_runner_ =
+        new base::DeferredSequencedTaskRunner(profile_->GetIOTaskRunner());

[tfarina, 2013/03/28 22:57:01]

Can we do something similar to |blocking_task_runner_| of
chrome/browser/chromeos/drive/drive_system_service.cc?

[msarda, 2013/03/29 14:40:03]

I think so. We should probably have a similar start-up task runner for the
drive_system_service.cc. This will be done in a subsequent CL (this is already
starting to be toooo long).

On 2013/03/28 22:57:01, tfarina wrote:

+  }
+  return bookmark_task_runner_;
+}