Revert of Change base::Value::ListStorage to std::vector<base::Value>

extensions/renderer/api_request_handler.cc

 // Copyright 2016 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 "extensions/renderer/api_request_handler.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/memory/ptr_util.h"
 #include "base/values.h"
@@ skipping 109 matching lines @@
   v8::Local<v8::Context> context = pending_request.context.Get(isolate);
   v8::Context::Scope context_scope(context);
   std::unique_ptr<content::V8ValueConverter> converter(
       content::V8ValueConverter::create());
   std::vector<v8::Local<v8::Value>> args;
   args.reserve(response_args.GetSize() +
                pending_request.callback_arguments.size());
   for (const auto& arg : pending_request.callback_arguments)
     args.push_back(arg.Get(isolate));
   for (const auto& arg : response_args)
-    args.push_back(converter->ToV8Value(&arg, context));
+    args.push_back(converter->ToV8Value(arg.get(), context));
 
   blink::WebScopedUserGesture user_gesture(pending_request.user_gesture_token);
   if (!error.empty())
     last_error_.SetError(context, error);
 
   // args.size() is converted to int, but args is controlled by chrome and is
   // never close to std::numeric_limits<int>::max.
   call_js_.Run(pending_request.callback.Get(isolate), context, args.size(),
                args.data());
 
@@ skipping 12 matching lines @@
 }
 
 std::set<int> APIRequestHandler::GetPendingRequestIdsForTesting() const {
   std::set<int> result;
   for (const auto& pair : pending_requests_)
     result.insert(pair.first);
   return result;
 }
 
 }  // namespace extensions