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

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

Issue 2598893002: [MD settings] lighter weight initializer for protocol handler enabled messages (Closed)
Patch Set: review changes Created 4 years 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/protocol_handlers_handler.h
diff --git a/chrome/browser/ui/webui/settings/protocol_handlers_handler.h b/chrome/browser/ui/webui/settings/protocol_handlers_handler.h
index 89115348b3aa4b9d0ac414ee0eccc6f830e1a399..269f27f606109dda89d6e06caeaee54c719cdd68 100644
--- a/chrome/browser/ui/webui/settings/protocol_handlers_handler.h
+++ b/chrome/browser/ui/webui/settings/protocol_handlers_handler.h
@@ -44,8 +44,15 @@ class ProtocolHandlersHandler : public SettingsPageUIHandler,
const content::NotificationDetails& details) override;
private:
- // Called to fetch the initial list of data to show.
- void HandleInitializeList(const base::ListValue* args);
+ // Called to fetch the state of the protocol handlers. If the full list of
+ // handlers is not needed, consider HandleObserveProtocolHandlersEnabledState
+ // instead.
+ void HandleObserveProtocolHandlers(const base::ListValue* args);
+
+ // Called to begin updates to the handlers enabled status. This is a subset
+ // (lighter alternative) of HandleObserveProtocolHandlers. There's no need to
+ // call this function if HandleObserveProtocolHandlers is called.
+ void HandleObserveProtocolHandlersEnabledState(const base::ListValue* args);
// Notifies the JS side whether the handlers are enabled or not.
void SendHandlersEnabledValue();

Powered by Google App Engine
This is Rietveld 408576698