Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(155)

Side by Side Diff: components/browsing_data_ui/history_notice_utils.h

Issue 1884553002: Add a testing switch to override the query for other forms of browsing history (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Renamed the switch. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_ 5 #ifndef COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_
6 #define COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_ 6 #define COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_
7 7
8 #include "base/callback_forward.h" 8 #include "base/callback_forward.h"
9 9
10 class ProfileSyncService; 10 class ProfileSyncService;
11 11
12 namespace history { 12 namespace history {
13 class WebHistoryService; 13 class WebHistoryService;
14 } 14 }
15 15
16 namespace browsing_data_ui { 16 namespace browsing_data_ui {
17 17
18 namespace testing {
19
20 // TODO(crbug.com/595332): A boolean flag indicating that
21 // ShouldShowNoticeAboutOtherFormsOfBrowsingHistory() should skip the query
22 // for other forms of browsing history and just assume some such forms were
23 // found. Used only for testing. The default is false.
24 extern bool g_override_other_forms_of_browsing_history_query;
25
26 }
27
18 // Whether the Clear Browsing Data UI should show a notice about the existence 28 // Whether the Clear Browsing Data UI should show a notice about the existence
19 // of other forms of browsing history stored in user's account. The response 29 // of other forms of browsing history stored in user's account. The response
20 // is returned in a |callback|. 30 // is returned in a |callback|.
21 void ShouldShowNoticeAboutOtherFormsOfBrowsingHistory( 31 void ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(
22 const ProfileSyncService* sync_service, 32 const ProfileSyncService* sync_service,
23 history::WebHistoryService* history_service, 33 history::WebHistoryService* history_service,
24 base::Callback<void(bool)> callback); 34 base::Callback<void(bool)> callback);
25 35
26 // Whether the Clear Browsing Data UI should popup a dialog with information 36 // Whether the Clear Browsing Data UI should popup a dialog with information
27 // about the existence of other forms of browsing history stored in user's 37 // about the existence of other forms of browsing history stored in user's
28 // account when the user deletes their browsing history for the first time. 38 // account when the user deletes their browsing history for the first time.
29 // The response is returned in a |callback|. 39 // The response is returned in a |callback|.
30 void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory( 40 void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory(
31 const ProfileSyncService* sync_service, 41 const ProfileSyncService* sync_service,
32 history::WebHistoryService* history_service, 42 history::WebHistoryService* history_service,
33 base::Callback<void(bool)> callback); 43 base::Callback<void(bool)> callback);
34 44
35 } // namespace browsing_data_ui 45 } // namespace browsing_data_ui
36 46
37 #endif // COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_ 47 #endif // COMPONENTS_BROWSING_DATA_UI_HISTORY_NOTICE_UTILS_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/history_ui.cc ('k') | components/browsing_data_ui/history_notice_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698