Index: extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc |
diff --git a/extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc b/extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc |
index 021636e1d5dfbde01bc6ec4cfad54a96743cc05a..bec7d1c852016119b6a3df2e7c81ffbda0047867 100644 |
--- a/extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc |
+++ b/extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc |
@@ -427,11 +427,14 @@ void GuestViewInternalCustomBindings::RegisterView( |
void GuestViewInternalCustomBindings::RunWithGesture( |
const v8::FunctionCallbackInfo<v8::Value>& args) { |
// Gesture is required to request fullscreen. |
+ // TODO(devlin): All this needs to do is enter fullscreen. We should make this |
+ // EnterFullscreen() and do it directly rather than having a generic "run with |
+ // user gesture" function. |
blink::WebScopedUserGesture user_gesture(context()->web_frame()); |
CHECK_EQ(args.Length(), 1); |
CHECK(args[0]->IsFunction()); |
- v8::Local<v8::Value> no_args; |
- context()->CallFunction(v8::Local<v8::Function>::Cast(args[0]), 0, &no_args); |
+ context()->SafeCallFunction( |
+ v8::Local<v8::Function>::Cast(args[0]), 0, nullptr); |
} |
} // namespace extensions |