Bug 935567 - stop calling atk_focus_tracker_notify when handling focus, r=tbsaunde
authorAlexander Surkov <surkov.alexander@gmail.com>
Thu, 07 Nov 2013 08:24:47 -0500
changeset 154006 de54ccd868aa659015e006b482b6a20fb2197e08
parent 154005 17d0e890dd3b690a6ffeaf032bdd0f3161e4f068
child 154007 aa013ffd209b94cb06151a39c5212884cefce0aa
push id25617
push userryanvm@gmail.com
push dateThu, 07 Nov 2013 19:45:30 +0000
treeherdermozilla-central@545887140a1b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstbsaunde
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
Bug 935567 - stop calling atk_focus_tracker_notify when handling focus, r=tbsaunde
accessible/src/atk/AccessibleWrap.cpp
--- a/accessible/src/atk/AccessibleWrap.cpp
+++ b/accessible/src/atk/AccessibleWrap.cpp
@@ -977,17 +977,16 @@ 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
             nsRefPtr<AccEvent> stateChangeEvent =
               new AccStateChangeEvent(accessible, states::FOCUSED, true);
             return FireAtkStateChangeEvent(stateChangeEvent, atkObj);
         }
       } break;
 
     case nsIAccessibleEvent::EVENT_NAME_CHANGE:
@@ -1164,17 +1163,16 @@ 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;