Send AX focus changed events for modal dialog open/close

Source/core/dom/Document.cpp

Index: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index 5721682d37bbe6c898fe8031d0ab2fa09ba1da97..6fc5cadadbd44343b9d652b914079680ef08a4f7 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -3410,10 +3410,10 @@ bool Document::setFocusedElement(PassRefPtr<Element> prpNewFocusedElement, Focus
         }
     }
 
-    if (!focusChangeBlocked && m_focusedElement) {
+    if (m_focusedElement != oldFocusedElement) {

[dmazzoni, 2013/12/10 08:07:53]

Did you mean to get rid of the test for !focusChangeBlocked? Why?

[falken, 2013/12/10 12:19:57]

Hum, I remember needing to get rid of it (and thinking it makes sense to always
tell AT if focus changed), but upon re-testing it's OK to keep it in. Probably
it's best to leave it in then.

         // Create the AXObject cache in a focus change because Chromium relies on it.
         if (AXObjectCache* cache = axObjectCache())
-            cache->handleFocusedUIElementChanged(oldFocusedElement.get(), newFocusedElement.get());
+            cache->handleFocusedUIElementChanged(*this);
     }
 
     if (!focusChangeBlocked && page())