[ABANDONED] Enable V8 Promises

Source/core/workers/WorkerRunLoop.cpp

Index: Source/core/workers/WorkerRunLoop.cpp
diff --git a/Source/core/workers/WorkerRunLoop.cpp b/Source/core/workers/WorkerRunLoop.cpp
index e17016b7f7d08143495b40791bfebaec98005af1..d7bab86879820224725517ec67d814afc0cb303a 100644
--- a/Source/core/workers/WorkerRunLoop.cpp
+++ b/Source/core/workers/WorkerRunLoop.cpp
@@ -31,6 +31,7 @@
 #include "config.h"
 #include "core/workers/WorkerRunLoop.h"
 
+#include "bindings/v8/V8RecursionScope.h"
 #include "core/inspector/InspectorInstrumentation.h"
 #include "core/workers/WorkerGlobalScope.h"
 #include "core/workers/WorkerThread.h"
@@ -55,8 +56,10 @@ public:
     virtual void run() OVERRIDE
     {
         WorkerGlobalScope* workerGlobalScope = m_runLoop.context();
-        if ((!workerGlobalScope->isClosing() && !m_runLoop.terminated()) || m_task->isCleanupTask())
+        if ((!workerGlobalScope->isClosing() && !m_runLoop.terminated()) || m_task->isCleanupTask()) {
+            V8RecursionScope recursionScope(workerGlobalScope);

[adamk, 2014/03/28 17:55:05]

This seems like the wrong place to put this, V8RecursionScope is about script
execution, not general task running. Also, by wrapping the entire performTask
call, if the task dispatches events to multiple listeners, we'll get the wrong
end-of-microtask semantics (each event dispatch should end in performCheckpoint,
not just one for the whole task).

And it feels like layer violation for WorkerRunLoop to know about
V8RecursionScope.

             m_task->performTask(workerGlobalScope);
+        }
     }
 
 private: