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

Unified Diff: trunk/Source/bindings/scripts/code_generator_v8.pm

Issue 170893003: Revert 167276 "Remove isolatedWorldForEnteredContext() and isola..." (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 6 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | trunk/Source/bindings/tests/results/V8TestInterfaceDocument.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/Source/bindings/scripts/code_generator_v8.pm
===================================================================
--- trunk/Source/bindings/scripts/code_generator_v8.pm (revision 167359)
+++ trunk/Source/bindings/scripts/code_generator_v8.pm (working copy)
@@ -3109,7 +3109,7 @@
# and thus passing it around would cause leakage.
# 2) Errors cannot be cloned (or serialized):
# http://www.whatwg.org/specs/web-apps/current-work/multipage/common-dom-interfaces.html#safe-passing-of-structured-data
- $implementation{nameSpaceInternal}->add(" if (DOMWrapperWorld::current(info.GetIsolate())->isIsolatedWorld()) {\n");
+ $implementation{nameSpaceInternal}->add(" if (isolatedWorldForIsolate(info.GetIsolate())) {\n");
foreach my $attrName (@anyAttributeNames) {
my $setter = "setSerialized" . FirstLetterToUpperCase($attrName);
$implementation{nameSpaceInternal}->add(<<END);
@@ -5138,10 +5138,9 @@
$code .= <<END;
if (wrapper.IsEmpty())
return wrapper;
- DOMWrapperWorld* world = DOMWrapperWorld::current(isolate);
- if (world->isMainWorld()) {
+ if (!isolatedWorldForEnteredContext(isolate)) {
if (Frame* frame = impl->frame())
- frame->script().windowShell(world)->updateDocumentWrapper(wrapper);
+ frame->script().windowShell(DOMWrapperWorld::mainWorld())->updateDocumentWrapper(wrapper);
}
END
}
« no previous file with comments | « no previous file | trunk/Source/bindings/tests/results/V8TestInterfaceDocument.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698