Bug 17532 – HTML5 spec violation: Webkit imports styles from <link> element which doesn't have rel=stylesheet defined

• Status: VERIFIED
• Resolution: FIXED
• Component: DOM

[Robert Blaut]

HTML5 spec says: "The type of link indicated (the relationship) is given by the value of the rel attribute, which must be present, and must have a value that is a set of space-separated tokens. [...] If the rel attribute is absent, or if the value used is not allowed according to the definitions in this specification, then the element does not define a link." [http://www.whatwg.org/specs/web-apps/current-work/#link]

Webkit doesn't honor this regulation and imports styles from such defined links:

<link type="text/css" href="style.css" /> This violation causes broken rendering of a one of the top Polish sites fimweb.pl.

Test case attached.

[Robert Blaut]

Created attachment 19343 [details]
minimal test case

[Robert Blaut]

Gecko and Presto engines honor above mentioned regulation.

[Robert Blaut]

<rdar://problem/5762891>

[Robert Blaut]

Created attachment 19347 [details]
test case for all of combination rel="stylesheet" vs. text/css

[Robert Blaut]

Created attachment 19359 [details]
proposed patch

This patch matches Webkit behavior to Firefox and Opera.

These two existing tests should be updated to match the new re=stylesheet requirements:

fast/media/mq-simple-query-02.html
fast/media/mq-width-absolute-02.html

[Dave Hyatt]

Comment on attachment 19359 [details]
proposed patch

I need to see how you changed the existing tests.  Just want to make sure you are patching them to still load the same set of stylesheets they did before.

[Robert Blaut]

(In reply to comment #6)
> (From update of attachment 19359 [details] [edit])
> I need to see how you changed the existing tests.  Just want to make sure you
> are patching them to still load the same set of stylesheets they did before.
> 

Dave, I only informed that these two tests should be updated. The proposed patch only  contains the new layout test. These two tests updates are omitted in the patch.

[Robert Blaut]

Created attachment 19362 [details]
updated patch

Updated patch to reflect David's expectations.

[Darin Adler]

Comment on attachment 19362 [details]
updated patch

+</html>
\ No newline at end of file

New tests should have newlines at the end of the file.

r=me

[Mark Rowe (bdash)]

Landed in r30673.