Closed Bug 366520 Opened 18 years ago Closed 18 years ago

Extra unnecessary cache invalidations and events as XUL doc loads

Categories

(Core :: Disability Access APIs, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: aaronlev, Assigned: aaronlev)

References

Details

(Keywords: access)

Attachments

(1 file)

As I was debugging today I noticed that we keep doing InvalidateCacheSubtree() calls as the XUL document loads. That's a lot of extra unnecessary cache invalidations and events firing.

Apparently we act as if the XUL document is loaded as soon as its nsDocAccessible is created. I'm not sure why we need to do that -- there is no comment.

I made that change here, but unfortunately I forgot to put the bug # in the checkin comment:
1.78 <aaronleventhal@moonset.net> 2005-09-30 12:23
Context menus don't work with screen readers starting on 09/21. r=parente, sr=bryner
Ah, it was for bug 310350

The patch description says "1) Mark chrome docs as loaded so that mutation handling works"

The code that does that was added here:
http://lxr.mozilla.org/seamonkey/source/accessible/src/base/nsDocAccessible.cpp#110

Not sure if we still need it for mutation handling to work in XUL.
Attachment #251027 - Flags: review?(ginn.chen) → review+
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: