Change ShouldShowNoticeAboutOtherFormsOfBrowsingHistory to be asynchronous.

components/browsing_data_ui/history_notice_utils.cc

 // Copyright 2016 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 "components/browsing_data_ui/history_notice_utils.h"
 
+#include "base/callback.h"
 #include "components/browser_sync/browser/profile_sync_service.h"
 #include "components/history/core/browser/web_history_service.h"
 
 namespace browsing_data_ui {
 
 bool ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(

[msarda, 2016/03/23 10:35:42]

Fix the return type.

[msramek, 2016/03/23 10:56:51]

Ah, yes. It linked successfully, because it's not called anywhere yet.

     const ProfileSyncService* sync_service,
-    const history::WebHistoryService* history_service) {
-  return sync_service &&
-         sync_service->IsSyncActive() &&
-         !sync_service->IsUsingSecondaryPassphrase() &&
-         history_service &&
-         history_service->HasOtherFormsOfBrowsingHistory();
+    const history::WebHistoryService* history_service,
+    base::Callback<void(bool)> callback) {
+  // TODO(msramek): The asynchronicity is technically not needed here now,
+  // but when we change WebHistoryService::HasOtherFormsOfBrowsingHistory()
+  // to communicate with the server, we will get the response asynchronously.
+  // Therefore, we respond asynchronously here as well in anticipation of this
+  // change.
+  callback.Run(sync_service &&
+               sync_service->IsSyncActive() &&
+               !sync_service->IsUsingSecondaryPassphrase() &&
+               history_service &&
+               history_service->HasOtherFormsOfBrowsingHistory());
 }
 
 }  // namespace browsing_data_ui