Implemented argument wrapping for CallJS.

chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc

 // 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.
 
 #include "chrome/browser/ui/webui/chromeos/login/error_screen_handler.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/time.h"
 #include "base/values.h"

[bartfab (slow), 2013/04/18 07:39:31]

Nit: No longer used.

[ygorshenin1, 2013/04/18 07:58:23]

Done.

 #include "chrome/browser/chromeos/login/captive_portal_window_proxy.h"
 #include "chrome/browser/chromeos/login/webui_login_display_host.h"
 #include "chrome/browser/chromeos/net/network_portal_detector.h"
 #include "chrome/browser/ui/webui/chromeos/login/native_window_delegate.h"
 #include "chrome/browser/ui/webui/chromeos/login/network_state_informer.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "grit/generated_resources.h"
 
 namespace chromeos {
 
@@ skipping 60 matching lines @@
   captive_portal_window_proxy_->Show();
 }
 
 void ErrorScreenHandler::HideCaptivePortal() {
   if (captive_portal_window_proxy_.get())
     captive_portal_window_proxy_->Close();
 }
 
 void ErrorScreenHandler::SetUIState(ErrorScreen::UIState ui_state) {
   ui_state_ = ui_state;
-  base::FundamentalValue ui_state_value(static_cast<int>(ui_state));
-  CallJS("login.ErrorMessageScreen.setUIState", ui_state_value);
+  CallJS("login.ErrorMessageScreen.setUIState", static_cast<int>(ui_state_));
 }
 
 void ErrorScreenHandler::SetErrorState(ErrorScreen::ErrorState error_state,
                                        const std::string& network) {
   error_state_ = error_state;
-  base::FundamentalValue error_state_value(static_cast<int>(error_state));
-  base::StringValue network_value(network);
   CallJS("login.ErrorMessageScreen.setErrorState",
-         error_state_value,
-         network_value);
+         static_cast<int>(error_state_), network);
 }
 
 void ErrorScreenHandler::AllowGuestSignin(bool allowed) {
-  base::FundamentalValue allowed_value(allowed);
-  CallJS("login.ErrorMessageScreen.allowGuestSignin", allowed_value);
+  CallJS("login.ErrorMessageScreen.allowGuestSignin", allowed);
 }
 
 void ErrorScreenHandler::AllowOfflineLogin(bool allowed) {
-  base::FundamentalValue allowed_value(allowed);
-  CallJS("login.ErrorMessageScreen.allowOfflineLogin", allowed_value);
+  CallJS("login.ErrorMessageScreen.allowOfflineLogin", allowed);
 }
 
 void ErrorScreenHandler::NetworkErrorShown() {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_LOGIN_NETWORK_ERROR_SHOWN,
       content::NotificationService::AllSources(),
       content::NotificationService::NoDetails());
 }
 
 bool ErrorScreenHandler::GetScreenName(OobeUI::Screen screen,
@@ skipping 39 matching lines @@
 void ErrorScreenHandler::Initialize() {
   if (!page_is_ready())
     return;
   if (show_on_init_) {
     Show(parent_screen_, NULL);
     show_on_init_ = false;
   }
 }
 
 }  // namespace chromeos