Created attachment 486258 [details] [diff] [review] patch It's not honored any more (we cache accessibles, get them from cache, the only affect it has we always invalidate children when AT asks for children, that's not disabled cache really) and not used I think. http://kb.mozillazine.org/Accessibility.disablecache says is mostly for debugging propose.
Created attachment 486262 [details] [diff] [review] patch2 previous patch was wrong
Comment on attachment 486262 [details] [diff] [review] patch2 r=me. But is there some JS file that also has this pref defined where we need to remove it? E. G. this once showed up in about:config, right? So ti must be defined in some prefs.js file or the like. Please check before landing and remove it there, too.
Attachment #486262 - Flags: review?(marco.zehe) → review+
It isn't referred anywhere outside of a11y, it isn't presented in about:config.
All right, thanks for checking! So this is completely useless code now. :) Yeah, let's ditch it! Thanks for finding!
Benjamin, am I allowed to land it while tree is restricted to Gecko 2 beta 7 blockers?
landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/7f42cc721784
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
This isn't already documented anywhere on MDC, but I added a note to Fx4 for developers: https://developer.mozilla.org/en/Firefox_4_for_developers#Other_changes
Keywords: dev-doc-needed → dev-doc-complete
Yes, thank you. Perhaps it makes sense to update article on http://kb.mozillazine.org. I just don't know who is responsible for it.
You need to log in before you can comment on or make changes to this bug.