Update the accessibility tree when a modal dialog is opened/closed.

Source/core/accessibility/AXObject.cpp

 /*
  * Copyright (C) 2008, 2009, 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 271 matching lines @@
         return IgnoreObject;
 
     if (isPresentationalChildOfAriaRole())
         return IgnoreObject;
 
     return accessibilityPlatformIncludesObject();
 }
 
 bool AXObject::isInertOrAriaHidden() const
 {
-    if (equalIgnoringCase(getAttribute(aria_hiddenAttr), "true"))
-        return true;
-    if (node() && node()->isInert())
-        return true;
-
-    for (AXObject* object = parentObject(); object; object = object->parentObject()) {
+    bool canBeInInertSubtree = true;

[dmazzoni, 2013/11/07 16:35:59]

Do you mean "is allowed to be in inert subtreee" or "might possibly be in inert
subtree"?

If the latter, how about mightBeInInsertSubtree?

[falken, 2013/11/08 05:08:35]

Done.

+    for (const AXObject* object = this; object; object = object->parentObject()) {
         if (equalIgnoringCase(object->getAttribute(aria_hiddenAttr), "true"))
             return true;
+        if (canBeInInertSubtree && object->node()) {
+            if (object->node()->isInert())
+                return true;
+            canBeInInertSubtree = false;
+        }
     }
 
     return false;
 }
 
 bool AXObject::lastKnownIsIgnoredValue()
 {
     if (m_lastKnownIsIgnoredValue == DefaultBehavior)
         m_lastKnownIsIgnoredValue = accessibilityIsIgnored() ? IgnoreObject : IncludeObject;
 
@@ skipping 499 matching lines @@
             offsetY -= (scrollPosition.y() + point.y());
             point.move(scrollPosition.x() - innerRect.x(), scrollPosition.y() - innerRect.y());
         } else if (inner->isAXScrollView()) {
             // Otherwise, if the inner object is a scroll view, reset the coordinate transformation.
             offsetX = 0;
             offsetY = 0;
         }
     }
 }
 
-void AXObject::notifyIfIgnoredValueChanged()
+bool AXObject::notifyIfIgnoredValueChanged()
 {
     bool isIgnored = accessibilityIsIgnored();
     if (lastKnownIsIgnoredValue() != isIgnored) {
         axObjectCache()->childrenChanged(parentObject());
         setLastKnownIsIgnoredValue(isIgnored);
+        return true;
     }
+    return false;
 }
 
 void AXObject::selectionChanged()
 {
     if (AXObject* parent = parentObjectIfExists())
         parent->selectionChanged();
 }
 
 int AXObject::lineForPosition(const VisiblePosition& visiblePos) const
 {
@@ skipping 61 matching lines @@
         return ToggleButtonRole;
     if (ariaHasPopup())
         return PopUpButtonRole;
     // We don't contemplate RadioButtonRole, as it depends on the input
     // type.
 
     return ButtonRole;
 }
 
 } // namespace WebCore