Changeset 114458 in webkit

Timestamp:

Apr 17, 2012 4:02:44 PM (12 years ago)

Author:

rafaelw@chromium.org

Message:

[MutationObservers] enable V8RecursionScope debug ASSERT()
​https://bugs.webkit.org/show_bug.cgi?id=84094

Reviewed by Ojan Vafai.

This patch enables the ASSERT() which enforces usage of
V8RecursionScope.

• src/WebKit.cpp:

(WebKit::assertV8RecursionScope):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebKit.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-04-17  Rafael Weinstein  <rafaelw@chromium.org>
+
+        [MutationObservers] enable V8RecursionScope debug ASSERT()
+        https://bugs.webkit.org/show_bug.cgi?id=84094
+
+        Reviewed by Ojan Vafai.
+
+        This patch enables the ASSERT() which enforces usage of
+        V8RecursionScope.
+
+        * src/WebKit.cpp:
+        (WebKit::assertV8RecursionScope):
+
 2012-04-17  Dana Jansens  <danakj@chromium.org>
 

trunk/Source/WebKit/chromium/src/WebKit.cpp

@@ -93 +93 @@
 static void assertV8RecursionScope()
 {
-    // FIXME: Enable when chromium usage of WebScriptInvocation has landed.
-    // ASSERT(!isMainThread() || WebCore::V8RecursionScope::properlyUsed());
+    ASSERT(!isMainThread() || WebCore::V8RecursionScope::properlyUsed());
 }
 #endif