New issue
Advanced search Search tips

Issue 675455 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

favicon.svg request for every hash change

Reported by mmyan...@gmail.com, Dec 18 2016

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36

Example URL:

Steps to reproduce the problem:
1. build a simple HTTP server with index.html and a vaild favicon.svg
2. For index.html: <link rel="icon" type="image/svg" sizes="any" href="favicon.svg">
3. Type `window.location.replace("#a")' in console

What is the expected behavior?
No network traffic

What went wrong?
One favicon.svg request for one window.location.replace('#a')

Did this work before? N/A 

Chrome version: 55.0.2883.87  Channel: stable
OS Version: 
Flash Version: Shockwave Flash 24.0 r0

Won't work on favicon.png
 

Comment 1 by mmenke@chromium.org, Dec 18 2016

Cc: pkotw...@chromium.org sky@chromium.org steve...@chromium.org
Components: -Internals>Network
CCing components/favicon owners (Don't think there's a component for it?)

Comment 2 by ajha@chromium.org, Dec 18 2016

Labels: M-55 prestable-55.0.2883.87
Labels: Needs-Feedback
mmyangfl@ in order to triage this issue could you please help us with the sample html page.

Thank You...

Comment 4 by mmyan...@gmail.com, Dec 24 2016

It seems to me that every `window.location.replace("#a")' will trigger a query of favicon.svg in cache. If you disable the cache, you will see real requests from server side. The curious part is that it won't work if the browser decide to use the favicon.png rather than favicon.svg. I wonder is it the excepted behavior specially for favicon.svg?

It was originally found in https://daihan.cn

For favicon.svg, just simply create a empty file.

$ google-chrome-unstable --version
Google Chrome 57.0.2950.4 dev
index.html
167 bytes View Download
input.png
18.5 KB View Download
output.png
38.7 KB View Download
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 2 2017

Labels: -Needs-Feedback Needs-Review
Owner: kkaluri@chromium.org
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" for another review and adding "Needs-Review" label for tracking.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
I believe this difference in behavior is because Chrome does not support SVG favicons
Components: Blink>DOM
Cc: chrishtr@chromium.org
Components: -Blink>DOM
@chrishtr, this isn't a DOM bug. We duly collect the URLs from the link tags--of course we have no idea what their MIME types are, etc.--and it's a matter for the frame client/loader if it wants to try retrieving SVG icons repeatedly.

If I misunderstood that and there's something specific DOM can do, please let us know!
Components: Blink>Loader
Trying Blink>Loader instead...
Components: -Blink>Loader
Confirmed favicon.svg is requested multiples times and favicon.ico is requtested only once.

Confirmed that WebAssociatedURLLoader is used for favicons (i.e., they are requested from outside of blink). So this is not a Blink>Loader issue.


Cleaning up "Needs-Review" label as we are not using this label for triage. Ref  bug 684919 
Labels: -Needs-Review
Project Member

Comment 13 by sheriffbot@chromium.org, Mar 15 2018

Status: Archived (was: Unconfirmed)
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment