Reviewers: rossberg,

Message:
It turns out the the blink dev tools needs to be able to fire a microtask with script on the stack. The basic idea here is if the embedder is taking control of
microtask delivery, it should have freedom to do what it knows is correct.

Description:
Assert that either autorun mictasks is off or stack frame count is 0

R=rossberg
BUG=

Please review this at https://codereview.chromium.org/205243012/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+2, -1 lines):
  M src/v8.cc


Index: src/v8.cc
diff --git a/src/v8.cc b/src/v8.cc
index b49e0eb5f2789240bb4e3742e83c8d0eaca29a6e..edf862c6387ac161770749bd74ede5b9d29915ee 100644
--- a/src/v8.cc
+++ b/src/v8.cc
@@ -175,7 +175,8 @@ void V8::RunMicrotasks(Isolate* isolate) {

   HandleScopeImplementer* handle_scope_implementer =
       isolate->handle_scope_implementer();
-  ASSERT(handle_scope_implementer->CallDepthIsZero());
+  ASSERT(!isolate->autorun_microtasks() ||
+         handle_scope_implementer->CallDepthIsZero());

   // Increase call depth to prevent recursive callbacks.
   handle_scope_implementer->IncrementCallDepth();


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to