534546 - [HTML5] script with defer attribute never runs

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P1)

Description

[Jesse Ruderman]

The single-key shortcuts on http://www.squarefree.com/start/ don't work with html5.enable set to true.  Apparently the <script> doesn't run unless I remove its defer attribute.

I think this bug was introduced within the last week.

Comment 1

[Henri Sivonen (:hsivonen)]

I thought the HTML5 parser was passing our test cases for defer.

Comment 2

[Henri Sivonen (:hsivonen)]

WFM with my local build. As far as I can tell, the only difference in my local build is that I have the patch for bug 531106 applied.

Comment 3

[Henri Sivonen (:hsivonen)]

I see the problem with http://hsivonen.iki.fi/test/moz/defer.html

Comment 4

[Henri Sivonen (:hsivonen)]

OK. It seems hg pull --rebase or something of that nature has caused part of the implementation of defer scripts to get dropped, so the patch didn't land as intended.

Comment 5

[Henri Sivonen (:hsivonen)]

http://hg.mozilla.org/mozilla-central/rev/a732c6d3c078

Thanks for catching this.