Use V8RecursionScope::MicrotaskSuppression at ChromeClinetImpl::postAccessibilityNotification

Source/web/ChromeClientImpl.cpp

Index: Source/web/ChromeClientImpl.cpp
diff --git a/Source/web/ChromeClientImpl.cpp b/Source/web/ChromeClientImpl.cpp
index f28aec86e7e712dbeb3bc4b9686c734b380ae56e..a64dd4e579ba5c7886c7b225486ed65906f572a9 100644
--- a/Source/web/ChromeClientImpl.cpp
+++ b/Source/web/ChromeClientImpl.cpp
@@ -67,6 +67,7 @@
 #include "WebViewImpl.h"
 #include "WebWindowFeatures.h"
 #include "bindings/v8/ScriptController.h"
+#include "bindings/v8/V8RecursionScope.h"
 #include "core/accessibility/AXObject.h"
 #include "core/accessibility/AXObjectCache.h"
 #include "core/dom/Document.h"
@@ -678,6 +679,8 @@ void ChromeClientImpl::formStateDidChange(const Node* node)
 
 void ChromeClientImpl::postAccessibilityNotification(AXObject* obj, AXObjectCache::AXNotification notification)
 {
+    V8RecursionScope::MicrotaskSuppression scope;

[haraken, 2014/02/24 11:06:09]

hmm, this looks fragile. We don't want to add MicrotaskSuppression to all web/
methods that can potentially invoke Function::Call(), Script::Run() etc.

BTW, what is MicrotaskSuppression for? MicrotaskSuppression is doing nothing
except for checking recursion level. I think that V8 already has a mechanism to
throw an exception when a recursion level is exceeded, so probably can we simply
remove MicrotaskSuppression completely?

+
     // Alert assistive technology about the accessibility object notification.
     if (!obj)
         return;