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

Side by Side Diff: chrome/browser/ui/webui/options/automatic_settings_reset_handler.h

Issue 151003004: Add an automatic settings reset banner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pre-review cleanup. Created 6 years, 10 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_AUTOMATIC_SETTINGS_RESET_HANDLER_H_
6 #define CHROME_BROWSER_UI_WEBUI_OPTIONS_AUTOMATIC_SETTINGS_RESET_HANDLER_H_
7
8 #include "base/memory/weak_ptr.h"
9 #include "chrome/browser/ui/webui/options/options_ui.h"
10
11 namespace options {
12
13 // Handler for both the banner that displays a settings reset event at the
erikwright (departed) 2014/02/05 15:50:03 "both"?
robertshield 2014/02/06 18:56:50 yes, both the banner and the banner. Also, the ban
14 // top of the settings page.
15 class AutomaticSettingsResetHandler
16 : public OptionsPageUIHandler,
17 public base::SupportsWeakPtr<AutomaticSettingsResetHandler> {
gab 2014/02/05 15:23:52 Doesn't seem you use weak ptrs; remove this?
robertshield 2014/02/06 18:56:50 Using them now. See .cc file.
18 public:
19 AutomaticSettingsResetHandler();
20 virtual ~AutomaticSettingsResetHandler();
21
22 // OptionsPageUIHandler implementation.
23 virtual void GetLocalizedValues(
24 base::DictionaryValue* localized_strings) OVERRIDE;
25 virtual void InitializePage() OVERRIDE;
26
27 // WebUIMessageHandler implementation.
28 virtual void RegisterMessages() OVERRIDE;
29
30 private:
31 // Called when the user dismisses the automatic settings reset banner.
32 void OnDismissedAutomaticSettingsResetBanner(const base::ListValue* value);
33
34 DISALLOW_COPY_AND_ASSIGN(AutomaticSettingsResetHandler);
35 };
36
37 } // namespace options
38
39 #endif // CHROME_BROWSER_UI_WEBUI_OPTIONS_AUTOMATIC_SETTINGS_RESET_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698