Revert recent changes to base::Value

chrome/test/webdriver/webdriver_dispatch.cc

 // Copyright (c) 2011 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/test/webdriver/webdriver_dispatch.h"
 
 #include <sstream>
 #include <string>
 #include <vector>
 
@@ skipping 134 matching lines @@
   ErrorCode status = command_response.GetStatus();
   switch (status) {
     case kSuccess:
       http_response->set_status(HttpResponse::kOk);
       break;
 
     // TODO(jleyba): kSeeOther, kBadRequest, kSessionNotFound,
     // and kMethodNotAllowed should be detected before creating
     // a command_response, and should thus not need conversion.
     case kSeeOther: {
-      Value* value = command_response.GetValue();
+      const Value* const value = command_response.GetValue();
       std::string location;
       if (!value->GetAsString(&location)) {
         // This should never happen.
         http_response->set_status(HttpResponse::kInternalServerError);
         http_response->SetBody("Unable to set 'Location' header: response "
                                "value is not a string: " +
                                command_response.ToJSON());
         return;
       }
       http_response->AddHeader("Location", location);
       http_response->set_status(HttpResponse::kSeeOther);
       break;
     }
 
     case kBadRequest:
     case kSessionNotFound:
       http_response->set_status(status);
       break;
 
     case kMethodNotAllowed: {
-      Value* value = command_response.GetValue();
-      ListValue* list_value = value->AsList();
-      if (!list_value) {
+      const Value* const value = command_response.GetValue();
+      if (!value->IsType(Value::TYPE_LIST)) {
         // This should never happen.
         http_response->set_status(HttpResponse::kInternalServerError);
         http_response->SetBody(
             "Unable to set 'Allow' header: response value was "
             "not a list of strings: " + command_response.ToJSON());
         return;
       }
 
+      const ListValue* const list_value =
+          static_cast<const ListValue* const>(value);
       std::vector<std::string> allowed_methods;
       for (size_t i = 0; i < list_value->GetSize(); ++i) {
         std::string method;
         if (list_value->GetString(i, &method)) {
           allowed_methods.push_back(method);
         } else {
           // This should never happen.
           http_response->set_status(HttpResponse::kInternalServerError);
           http_response->SetBody(
               "Unable to set 'Allow' header: response value was "
@@ skipping 134 matching lines @@
 void Dispatcher::SetNotImplemented(const std::string& pattern) {
   mg_set_uri_callback(context_, (root_ + pattern).c_str(),
                       &SendNotImplementedError, NULL);
 }
 
 void Dispatcher::ForbidAllOtherRequests() {
   mg_set_uri_callback(context_, "*", &SendForbidden, NULL);
 }
 
 }  // namespace webdriver