cros: Move call js prefix out of ctor.

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 18 matching lines @@
 
 namespace chromeos {
 
 class BaseScreen;
 class OobeUI;
 
 // Base class for the OOBE/Login WebUI handlers.
 class BaseScreenHandler : public content::WebUIMessageHandler,
                           public ModelViewChannel {
  public:
-  // C-tor used when JS screen prefix is not needed.
   BaseScreenHandler();
-
-  // C-tor used when JS screen prefix is needed.
-  explicit BaseScreenHandler(const std::string& js_screen_path);
-
   ~BaseScreenHandler() override;
 
   // Gets localized strings to be used on the page.
   void GetLocalizedStrings(
       base::DictionaryValue* localized_strings);
 
   // WebUIMessageHandler implementation:
   void RegisterMessages() override;
 
   // ModelViewChannel implementation:
   void CommitContextChanges(const base::DictionaryValue& diff) override;
 
   // This method is called when page is ready. It propagates to inherited class
   // 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_;
   }
 
+  // Set the prefix used when running CallJs with a method. For example,
+  //    set_call_js_prefix("Oobe")
+  //    CallJs("lock") -> Invokes JS global named "Oobe.lock"
+  void set_call_js_prefix(const std::string& prefix) {
+    js_screen_path_prefix_ = prefix + ".";
+  }
+
  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.
@@ skipping 93 matching lines @@
   // names.
   std::string FullMethodPath(const std::string& method) const;
 
   // Handles user action.
   void HandleUserAction(const std::string& action_id);
 
   // Handles situation when screen context is changed.
   void HandleContextChanged(const base::DictionaryValue* diff);
 
   // Keeps whether page is ready.
-  bool page_is_ready_;
+  bool page_is_ready_ = false;
 
-  BaseScreen* base_screen_;
+  BaseScreen* base_screen_ = nullptr;
 
   // Full name of the corresponding JS screen object. Can be empty, if
   // there are no corresponding screen object or several different
   // objects.
   std::string js_screen_path_prefix_;
 
   // The string id used in the async asset load in JS. If it is set to a
   // non empty value, the Initialize will be deferred until the underlying load
   // is finished.
   std::string async_assets_load_id_;
 
   // 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_