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

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

Issue 9693032: [uber page] Split up initialization of handlers from initialization of webui pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rename Initialize -> InitializeHandler, SendPageValues -> InitializePage Created 8 years, 9 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
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_
6 #define CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_ 6 #define CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_
7 #pragma once 7 #pragma once
8 8
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "chrome/browser/ui/webui/options/options_ui.h" 10 #include "chrome/browser/ui/webui/options/options_ui.h"
11 11
12 namespace chromeos { 12 namespace chromeos {
13 13
14 // ChromeOS handler for "Stats/crash reporting to Google" option of the Advanced 14 // ChromeOS handler for "Stats/crash reporting to Google" option of the Advanced
15 // settings page. This handler does only ChromeOS-specific actions while default 15 // settings page. This handler does only ChromeOS-specific actions while default
16 // code is in Chrome's AdvancedOptionsHandler 16 // code is in Chrome's AdvancedOptionsHandler
17 // (chrome/browser/webui/advanced_options_handler.cc). 17 // (chrome/browser/webui/advanced_options_handler.cc).
18 class StatsOptionsHandler : public OptionsPageUIHandler { 18 class StatsOptionsHandler : public OptionsPageUIHandler {
19 public: 19 public:
20 StatsOptionsHandler(); 20 StatsOptionsHandler();
21 21
22 // OptionsPageUIHandler implementation. 22 // OptionsPageUIHandler implementation.
23 virtual void GetLocalizedValues( 23 virtual void GetLocalizedValues(
24 base::DictionaryValue* localized_strings) OVERRIDE; 24 base::DictionaryValue* localized_strings) OVERRIDE;
25 virtual void Initialize() OVERRIDE; 25 virtual void InitializeHandler() OVERRIDE;
26 26
27 // WebUIMessageHandler implementation. 27 // WebUIMessageHandler implementation.
28 virtual void RegisterMessages() OVERRIDE; 28 virtual void RegisterMessages() OVERRIDE;
29 29
30 private: 30 private:
31 void HandleMetricsReportingCheckbox(const base::ListValue* args); 31 void HandleMetricsReportingCheckbox(const base::ListValue* args);
32 32
33 DISALLOW_COPY_AND_ASSIGN(StatsOptionsHandler); 33 DISALLOW_COPY_AND_ASSIGN(StatsOptionsHandler);
34 }; 34 };
35 35
36 } // namespace chromeos 36 } // namespace chromeos
37 37
38 #endif // CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_ 38 #endif // CHROME_BROWSER_UI_WEBUI_OPTIONS_CHROMEOS_STATS_OPTIONS_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698