EVENT_STATE_CHANGE with STATE_BUSY flag cleared is fired too early for new page loads

RESOLVED FIXED

Status

()

Core
Disability Access APIs
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

16 years ago
We are notifying assistive technology that a page is finished loaded when it is
first displayed, instead of when the text of the page is completely loaded.

We should wait to fire the event until the page is completely loaded, otherwise
assistive tech has difficulty loading the page contents into its own buffer.
(Assignee)

Comment 1

16 years ago
Waiting to find out from Bill Smith if we're firing this event at the right time
now, in a test build I gave him.
(Assignee)

Comment 2

16 years ago
Created attachment 93426 [details] [diff] [review]
Fires final MSAA EVENT_STATE_CHANGE event onwebprogresslistener's STATE_STOP, instead of new window creation
(Assignee)

Comment 3

16 years ago
Created attachment 93763 [details] [diff] [review]
Same solution, better patch
Attachment #93426 - Attachment is obsolete: true
(Assignee)

Comment 4

16 years ago
Created attachment 93778 [details] [diff] [review]
Deterministic solution - just use progress listener's STATE_STOP to tell us when page is finished. Doesn't appear that window is ever hidden when this happens.
Attachment #93763 - Attachment is obsolete: true

Comment 5

16 years ago
Comment on attachment 93778 [details] [diff] [review]
Deterministic solution - just use progress listener's STATE_STOP to tell us when page is finished. Doesn't appear that window is ever hidden when this happens.

r=jgaunt
Attachment #93778 - Flags: review+
Comment on attachment 93778 [details] [diff] [review]
Deterministic solution - just use progress listener's STATE_STOP to tell us when page is finished. Doesn't appear that window is ever hidden when this happens.

I'd make sure that this works ok in places where there are multiple documents
in the page (framesets, iframes).  Assuming that all works fine, sr=bryner.
Attachment #93778 - Flags: superreview+
(Assignee)

Comment 7

16 years ago
cbecked in a long time ago.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.