Closed
Bug 303133
Opened 19 years ago
Closed 19 years ago
Regression prevents outer doc accessible from having children
Categories
(Firefox :: Disability Access, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
Details
(Keywords: access)
Attachments
(1 file)
9.34 KB,
patch
|
parente
:
review+
neil
:
superreview+
mkaply
:
approval1.8b4+
|
Details | Diff | Splinter Review |
When a frame or iframe is finished loading, we are invalidating the children of the outer accessible (which represents the frame or iframe element). Unfortunately, we always create its children on Init() and don't allow rebuilding of the child information. Also bad, we always say there is 1 child even if for some reason we can't get create an inner doc accessible.
Assignee | ||
Comment 1•19 years ago
|
||
Attachment #191441 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #191441 -
Flags: review?(parente)
Comment on attachment 191441 [details] [diff] [review] Have inner and outer doc get each other on demand, and remove a couple unnecessary method overrides in nsOuterDocAccessible In nsDocAccessible, where does mParent get set to a value?
Assignee | ||
Comment 3•19 years ago
|
||
(In reply to comment #2) > (From update of attachment 191441 [details] [diff] [review] [edit]) > In nsDocAccessible, where does mParent get set to a value? > In ::GetParent() + accService->GetAccessibleFor(ownerNode, getter_AddRefs(mParent));
Attachment #191441 -
Flags: review?(parente) → review+
Updated•19 years ago
|
Attachment #191441 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Assignee | ||
Updated•19 years ago
|
Attachment #191441 -
Flags: approval1.8b4?
Comment 4•19 years ago
|
||
Comment on attachment 191441 [details] [diff] [review] Have inner and outer doc get each other on demand, and remove a couple unnecessary method overrides in nsOuterDocAccessible a=mkaply
Attachment #191441 -
Flags: approval1.8b4? → approval1.8b4+
Assignee | ||
Updated•19 years ago
|
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•