Bug 1461912 - do not clear accessible actors cache when document becomes non-busy. r=pbro
authorYura Zenevich <yura.zenevich@gmail.com>
Fri, 18 May 2018 13:43:02 -0400
changeset 419334 ba4ce98ebac5545372dd9ef98e25d6cb763d0566
parent 419333 abf3732da1308121cdcf98a963364a769fe1444b
child 419335 11740609e1dba90ff53604ea369c0cdb95dc48d5
push id103501
push useryura.zenevich@gmail.com
push dateTue, 22 May 2018 13:50:12 +0000
treeherdermozilla-inbound@ba4ce98ebac5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbro
bugs1461912
milestone62.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 1461912 - do not clear accessible actors cache when document becomes non-busy. r=pbro MozReview-Commit-ID: 9DwTcF3w4Eo
devtools/server/actors/accessibility.js
--- a/devtools/server/actors/accessibility.js
+++ b/devtools/server/actors/accessibility.js
@@ -601,22 +601,16 @@ const AccessibleWalkerActor = ActorClass
         events.emit(this, "document-ready", rawAccessible);
       }
     }
 
     switch (event.eventType) {
       case EVENT_STATE_CHANGE:
         let { state, isEnabled } = event.QueryInterface(nsIAccessibleStateChangeEvent);
         let isBusy = state & Ci.nsIAccessibleStates.STATE_BUSY;
-        // Accessible document is recreated.
-        if (isBusy && !isEnabled && rawAccessible instanceof Ci.nsIAccessibleDocument) {
-          // Remove its existing cache from tree.
-          this.purgeSubtree(rawAccessible, event.DOMNode);
-        }
-
         if (accessible) {
           // Only propagate state change events for active accessibles.
           if (isBusy && isEnabled) {
             if (rawAccessible instanceof Ci.nsIAccessibleDocument) {
               // Remove its existing cache from tree.
               this.purgeSubtree(rawAccessible, event.DOMNode);
             }
             return;