775467 - Make readyState progress through all states without duplicate transitions

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Henri Sivonen (:hsivonen)]

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.

Comment 1

[Henri Sivonen (:hsivonen)]

Attachment: Make readyState less broken

Waiting for Windows try results still.

Comment 2

[Henri Sivonen (:hsivonen)]

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.

Comment 3

[Henri Sivonen (:hsivonen)]

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 4

[Boris Zbarsky [:bzbarsky]]

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

r=me

Comment 5

[Henri Sivonen (:hsivonen)]

Thanks.

http://hg.mozilla.org/integration/mozilla-inbound/rev/fa6a84e7ba53

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fa6a84e7ba53

Comment 7

[Henri Sivonen (:hsivonen)]

Follow-up filed: bug 779100.