Deactivate IDBTransactions created within Microtasks

Source/core/inspector/AsyncCallStackTracker.cpp

Index: Source/core/inspector/AsyncCallStackTracker.cpp
diff --git a/Source/core/inspector/AsyncCallStackTracker.cpp b/Source/core/inspector/AsyncCallStackTracker.cpp
index 345ef307e8e62717c9f6606af9d20b509a1d290a..e416ac74151f653e8d0a62c529d820a33b4fe379 100644
--- a/Source/core/inspector/AsyncCallStackTracker.cpp
+++ b/Source/core/inspector/AsyncCallStackTracker.cpp
@@ -35,6 +35,7 @@
 #include "bindings/core/v8/V8RecursionScope.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/dom/ExecutionContextTask.h"
+#include "core/dom/Microtask.h"
 #include "core/events/Event.h"
 #include "core/events/EventTarget.h"
 #include "core/xml/XMLHttpRequest.h"
@@ -417,7 +418,7 @@ PassRefPtrWillBeRawPtr<AsyncCallStackTracker::AsyncCallChain> AsyncCallStackTrac
 
 void AsyncCallStackTracker::setCurrentAsyncCallChain(ExecutionContext* context, PassRefPtrWillBeRawPtr<AsyncCallChain> chain)
 {
-    if (chain && !V8RecursionScope::recursionLevel(toIsolate(context))) {
+    if (chain && (!V8RecursionScope::recursionLevel(toIsolate(context)) || (V8RecursionScope::recursionLevel(toIsolate(context)) == 1 && Microtask::performingCheckpoint(toIsolate(context))))) {

[jsbell, 2014/09/22 17:26:29]

That's a lot of calls to toIsolate(context).

I could pull `v8::Isolate* isolate = toIsolate(context);` out above the if(),
but I'm not sure if this is called frequently with an empty chain and/or how
convoluted we want to make this method to maintain performance - or if that's at
all important.

[adamk, 2014/09/22 18:30:57]

I'd recommend pulling it out just to clean up the code.

[jsbell, 2014/09/22 18:49:51]

Done.

         // Current AsyncCallChain corresponds to the bottommost JS call frame.
         m_currentAsyncCallChain = chain;
         m_nestedAsyncCallCount = 1;