New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 19237 link

Starred by 48 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature

Blocked on:
issue 58456


Show other hotlists

Hotlists containing this issue:
Top-Starred-Bugs


Sign in to add a comment

Stylesheets referenced by the Link header are not included

Reported by jeremy.v...@gmail.com, Aug 13 2009

Issue description

Chrome Version       : 3.0.201.0 (23024)
URLs (if applicable) : http://tools.sunriseroad.net/jeremy/tests/link-
stylesheet

Other browsers tested:
  Firefox 3.x: OK
         IE 7: FAIL
         IE 8: FAIL

What steps will reproduce the problem?

1. In the HTTP headers for a HTML page, specify a Link header referencing a 
stylesheet (Link: <style.css>; rel=stylesheet)

What is the expected result?

I expect the stylesheet referenced in the Link header to be included, just 
like one referenced by a <link> tag.

What happens instead?

The stylesheet is not included. Only stylesheets referenced by the <link> tag 
or <style> tag are included.

Please provide any additional information below. Attach a screenshot if
possible.

Test case is here: http://tools.sunriseroad.net/jeremy/tests/link-stylesheet

Two stylesheets are included; one via the <link> tag, and one via the Link 
header.
 
Labels: -Area-Misc Area-WebKit
Status: Untriaged
i liked the testcase
but you should have added a note to the people seeing the Nigerian flag ;)

Opera supports it too btw, not just gecko browsers

i am tagging this as confirmed so a PM can decide if this is something that should be 
supported
Cheers. I can also confirm that the test passes in Opera 9.64.

Didn't realise it looks like the Nigerian flag when it passes. Updated the test. ;)

Comment 3 by fqq...@gmail.com, Sep 13 2009

http://annevankesteren.nl/test/html-element/style-header.php
this is related :)
actual.bmp
942 KB Download
expected.bmp
1.2 MB Download

Comment 4 by karen@chromium.org, Oct 21 2009

Labels: Mstone-X
Status: Upstream
https://bugs.webkit.org/show_bug.cgi?id=20018
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-WebKit Cr-Content
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content Cr-Blink
Hope this gets considered for Blink as its a useful feature, + need more adoption to get it into other browsers too.

Comment 8 by math...@qiwi.be, Apr 8 2013

Re: comment #7, it will have to be implemented in Blink, else it would break backwards compatibility for Opera users. Opera has supported the Link HTTP header since 2005.

Comment 9 by tonyg@chromium.org, Jul 12 2014

Cc: zh...@chromium.org

Comment 10 by tonyg@chromium.org, Jul 12 2014

Blockedon: chromium:58456

Comment 11 by math...@qiwi.be, Feb 2 2015

Status: Available

Comment 12 by y...@yoav.ws, Feb 11 2015

Well, now we have a Link header parser in place, but I believe that adding stylesheet support is still a fundamentally different effort than adding dns-prefetch/preconnect/preload/etc.

I think we'd have to add some sort of a "fake" HTMLLinkElement, which doesn't trigger any of the events, and is invisible when observing the DOM, but still gets applied. Would that be the best way to tackle that, or are there other options?

It's also not clear to me what happens to the cascade with multiple Link headers stylesheets.
HTTP header order is far from guarantied to remain intact. Intermediaries can (and some most probably will) change it.
"HTTP header order is far from guarantied to remain intact. Intermediaries can (and some most probably will) change it."

Preservation of ordering of header fields with the same name *is* required by the spec. Do you have evidence of intermediaries getting this wrong?

Comment 14 by zh...@chromium.org, Feb 11 2015

Cc: -zh...@chromium.org
Labels: Cr-Blink-CSS Cr-Internals-Network-HTTP
Labels: Hotlist-Interop
Tested https://annevankesteren.nl/test/html-element/style-header.php on Windows 7:

Chrome 43.0.2357.130: FAIL
Firefox 39: PASS
IE 11: FAIL
Labels: -Cr-Blink
Labels: -Cr-Internals-Network-HTTP
Removing HTTP label (This isn't handled by the network stack, but rather by blink, despite being an HTTP header)
Labels: -Type-Bug Type-Feature

Comment 20 by math...@qiwi.be, Jan 26 2016

Cc: y...@yoav.ws
Labels: -Pri-2 Pri-3

Comment 22 by ise...@gmail.com, Sep 7 2016

It's nice for providing styling for raw text/plain documents.
Labels: Update-Quarterly

Comment 24 by math...@qiwi.be, Feb 15 2017

 Issue 692359  has been merged into this issue.
It's available. Does that mean someone could submit a patch?

Comment 26 by phistuck@gmail.com, Mar 11 2017

#25 - yep.
Labels: ApproachableBug Code-Parser
Labels: -Update-Quarterly
Project Member

Comment 30 by sheriffbot@chromium.org, Dec 7

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
This is still important. Link headers have the same meaning as link elements.
Components: Blink>Loader
Status: Available (was: Untriaged)
Cc: domfarolino@gmail.com

Sign in to add a comment