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

Unified Diff: Source/bindings/v8/V8WindowShell.cpp

Issue 182903003: Make DOMWrapperWorld::current() callable from workers (Closed) Base URL: svn://svn.chromium.org/blink/trunk
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
« Source/bindings/v8/DOMWrapperWorld.h ('K') | « Source/bindings/v8/V8WindowShell.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/v8/V8WindowShell.cpp
diff --git a/Source/bindings/v8/V8WindowShell.cpp b/Source/bindings/v8/V8WindowShell.cpp
index 5f2779896fda7635fe8ceface7710b87c02fb1bf..3e672113ae1ddc79a3163a603e3097ac82fb9e6f 100644
--- a/Source/bindings/v8/V8WindowShell.cpp
+++ b/Source/bindings/v8/V8WindowShell.cpp
@@ -65,8 +65,6 @@
namespace WebCore {
-static bool contextBeingInitialized = false;
-
static void checkDocumentWrapper(v8::Handle<v8::Object> wrapper, Document* document)
{
ASSERT(V8Document::toNative(wrapper) == document);
@@ -184,10 +182,9 @@ bool V8WindowShell::initializeIfNeeded()
if (m_perContextData)
return true;
- ASSERT(!contextBeingInitialized);
- contextBeingInitialized = true;
+ DOMWrapperWorld::setWorldOfInitializingWindow(m_world.get());
bool result = initialize();
- contextBeingInitialized = false;
+ DOMWrapperWorld::setWorldOfInitializingWindow(0);
return result;
}
@@ -500,15 +497,4 @@ void V8WindowShell::updateSecurityOrigin(SecurityOrigin* origin)
setSecurityToken(origin);
}
-bool V8WindowShell::contextHasCorrectPrototype(v8::Handle<v8::Context> context)
-{
- if (!isMainThread())
- return true;
- // We're initializing the context, so it is not yet in a status where we can
- // validate the context.
- if (contextBeingInitialized)
- return true;
- return !!toDOMWindow(context);
-}
-
} // WebCore
« Source/bindings/v8/DOMWrapperWorld.h ('K') | « Source/bindings/v8/V8WindowShell.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698