[webkit-reviews] review requested: [Bug 170827] Enough with these favicon.ico errors : [Attachment 307086] [PATCH] Proposed Fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 13 20:55:17 PDT 2017


Joseph Pecoraro <joepeck at webkit.org> has asked	for review:
Bug 170827: Enough with these favicon.ico errors
https://bugs.webkit.org/show_bug.cgi?id=170827

Attachment 307086: [PATCH] Proposed Fix

https://bugs.webkit.org/attachment.cgi?id=307086&action=review




--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org> ---
Created attachment 307086

  --> https://bugs.webkit.org/attachment.cgi?id=307086&action=review

[PATCH] Proposed Fix

I still want to get rid of the file:///favicon.ico but my brain continues to
think "maybe this could be legit".

Maybe we should just prevent against this single file URL path, since its so
unlikely to be real?


More information about the webkit-reviews mailing list