Closed Bug 1735706 Opened 3 months ago Closed 3 months ago

Non-Windows RemoteAccessible should check cache pref instead of mCachedFields

Categories

(Core :: Disability Access APIs, task)

Desktop
Unspecified
task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: Jamie, Assigned: Jamie)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

On platforms other than Windows, RemoteAccessible currently checks mCachedFields to determine whether to call the base class method to fetch from the cache. After bug 1728437, mCachedFields can be null if all fields are default; i.e. mCachedFields being null doesn't mean the cache is invalid/disabled. We should check the pref instead like we do on Windows.

mCachedFields might be null because all fields are default; i.e. mCachedFields being null doesn't mean the cache is invalid/disabled.
We should check the pref instead when determining whether to call base class methods to query the cache.

Pushed by jteh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d3b53386023f
Check the cache pref instead of mCachedFields in non-Windows RemoteAccessible methods. r=eeejay
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.