Chromium Code Reviews| Index: chrome/browser/ui/webui/settings/settings_startup_pages_handler.h |
| diff --git a/chrome/browser/ui/webui/settings/settings_startup_pages_handler.h b/chrome/browser/ui/webui/settings/settings_startup_pages_handler.h |
| index dbfabf2801be58d5b639f38aecc151ea64ebf83f..64a4a76b1ff765784649d2b9571de1cb07667674 100644 |
| --- a/chrome/browser/ui/webui/settings/settings_startup_pages_handler.h |
| +++ b/chrome/browser/ui/webui/settings/settings_startup_pages_handler.h |
| @@ -41,20 +41,20 @@ class StartupPagesHandler : public SettingsPageUIHandler, |
| private: |
| // Adds a startup page with the given URL after the given index. |
| // Called from WebUI. |
| - void AddStartupPage(const base::ListValue* args); |
| + void HandleAddStartupPage(const base::ListValue* args); |
|
Dan Beam
2016/03/10 01:01:08
typical callback naming precedent is "Handle" + "S
|
| // Informs the code that the JS page has loaded. Called from WebUI. |
| - void OnStartupPrefsPageLoad(const base::ListValue* args); |
| + void HandleOnStartupPrefsPageLoad(const base::ListValue* args); |
| // Removes the startup page at the given index. Called from WebUI. |
| - void RemoveStartupPage(const base::ListValue* args); |
| + void HandleRemoveStartupPage(const base::ListValue* args); |
| + |
| + // Sets the startup page set to the current pages. Called from WebUI. |
| + void HandleSetStartupPagesToCurrentPages(const base::ListValue* args); |
| // Stores the current state of the startup page preferences. |
| void SaveStartupPagesPref(); |
| - // Sets the startup page set to the current pages. Called from WebUI. |
| - void SetStartupPagesToCurrentPages(const base::ListValue* args); |
| - |
| // Informs the that the preferences have changed. It is a callback |
| // for the pref changed registrar. |
| void UpdateStartupPages(); |