Add an infrastructure to query history.google.com about other forms of browsing history.

components/browsing_data/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_HISTORY_NOTICE_UTILS_H_
+#define COMPONENTS_BROWSING_DATA_HISTORY_NOTICE_UTILS_H_
+
+namespace history {
+class WebHistoryService;
+}
+
+class ProfileSyncService;
+
+namespace browsing_data {
+
+// Whether the Clear Browsing Data UI should show a notice about the existence
+// of other forms of browsing history stored in user's account.
+bool ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(

[sdefresne, 2016/03/17 09:49:39]

You CL description says "we need to put this logic to a component (especially
because of iOS)" however iOS cannot use components that depends on //content. So
you'll either have to remove the dependency on //content from
//components/browsing_data or move the code into another component.

See components/browsing_data.gypi, components/browsing_data/BUILD.gn and
components/browsing_data/DEPS for the dependency on //content.

What is your plan here?

[msramek, 2016/03/17 13:04:34]

OK, I wasn't aware of that. Now I can see that browsing_data is also explicitly
listed in the non-iOS sources of components.gyp/BUILD.gn.

After an offline chat with msarda@, I decided to move this to a separate
component. It feels to be a bit of an overkill for one method, but since this CL
introduces the glue that both of us need for the UI work, and it also fixes the
dependency chain in Sync, let me just roll with it. In the end, it's not
unlikely that we will add more stuff here, or repurpose the component for more
CBD dialog-related things.

+    const ProfileSyncService* sync_service,
+    const history::WebHistoryService* history_service);
+
+}  // namespace browsing_data
+
+#endif  // COMPONENTS_BROWSING_DATA_HISTORY_NOTICE_UTILS_H_