Bring up of the enhanced bookmarks cluster service.

chrome/browser/enhanced_bookmarks/chrome_bookmark_server_cluster_service.cc

+// Copyright 2014 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/enhanced_bookmarks/chrome_bookmark_server_cluster_service.h"
+
+#include "chrome/browser/sync/profile_sync_service.h"
+#include "components/bookmarks/browser/bookmark_model.h"
+
+namespace enhanced_bookmarks {
+
+ChromeBookmarkServerClusterService::ChromeBookmarkServerClusterService(
+    const std::string& application_language_code,
+    scoped_refptr<net::URLRequestContextGetter> request_context_getter,
+    ProfileOAuth2TokenService* token_service,
+    SigninManagerBase* signin_manager,
+    BookmarkModel* bookmark_model,
+    PrefService* pref_service,
+    ProfileSyncService* sync_service)
+    : BookmarkServerClusterService(application_language_code,
+                                   request_context_getter,
+                                   token_service,
+                                   signin_manager,
+                                   bookmark_model,
+                                   pref_service),
+      sync_service_(sync_service) {
+  if (sync_service_)
+    sync_service_->AddObserver(this);
+}
+
+ChromeBookmarkServerClusterService::~ChromeBookmarkServerClusterService() {
+  if (sync_service_)
+    sync_service_->RemoveObserver(this);
+}
+
+void ChromeBookmarkServerClusterService::OnStateChanged() {
+  // Do nothing.
+}
+
+void ChromeBookmarkServerClusterService::OnSyncCycleCompleted() {

[Yaron, 2014/09/23 17:08:30]

So per previous comment I think this still happens too often to issue a request
every time. You could add a delay/throttling or maybe there's a better option
for deferring the cluster request.

[Yaron, 2014/09/23 17:11:21]

Err +Nicolas to comment

[noyau (Ping after 24h), 2014/10/01 16:04:42]

Acknowledged. Can I defer this to a subsequent CL? 

[Yaron, 2014/10/01 23:45:21]

I'm pretty sure this will be too often. Since it's not invoked unless the
experiment is active, that's fine but I think we'll have to fix it regardless

+  if (bookmark_model_->loaded())
+    TriggerTokenRequest(false);
+}
+}  // namespace enhanced_bookmarks