IndexedDB: Ensure transactions created in microtasks are deactivated

Source/bindings/v8/V8RecursionScope.cpp

Index: Source/bindings/v8/V8RecursionScope.cpp
diff --git a/Source/bindings/v8/V8RecursionScope.cpp b/Source/bindings/v8/V8RecursionScope.cpp
index d59765274ae476e156c8736787129bdc3cc13e08..94d4910bcd818f250b3bfc208b92ff85d5c4d8f5 100644
--- a/Source/bindings/v8/V8RecursionScope.cpp
+++ b/Source/bindings/v8/V8RecursionScope.cpp
@@ -40,11 +40,11 @@ void V8RecursionScope::didLeaveScriptContext()
 {
     // FIXME: Instrument any work that takes place when script exits to c++ (e.g. Mutation Observers).

[adamk, 2014/06/05 22:20:45]

I think this FIXME is super-old and should go away.

 
+    Microtask::performCheckpoint();
+
     // Indexed DB requires that transactions are created with an internal |active| flag
     // set to true, but the flag becomes false when control returns to the event loop.
     IDBPendingTransactionMonitor::from(m_executionContext).deactivateNewTransactions();
-
-    Microtask::performCheckpoint();
 }
 
 } // namespace WebCore