Make Use of Value::GetList API

chrome/browser/ui/webui/ntp/app_launcher_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/ntp/app_launcher_handler.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 618 matching lines @@
   UMA_HISTOGRAM_ENUMERATION("Apps.AppInfoDialog.Launches",
                             AppInfoLaunchSource::FROM_APPS_PAGE,
                             AppInfoLaunchSource::NUM_LAUNCH_SOURCES);
 
   ShowAppInfoInNativeDialog(
       web_ui()->GetWebContents(), GetAppInfoNativeDialogSize(),
       Profile::FromWebUI(web_ui()), extension, base::Closure());
 }
 
 void AppLauncherHandler::HandleReorderApps(const base::ListValue* args) {
-  CHECK(args->GetSize() == 2);
-
-  std::string dragged_app_id;
-  const base::ListValue* app_order;
-  CHECK(args->GetString(0, &dragged_app_id));
-  CHECK(args->GetList(1, &app_order));
+  CHECK(args->base::Value::GetList().size() == 2);
+  std::string dragged_app_id = args->base::Value::GetList()[0].GetString();
+  const base::ListValue* app_order =
+      static_cast<const base::ListValue*>(&args->base::Value::GetList()[1]);
 
   std::string predecessor_to_moved_ext;
   std::string successor_to_moved_ext;
   for (size_t i = 0; i < app_order->GetSize(); ++i) {
     std::string value;
     if (app_order->GetString(i, &value) && value == dragged_app_id) {
       if (i > 0)
         CHECK(app_order->GetString(i - 1, &predecessor_to_moved_ext));
       if (i + 1 < app_order->GetSize())
         CHECK(app_order->GetString(i + 1, &successor_to_moved_ext));
@@ skipping 228 matching lines @@
       base::Value(!extension_id_prompting_.empty()));
 }
 
 bool AppLauncherHandler::ShouldShow(const Extension* extension) const {
   if (ignore_changes_ || !has_loaded_apps_ || !extension->is_app())
     return false;
 
   Profile* profile = Profile::FromWebUI(web_ui());
   return extensions::ui_util::ShouldDisplayInNewTabPage(extension, profile);
 }