Make readyState progress through all states without duplicate transitions

RESOLVED FIXED in mozilla17

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

(Blocks: 1 bug)

unspecified
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Bug 744366 didn't fully fix readyState. Need to make it so that readyState always progresses through all states and never transitions to the state that's already the current state.
(Assignee)

Updated

5 years ago
Depends on: 755781
(Assignee)

Updated

5 years ago
No longer depends on: 755781
(Assignee)

Comment 1

5 years ago
Created attachment 644910 [details] [diff] [review]
Make readyState less broken

Waiting for Windows try results still.
(Assignee)

Comment 2

5 years ago
Comment on attachment 644910 [details] [diff] [review]
Make readyState less broken

This deals with neglected parts of MediaDocuments and XSLT readyState and fixes a couple of related bugs: aborting a doc that is being transformed by XSLT and making MediaDocuments come out of bfcache more properly.
Attachment #644910 - Flags: review?(bzbarsky)
(Assignee)

Comment 3

5 years ago
I should mention that the reason why this doesn't come with a test case is that except for the fatal assertions themselves, all the changes in the patch are changes required to make sure the fatal assertions don't fire on our existing test suite.
Comment on attachment 644910 [details] [diff] [review]
Make readyState less broken

r=me
Attachment #644910 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 5

5 years ago
Thanks.

http://hg.mozilla.org/integration/mozilla-inbound/rev/fa6a84e7ba53
https://hg.mozilla.org/mozilla-central/rev/fa6a84e7ba53
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
(Assignee)

Comment 7

5 years ago
Follow-up filed: bug 779100.

Updated

5 years ago
Depends on: 779430
Depends on: 787726

Updated

2 years ago
Depends on: 1221856
You need to log in before you can comment on or make changes to this bug.