Simplify adding callbacks.

chrome/browser/ui/webui/chromeos/login/network_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/network_screen_handler.h"
 
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 116 matching lines @@
 void NetworkScreenHandler::RegisterMessages() {
   AddCallback(kJsApiNetworkOnExit, &NetworkScreenHandler::HandleOnExit);
   AddCallback(kJsApiNetworkOnLanguageChanged,
               &NetworkScreenHandler::HandleOnLanguageChanged);
   AddCallback(kJsApiNetworkOnInputMethodChanged,
               &NetworkScreenHandler::HandleOnInputMethodChanged);
 }
 
 // NetworkScreenHandler, private: ----------------------------------------------
 
-void NetworkScreenHandler::HandleOnExit(const ListValue* args) {
+void NetworkScreenHandler::HandleOnExit() {
   ClearErrors();
   if (screen_)
     screen_->OnContinuePressed();
 }
 
-void NetworkScreenHandler::HandleOnLanguageChanged(const ListValue* args) {
-  DCHECK(args->GetSize() == 1);
-  std::string locale;
-  if (!args->GetString(0, &locale))
-    NOTREACHED();
-
+void NetworkScreenHandler::HandleOnLanguageChanged(const std::string& locale) {
   const std::string app_locale = g_browser_process->GetApplicationLocale();
   if (app_locale == locale)
     return;
 
   // TODO(altimofeev): make language change async.
   LanguageSwitchMenu::SwitchLanguageAndEnableKeyboardLayouts(locale);
 
   DictionaryValue localized_strings;
   static_cast<OobeUI*>(web_ui()->GetController())->GetLocalizedStrings(
       &localized_strings);
   CallJS("cr.ui.Oobe.reloadContent", localized_strings);
   // Buttons are recreated, updated "Continue" button state.
   EnableContinue(is_continue_enabled_);
 }
 
-void NetworkScreenHandler::HandleOnInputMethodChanged(const ListValue* args) {
-  DCHECK(args->GetSize() == 1);
-  std::string id;
-  if (!args->GetString(0, &id))
-    NOTREACHED();
+void NetworkScreenHandler::HandleOnInputMethodChanged(const std::string& id) {
   input_method::GetInputMethodManager()->ChangeInputMethod(id);
 }
 
 ListValue* NetworkScreenHandler::GetLanguageList() {
   const std::string app_locale = g_browser_process->GetApplicationLocale();
   input_method::InputMethodManager* manager =
       input_method::GetInputMethodManager();
   // GetSupportedInputMethods() never returns NULL.
   scoped_ptr<input_method::InputMethodDescriptors> descriptors(
       manager->GetSupportedInputMethods());
@@ skipping 39 matching lines @@
         "title",
         util->GetInputMethodLongName(input_methods->at(i)));
     input_method->SetBoolean("selected",
         ime_id == current_input_method_id);
     input_methods_list->Append(input_method);
   }
   return input_methods_list;
 }
 
 }  // namespace chromeos