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

extensions/renderer/api_event_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_event_handler.h"
 
 #include <algorithm>
 #include <map>
 #include <memory>
 #include <vector>
@@ skipping 202 matching lines @@
   // object (including an array), other listeners will see that modification.
   // TODO(devlin): This is how it's always been, but should it be?
   std::unique_ptr<content::V8ValueConverter> converter(
       content::V8ValueConverter::create());
 
   auto massager_iter = data->massagers.find(event_name);
   if (massager_iter == data->massagers.end()) {
     std::vector<v8::Local<v8::Value>> v8_args;
     v8_args.reserve(args.GetSize());
     for (const auto& arg : args)
-      v8_args.push_back(converter->ToV8Value(&arg, context));
+      v8_args.push_back(converter->ToV8Value(arg.get(), context));
     emitter->Fire(context, &v8_args, &filter);
   } else {
     v8::Isolate* isolate = context->GetIsolate();
     v8::HandleScope handle_scope(isolate);
     v8::Local<v8::Function> massager = massager_iter->second.Get(isolate);
     v8::Local<v8::Value> v8_args = converter->ToV8Value(&args, context);
     DCHECK(!v8_args.IsEmpty());
     DCHECK(v8_args->IsArray());
 
     // Curry in the native dispatch function. Some argument massagers take
@@ skipping 67 matching lines @@
   auto iter = data->emitters.find(event_name);
   DCHECK(iter != data->emitters.end());
   EventEmitter* emitter = nullptr;
   gin::Converter<EventEmitter*>::FromV8(
       context->GetIsolate(), iter->second.Get(context->GetIsolate()), &emitter);
   CHECK(emitter);
   return emitter->GetNumListeners();
 }
 
 }  // namespace extensions