Query the existence other forms of browsing history.

components/browsing_data_ui/history_notice_utils.h

 // 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.
 
 #ifndef COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_
 #define COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_
 
+#include <string>
+
 #include "base/callback_forward.h"
 
-class ProfileSyncService;
-
 namespace history {
 class WebHistoryService;
 }
 
+namespace sync_driver {
+class SyncService;
+}
+
+namespace version_info {
+enum class Channel;
+}
+
 namespace browsing_data_ui {
 
 namespace testing {
 
 // TODO(crbug.com/595332): A boolean flag indicating that
 // ShouldShowNoticeAboutOtherFormsOfBrowsingHistory() should skip the query
 // for other forms of browsing history and just assume some such forms were
 // found. Used only for testing. The default is false.
 extern bool g_override_other_forms_of_browsing_history_query;
 
-}
+}  // testing
 
 // Whether the Clear Browsing Data UI should show a notice about the existence
 // of other forms of browsing history stored in user's account. The response
 // is returned in a |callback|.
 void ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(
-    const ProfileSyncService* sync_service,
+    const sync_driver::SyncService* sync_service,
     history::WebHistoryService* history_service,
     base::Callback<void(bool)> callback);
 
 // Whether the Clear Browsing Data UI should popup a dialog with information
 // about the existence of other forms of browsing history stored in user's
 // account when the user deletes their browsing history for the first time.
-// The response is returned in a |callback|.
+// The response is returned in a |callback|. The |channel| and |is_tablet|
+// parameters must be provided for successful communication with the Sync
+// server, but the result does not depend from them.
 void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory(
-    const ProfileSyncService* sync_service,
+    const sync_driver::SyncService* sync_service,
+    history::WebHistoryService* history_service,
+    version_info::Channel channel,
+    bool is_tablet,

[sdefresne, 2016/05/24 08:32:29]

Since you already indirectly depends on ui/base (through
components/history/core/browser), why not take the device form factor here
instead of a boolean?

#include "ui/base/device_form_factor.h"

void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory(
    const sync_driver::SyncService* sync_service,
    history::WebHistoryService* history_service,
    version_info::Channel channel,
    ui::DeviceFormFactor form_factor,
    base::Callback<void(bool)> callback);

[msramek, 2016/05/24 12:03:00]

Actually in that case, WebHistoryService itself can call DeviceFormFactor, it
doesn't have to be passed from outside.

+    base::Callback<void(bool)> callback);
+
+// A deprecated overloaded version of the above function called by iOS.
+// TODO(msarda): Remove this when iOS calls the correct version.

[sdefresne, 2016/05/24 08:32:29]

Can you create a bug and use the format "TODO(crbug/xxxxxx)" instead? 

[msramek, 2016/05/24 12:03:00]

Done. crbug.com/614319

+void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory(
+    const sync_driver::SyncService* sync_service,
     history::WebHistoryService* history_service,
     base::Callback<void(bool)> callback);
 
 }  // namespace browsing_data_ui
 
 #endif  // COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_