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)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access)
Attachments
(1 file)
(deleted),
patch
|
ginnchen+exoracle
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•18 years ago
|
||
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.
Assignee | ||
Comment 2•18 years ago
|
||
Attachment #251027 -
Flags: review?(ginn.chen)
Attachment #251027 -
Flags: review?(ginn.chen) → review+
Assignee | ||
Updated•18 years ago
|
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.
Description
•