Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1143)

Side by Side Diff: extensions/renderer/guest_view/extensions_guest_view_container.cc

Issue 1804043002: Revert of Remove V8RecrusionScope, cleanup call sites. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/renderer/guest_view/extensions_guest_view_container.h" 5 #include "extensions/renderer/guest_view/extensions_guest_view_container.h"
6 6
7 #include "content/public/renderer/render_frame.h" 7 #include "content/public/renderer/render_frame.h"
8 #include "third_party/WebKit/public/web/WebScopedMicrotaskSuppression.h"
8 #include "ui/gfx/geometry/size.h" 9 #include "ui/gfx/geometry/size.h"
9 10
10 namespace extensions { 11 namespace extensions {
11 12
12 ExtensionsGuestViewContainer::ExtensionsGuestViewContainer( 13 ExtensionsGuestViewContainer::ExtensionsGuestViewContainer(
13 content::RenderFrame* render_frame) 14 content::RenderFrame* render_frame)
14 : GuestViewContainer(render_frame), 15 : GuestViewContainer(render_frame),
15 element_resize_isolate_(nullptr), 16 element_resize_isolate_(nullptr),
16 weak_ptr_factory_(this) { 17 weak_ptr_factory_(this) {
17 } 18 }
(...skipping 30 matching lines...) Expand all
48 v8::Local<v8::Context> context = callback->CreationContext(); 49 v8::Local<v8::Context> context = callback->CreationContext();
49 if (context.IsEmpty()) 50 if (context.IsEmpty())
50 return; 51 return;
51 52
52 const int argc = 2; 53 const int argc = 2;
53 v8::Local<v8::Value> argv[argc] = { 54 v8::Local<v8::Value> argv[argc] = {
54 v8::Integer::New(element_resize_isolate_, new_size.width()), 55 v8::Integer::New(element_resize_isolate_, new_size.width()),
55 v8::Integer::New(element_resize_isolate_, new_size.height())}; 56 v8::Integer::New(element_resize_isolate_, new_size.height())};
56 57
57 v8::Context::Scope context_scope(context); 58 v8::Context::Scope context_scope(context);
58 v8::MicrotasksScope microtasks( 59 blink::WebScopedMicrotaskSuppression suppression;
59 element_resize_isolate_, v8::MicrotasksScope::kDoNotRunMicrotasks);
60 60
61 callback->Call(context->Global(), argc, argv); 61 callback->Call(context->Global(), argc, argv);
62 } 62 }
63 63
64 } // namespace extensions 64 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/renderer/activity_log_converter_strategy_unittest.cc ('k') | extensions/renderer/messaging_bindings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698