Fix of "login is not defined" error in OOBE

chrome/browser/ui/webui/chromeos/login/base_screen_handler.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_BASE_SCREEN_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_BASE_SCREEN_HANDLER_H_
 
 #include <string>
 
 #include "base/bind.h"
@@ skipping 47 matching lines @@
   // via virtual Initialize() method (see below).
   void InitializeBase();
 
   void set_async_assets_load_id(const std::string& async_assets_load_id) {
     async_assets_load_id_ = async_assets_load_id;
   }
   const std::string& async_assets_load_id() const {
     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);
 
   template<typename A1>
   void CallJS(const std::string& method, const A1& arg1) {
     web_ui()->CallJavascriptFunctionUnsafe(FullMethodPath(method),
                                            ::login::MakeValue(arg1));
   }
 
   template<typename A1, typename A2>
@@ skipping 18 matching lines @@
               const A1& arg1,
               const A2& arg2,
               const A3& arg3,
               const A4& arg4) {
     web_ui()->CallJavascriptFunctionUnsafe(
         FullMethodPath(method), ::login::MakeValue(arg1),
         ::login::MakeValue(arg2), ::login::MakeValue(arg3),
         ::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,
                       void (T::*method)(const base::ListValue* args)) {
     web_ui()->RegisterMessageCallback(
         name,
         base::Bind(method, base::Unretained(static_cast<T*>(this))));
   }
 
   template<typename T, typename... Args>
@@ skipping 61 matching lines @@
 
   // Pending changes to context which will be sent when the page will be ready.
   base::DictionaryValue pending_context_changes_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseScreenHandler);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_BASE_SCREEN_HANDLER_H_