[Extensions Bindings] Remove ScriptContext::DispatchEvent

chrome/renderer/extensions/cast_streaming_native_handler.cc

 // Copyright 2013 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/renderer/extensions/cast_streaming_native_handler.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 15 matching lines @@
 #include "chrome/common/extensions/api/cast_streaming_receiver_session.h"
 #include "chrome/common/extensions/api/cast_streaming_rtp_stream.h"
 #include "chrome/common/extensions/api/cast_streaming_udp_transport.h"
 #include "chrome/renderer/media/cast_receiver_session.h"
 #include "chrome/renderer/media/cast_rtp_stream.h"
 #include "chrome/renderer/media/cast_session.h"
 #include "chrome/renderer/media/cast_udp_transport.h"
 #include "content/public/child/v8_value_converter.h"
 #include "content/public/renderer/media_stream_utils.h"
 #include "extensions/common/extension.h"
+#include "extensions/renderer/extension_bindings_system.h"
 #include "extensions/renderer/script_context.h"
 #include "media/base/audio_parameters.h"
 #include "media/base/limits.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/ip_address.h"
 #include "third_party/WebKit/public/platform/WebMediaStream.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 #include "third_party/WebKit/public/web/WebDOMMediaStreamTrack.h"
 #include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
@@ skipping 241 matching lines @@
 
 }  // namespace
 
 // |last_transport_id_| is the identifier for the next created RTP stream. To
 // create globally unique IDs used for referring to RTP stream objects in
 // browser process, we set its higher 16 bits as HASH(extension_id)&0x7fff, and
 // lower 16 bits as the sequence number of the RTP stream created in the same
 // extension. Collision will happen when the first RTP stream keeps alive after
 // creating another 64k-1 RTP streams in the same extension, which is very
 // unlikely to happen in normal use cases.
-CastStreamingNativeHandler::CastStreamingNativeHandler(ScriptContext* context)
+CastStreamingNativeHandler::CastStreamingNativeHandler(
+    ScriptContext* context,
+    ExtensionBindingsSystem* bindings_system)
     : ObjectBackedNativeHandler(context),
       last_transport_id_(
           context->extension()
               ? (((base::Hash(context->extension()->id()) & 0x7fff) << 16) + 1)
               : 1),
+      bindings_system_(bindings_system),
       weak_factory_(this) {
   RouteFunction("CreateSession", "cast.streaming.session",
                 base::Bind(&CastStreamingNativeHandler::CreateCastSession,
                            weak_factory_.GetWeakPtr()));
   RouteFunction("DestroyCastRtpStream", "cast.streaming.rtpStream",
                 base::Bind(&CastStreamingNativeHandler::DestroyCastRtpStream,
                            weak_factory_.GetWeakPtr()));
   RouteFunction(
       "GetSupportedParamsCastRtpStream", "cast.streaming.rtpStream",
       base::Bind(&CastStreamingNativeHandler::GetSupportedParamsCastRtpStream,
@@ skipping 127 matching lines @@
   const int udp_id = last_transport_id_++;
   udp_transport_map_[udp_id] = std::move(udp_transport);
   callback_args[2] = v8::Integer::New(isolate, udp_id);
   context()->SafeCallFunction(
       v8::Local<v8::Function>::New(isolate, create_callback_), 3,
       callback_args);
   create_callback_.Reset();
 }
 
 void CastStreamingNativeHandler::CallStartCallback(int stream_id) const {
-  v8::Isolate* isolate = context()->isolate();
-  v8::HandleScope handle_scope(isolate);
-  v8::Context::Scope context_scope(context()->v8_context());
-  v8::Local<v8::Array> event_args = v8::Array::New(isolate, 1);
-  event_args->Set(0, v8::Integer::New(isolate, stream_id));
-  context()->DispatchEvent("cast.streaming.rtpStream.onStarted", event_args);
+  base::ListValue event_args;
+  event_args.AppendInteger(stream_id);
+  bindings_system_->DispatchEventInContext("cast.streaming.rtpStream.onStarted",
+                                           &event_args, nullptr, context());
 }
 
 void CastStreamingNativeHandler::CallStopCallback(int stream_id) const {
-  v8::Isolate* isolate = context()->isolate();
-  v8::HandleScope handle_scope(isolate);
-  v8::Context::Scope context_scope(context()->v8_context());
-  v8::Local<v8::Array> event_args = v8::Array::New(isolate, 1);
-  event_args->Set(0, v8::Integer::New(isolate, stream_id));
-  context()->DispatchEvent("cast.streaming.rtpStream.onStopped", event_args);
+  base::ListValue event_args;
+  event_args.AppendInteger(stream_id);
+  bindings_system_->DispatchEventInContext("cast.streaming.rtpStream.onStopped",
+                                           &event_args, nullptr, context());
 }
 
 void CastStreamingNativeHandler::CallErrorCallback(
     int stream_id,
     const std::string& message) const {
-  v8::Isolate* isolate = context()->isolate();
-  v8::HandleScope handle_scope(isolate);
-  v8::Context::Scope context_scope(context()->v8_context());
-  v8::Local<v8::Array> event_args = v8::Array::New(isolate, 2);
-  event_args->Set(0, v8::Integer::New(isolate, stream_id));
-  event_args->Set(
-      1,
-      v8::String::NewFromUtf8(
-          isolate, message.data(), v8::String::kNormalString, message.size()));
-  context()->DispatchEvent("cast.streaming.rtpStream.onError", event_args);
+  base::ListValue event_args;
+  event_args.AppendInteger(stream_id);
+  event_args.AppendString(message);
+  bindings_system_->DispatchEventInContext("cast.streaming.rtpStream.onError",
+                                           &event_args, nullptr, context());
 }
 
 void CastStreamingNativeHandler::DestroyCastRtpStream(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
   CHECK_EQ(1, args.Length());
   CHECK(args[0]->IsInt32());
 
   const int transport_id = args[0]->ToInt32(args.GetIsolate())->Value();
   if (!GetRtpStreamOrThrow(transport_id))
     return;
@@ skipping 504 matching lines @@
     LOG(ERROR) << "Failed to add Cast audio track to media stream.";
   }
   if (!content::AddVideoTrackToMediaStream(std::move(video), true,  // is_remote
                                            true,  // is_readonly
                                            &web_stream)) {
     LOG(ERROR) << "Failed to add Cast video track to media stream.";
   }
 }
 
 }  // namespace extensions