Index: chrome/browser/ui/webui/chromeos/login/base_screen_handler.h |
diff --git a/chrome/browser/ui/webui/chromeos/login/base_screen_handler.h b/chrome/browser/ui/webui/chromeos/login/base_screen_handler.h |
index 4c485f58039364806eff24c459b1552bc4f988f3..df5f22ba54e989c2c9124383e78598cd429496da 100644 |
--- a/chrome/browser/ui/webui/chromeos/login/base_screen_handler.h |
+++ b/chrome/browser/ui/webui/chromeos/login/base_screen_handler.h |
@@ -65,23 +65,6 @@ class BaseScreenHandler : public content::WebUIMessageHandler, |
return async_assets_load_id_; |
} |
- protected: |
- // All subclasses should implement this method to provide localized values. |
- virtual void DeclareLocalizedValues( |
- ::login::LocalizedValuesBuilder* builder) = 0; |
- |
- // All subclasses should implement this method to register callbacks for JS |
- // messages. |
- // |
- // TODO (ygorshenin, crbug.com/433797): make this method purely vrtual when |
- // all screens will be switched to use ScreenContext. |
- virtual void DeclareJSCallbacks() {} |
- |
- // Subclasses can override these methods to pass additional parameters |
- // to loadTimeData. Generally, it is a bad approach, and it should be replaced |
- // with Context at some point. |
- virtual void GetAdditionalParameters(base::DictionaryValue* parameters); |
- |
// Shortcut for calling JS methods on WebUI side. |
void CallJS(const std::string& method); |
@@ -120,6 +103,23 @@ class BaseScreenHandler : public content::WebUIMessageHandler, |
::login::MakeValue(arg4)); |
} |
+ protected: |
+ // All subclasses should implement this method to provide localized values. |
+ virtual void DeclareLocalizedValues( |
+ ::login::LocalizedValuesBuilder* builder) = 0; |
+ |
+ // All subclasses should implement this method to register callbacks for JS |
+ // messages. |
+ // |
+ // TODO (ygorshenin, crbug.com/433797): make this method purely vrtual when |
+ // all screens will be switched to use ScreenContext. |
+ virtual void DeclareJSCallbacks() {} |
+ |
+ // Subclasses can override these methods to pass additional parameters |
+ // to loadTimeData. Generally, it is a bad approach, and it should be replaced |
+ // with Context at some point. |
+ virtual void GetAdditionalParameters(base::DictionaryValue* parameters); |
+ |
// Shortcut methods for adding WebUI callbacks. |
template<typename T> |
void AddRawCallback(const std::string& name, |