[PATCH 2/2] revert bug 935567 because it unnecessarily breaks
authorTrevor Saunders <trev.saunders@gmail.com>
Mon, 18 Nov 2013 08:16:21 -0500
changeset 155196 81f5a2afce6e71790205fc68e865935da3aead3c
parent 155195 fa48287a7eca41c0c68d2ece09e2acedbcf5b8ed
child 155197 46b36ddc64622ab6159eb3f706251c01eb7b3e06
push id36274
push usertrev.saunders@gmail.com
push dateMon, 18 Nov 2013 13:22:12 +0000
treeherdermozilla-inbound@81f5a2afce6e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs935567
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
[PATCH 2/2] revert bug 935567 because it unnecessarily breaks From 81f1fe2ccb9710ca2a956e1993ffef9e8b970c09 Mon Sep 17 00:00:00 2001 backward compat --- accessible/src/atk/AccessibleWrap.cpp | 2 ++ 1 file changed, 2 insertions(+)
accessible/src/atk/AccessibleWrap.cpp
--- a/accessible/src/atk/AccessibleWrap.cpp
+++ b/accessible/src/atk/AccessibleWrap.cpp
@@ -991,16 +991,17 @@ AccessibleWrap::HandleAccEvent(AccEvent*
     case nsIAccessibleEvent::EVENT_TEXT_REMOVED:
     case nsIAccessibleEvent::EVENT_TEXT_INSERTED:
         return FireAtkTextChangedEvent(aEvent, atkObj);
 
     case nsIAccessibleEvent::EVENT_FOCUS:
       {
         a11y::RootAccessible* rootAccWrap = accWrap->RootAccessible();
         if (rootAccWrap && rootAccWrap->mActivated) {
+            atk_focus_tracker_notify(atkObj);
             // Fire state change event for focus
             atk_object_notify_state_change(atkObj, ATK_STATE_FOCUSED, true);
             return NS_OK;
         }
       } break;
 
     case nsIAccessibleEvent::EVENT_NAME_CHANGE:
       {
@@ -1176,16 +1177,17 @@ AccessibleWrap::HandleAccEvent(AccEvent*
         g_signal_emit_by_name (atkObj, "reload");
       break;
 
     case nsIAccessibleEvent::EVENT_DOCUMENT_LOAD_STOPPED:
         g_signal_emit_by_name (atkObj, "load_stopped");
       break;
 
     case nsIAccessibleEvent::EVENT_MENUPOPUP_START:
+        atk_focus_tracker_notify(atkObj); // fire extra focus event
         atk_object_notify_state_change(atkObj, ATK_STATE_VISIBLE, true);
         atk_object_notify_state_change(atkObj, ATK_STATE_SHOWING, true);
         break;
 
     case nsIAccessibleEvent::EVENT_MENUPOPUP_END:
         atk_object_notify_state_change(atkObj, ATK_STATE_VISIBLE, false);
         atk_object_notify_state_change(atkObj, ATK_STATE_SHOWING, false);
         break;