[Extensions] Convert some callers of ScriptContext::CallFunction

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 422 matching lines @@
     rtp_stream_map_[stream1_id] = std::move(stream1);
   }
   if (stream2) {
     const int stream2_id = last_transport_id_++;
     callback_args[1] = v8::Integer::New(isolate, stream2_id);
     rtp_stream_map_[stream2_id] = std::move(stream2);
   }
   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()->CallFunction(
+  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);
@@ skipping 251 matching lines @@
   v8::HandleScope handle_scope(isolate);
   v8::Context::Scope context_scope(context()->v8_context());
 
   RtpStreamCallbackMap::iterator it =
       get_raw_events_callbacks_.find(transport_id);
   if (it == get_raw_events_callbacks_.end())
     return;
   std::unique_ptr<V8ValueConverter> converter(V8ValueConverter::create());
   v8::Local<v8::Value> callback_args[] = {
       converter->ToV8Value(raw_events.get(), context()->v8_context())};
-  context()->CallFunction(v8::Local<v8::Function>::New(isolate, it->second),
-                          arraysize(callback_args), callback_args);
+  context()->SafeCallFunction(v8::Local<v8::Function>::New(isolate, it->second),
+                              arraysize(callback_args), callback_args);
   get_raw_events_callbacks_.erase(it);
 }
 
 void CastStreamingNativeHandler::CallGetStatsCallback(
     int transport_id,
     std::unique_ptr<base::DictionaryValue> stats) {
   v8::Isolate* isolate = context()->isolate();
   v8::HandleScope handle_scope(isolate);
   v8::Context::Scope context_scope(context()->v8_context());
 
   RtpStreamCallbackMap::iterator it = get_stats_callbacks_.find(transport_id);
   if (it == get_stats_callbacks_.end())
     return;
 
   std::unique_ptr<V8ValueConverter> converter(V8ValueConverter::create());
   v8::Local<v8::Value> callback_args[] = {
       converter->ToV8Value(stats.get(), context()->v8_context())};
-  context()->CallFunction(v8::Local<v8::Function>::New(isolate, it->second),
-                          arraysize(callback_args), callback_args);
+  context()->SafeCallFunction(v8::Local<v8::Function>::New(isolate, it->second),
+                              arraysize(callback_args), callback_args);
   get_stats_callbacks_.erase(it);
 }
 
 CastRtpStream* CastStreamingNativeHandler::GetRtpStreamOrThrow(
     int transport_id) const {
   RtpStreamMap::const_iterator iter = rtp_stream_map_.find(
       transport_id);
   if (iter != rtp_stream_map_.end())
     return iter->second.get();
   v8::Isolate* isolate = context()->v8_context()->GetIsolate();
@@ skipping 218 matching lines @@
 }
 
 void CastStreamingNativeHandler::CallReceiverErrorCallback(
     v8::CopyablePersistentTraits<v8::Function>::CopyablePersistent function,
     const std::string& error_message) {
   v8::Isolate* isolate = context()->v8_context()->GetIsolate();
   v8::Local<v8::Value> arg = v8::String::NewFromUtf8(isolate,
                                                       error_message.data(),
                                                       v8::String::kNormalString,
                                                       error_message.size());
-  context()->CallFunction(
-      v8::Local<v8::Function>::New(isolate, function), 1, &arg);
+  context()->SafeCallFunction(v8::Local<v8::Function>::New(isolate, function),
+                              1, &arg);
 }
 
 void CastStreamingNativeHandler::AddTracksToMediaStream(
     const std::string& url,
     const media::AudioParameters& params,
     scoped_refptr<media::AudioCapturerSource> audio,
     std::unique_ptr<media::VideoCapturerSource> video) {
   blink::WebMediaStream web_stream =
       blink::WebMediaStreamRegistry::lookupMediaStreamDescriptor(GURL(url));
   if (web_stream.isNull()) {
     LOG(DFATAL) << "Stream not found.";
     return;
   }
   if (!content::AddAudioTrackToMediaStream(
           audio, params.sample_rate(), params.channel_layout(),
           params.frames_per_buffer(), true,  // is_remote
           true,                              // is_readonly
           &web_stream)) {
     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