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

Unified Diff: chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h

Issue 2716333002: Implement important sites dialog for desktop. (Closed)
Patch Set: replace task observer in -Bridge and -Handler with a callback Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h
diff --git a/chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h b/chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h
index 6137c84bcd618fa5c0df63b042b03def58b2e737..717da110a1fe0efb940455cad330432390cce381 100644
--- a/chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h
+++ b/chrome/browser/ui/webui/settings/settings_clear_browsing_data_handler.h
@@ -22,6 +22,7 @@ class ListValue;
namespace content {
class WebUI;
+class BrowsingDataFilterBuilder;
Dan Beam 2017/05/05 17:52:51 alpha
dullweber 2017/05/09 08:56:39 I sorted the declarations
}
namespace settings {
@@ -39,17 +40,21 @@ class ClearBrowsingDataHandler : public SettingsPageUIHandler,
void OnJavascriptDisallowed() override;
private:
- // Observes one |remover| task initiated from ClearBrowsingDataHandler.
- // Calls |callback| when the task is finished.
- class TaskObserver;
-
// Clears browsing data, called by Javascript.
void HandleClearBrowsingData(const base::ListValue* value);
+ // Parses a ListValue with important site information and creates a
+ // BrowsingDataFilterBuilder.
+ std::unique_ptr<content::BrowsingDataFilterBuilder> ProcessImportantSites(
+ const base::ListValue* important_sites);
+
// Called when a clearing task finished. |webui_callback_id| is provided
// by the WebUI action that initiated it.
void OnClearingTaskFinished(const std::string& webui_callback_id);
+ // Fetches important sites, called by Javascript.
+ void HandleFetchImportantSites(const base::ListValue* value);
+
// Initializes the dialog UI. Called by JavaScript when the DOM is ready.
void HandleInitialize(const base::ListValue* args);
@@ -85,9 +90,6 @@ class ClearBrowsingDataHandler : public SettingsPageUIHandler,
// Counters that calculate the data volume for individual data types.
std::vector<std::unique_ptr<browsing_data::BrowsingDataCounter>> counters_;
- // Observes the currently active data clearing task.
- std::unique_ptr<TaskObserver> task_observer_;
-
// ProfileSyncService to observe sync state changes.
browser_sync::ProfileSyncService* sync_service_;
ScopedObserver<browser_sync::ProfileSyncService, syncer::SyncServiceObserver>

Powered by Google App Engine
This is Rietveld 408576698