174796 - XUL browser's MSAA cache not working correctly

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

I have experienced a number of problems with the IEnumVARIANT MSAA Cache checked
in this summer.
- Incorrect or repeated nodes
- cache not getting reset for new documents
- no way to test with cache turned off, to see where problem lies

Comment 1

[Aaron Leventhal]

Attachment: Fixes IEnumVARIANT::Next() loop, implements hidden cache prefs, clears cache when new doc is loaded

These are the 2 new user prefs for controlling the cache:
user_pref("accessibility.disablecache", true/false);
user_pref("accessibility.disableenumvariant", true/false);

Comment 2

[Aaron Leventhal]

Seeking r=jgaunt

Comment 3

[Aaron Leventhal]

*** Bug 174949 has been marked as a duplicate of this bug. ***

Comment 4

[John Gaunt (redfive)]

Comment on attachment 103069 [details] [diff] [review]
Fixes IEnumVARIANT::Next() loop, implements hidden cache prefs, clears cache when new doc is loaded

r=jgaunt

Comment 5

[Brian Ryner (not reading)]

Comment on attachment 103069 [details] [diff] [review]
Fixes IEnumVARIANT::Next() loop, implements hidden cache prefs, clears cache when new doc is loaded

sr=bryner

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 103069 [details] [diff] [review]
Fixes IEnumVARIANT::Next() loop, implements hidden cache prefs, clears cache when new doc is loaded

a=roc+moz for trunk

Comment 7

[Christopher Blizzard (:blizzard)]

This has approval.  If it doesn't get checked in by the early morning tree
closure on Nov 5, 2002 it's going to have to wait until after the branch is cut.

Comment 8

[Aaron Leventhal]

Fixed

Comment 9

[Christopher Blizzard (:blizzard)]

Was this checked in before the 1.2 branch was cut?  Or was it checked in onto
the branch?

Comment 10

[Aaron Leventhal]

checked in before the cut. (I thought the cut occured on the 5th?!).