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

Unified Diff: chrome/browser/ui/webui/options/clear_browser_data_handler.h

Issue 2657673004: Add shutdown notification and service refs to SyncServiceObserver. (Closed)
Patch Set: Chromeos fix Created 3 years, 11 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/options/clear_browser_data_handler.h
diff --git a/chrome/browser/ui/webui/options/clear_browser_data_handler.h b/chrome/browser/ui/webui/options/clear_browser_data_handler.h
index d155e11436899d7f2b35c0dc4b24cd4862975a85..e0749461103a5743f6d009106b2af69665926e3c 100644
--- a/chrome/browser/ui/webui/options/clear_browser_data_handler.h
+++ b/chrome/browser/ui/webui/options/clear_browser_data_handler.h
@@ -57,9 +57,11 @@ class ClearBrowserDataHandler : public OptionsPageUIHandler,
void UpdateCounterText(
std::unique_ptr<browsing_data::BrowsingDataCounter::Result> result);
- // Implementation of SyncServiceObserver. Updates the support string at the
- // bottom of the dialog.
- void OnStateChanged() override;
+ // Implementation of SyncServiceObserver.
+ void OnStateChanged(syncer::SyncService* sync) override;
+
+ // Updates the support string at the bottom of the dialog.
+ void UpdateSyncState();
// Finds out whether we should show a notice informing the user about other
// forms of browsing history. Responds with an asynchronous callback to

Powered by Google App Engine
This is Rietveld 408576698