[Content] Update V8ValueConverter::create to return a std::unique_ptr

extensions/renderer/event_bindings.cc

 // Copyright 2014 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/event_bindings.h"
 
 #include <stdint.h>
 
 #include <map>
 #include <memory>
@@ skipping 200 matching lines @@
     ScriptContext* context) {
   v8::HandleScope handle_scope(context->isolate());
   v8::Context::Scope context_scope(context->v8_context());
 
   v8::Local<v8::Array> listener_ids;
   if (filtering_info && !filtering_info->empty())
     listener_ids = GetMatchingListeners(context, event_name, *filtering_info);
   else
     listener_ids = v8::Array::New(context->isolate());
 
-  std::unique_ptr<content::V8ValueConverter> converter(
-      content::V8ValueConverter::create());
   v8::Local<v8::Value> v8_args[] = {
       gin::StringToSymbol(context->isolate(), event_name),
-      converter->ToV8Value(event_args, context->v8_context()), listener_ids,
+      content::V8ValueConverter::Create()->ToV8Value(event_args,
+                                                     context->v8_context()),
+      listener_ids,
   };
 
   context->module_system()->CallModuleMethodSafe(
       kEventBindings, "dispatchEvent", arraysize(v8_args), v8_args);
 }
 
 void EventBindings::AttachEventHandler(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
   CHECK_EQ(1, args.Length());
   CHECK(args[0]->IsString());
@@ skipping 79 matching lines @@
   CHECK_EQ(2, args.Length());
   CHECK(args[0]->IsString());
   CHECK(args[1]->IsObject());
 
   std::string event_name = *v8::String::Utf8Value(args[0]);
   if (!context()->HasAccessOrThrowError(event_name))
     return;
 
   std::unique_ptr<base::DictionaryValue> filter;
   {
-    std::unique_ptr<content::V8ValueConverter> converter(
-        content::V8ValueConverter::create());
-    std::unique_ptr<base::Value> filter_value(converter->FromV8Value(
-        v8::Local<v8::Object>::Cast(args[1]), context()->v8_context()));
+    std::unique_ptr<base::Value> filter_value =
+        content::V8ValueConverter::Create()->FromV8Value(
+            v8::Local<v8::Object>::Cast(args[1]), context()->v8_context());
     if (!filter_value || !filter_value->IsType(base::Value::Type::DICTIONARY)) {
       args.GetReturnValue().Set(static_cast<int32_t>(-1));
       return;
     }
     filter = base::DictionaryValue::From(std::move(filter_value));
   }
 
   int id = g_event_filter.Get().AddEventMatcher(
       event_name, ParseEventMatcher(std::move(filter)));
   if (id == -1) {
@@ skipping 95 matching lines @@
   for (int matcher_id : attached_matcher_ids_safe) {
     DetachFilteredEvent(matcher_id, false /* is_manual */);
   }
   DCHECK(attached_matcher_ids_.empty())
       << "Filtered events cannot be attached during invalidation";
 
   g_unmanaged_listeners.Get().erase(context());
 }
 
 }  // namespace extensions