Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(942)

Unified Diff: ios/web/web_state/js/resources/common.js

Issue 2880723002: Favicon url is used only if it is valid (Closed)
Patch Set: Change the comment Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/web/web_state/favicon_callbacks_inttest.mm ('k') | ios/web/web_state/ui/crw_web_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/web_state/js/resources/common.js
diff --git a/ios/web/web_state/js/resources/common.js b/ios/web/web_state/js/resources/common.js
index 81d0df149cdc2ed5a7975d4af07266ea093a312d..c6a096999cca301d70fa4b25424fc1fcbd13051f 100644
--- a/ios/web/web_state/js/resources/common.js
+++ b/ios/web/web_state/js/resources/common.js
@@ -634,19 +634,6 @@ __gCrWeb['common'] = __gCrWeb.common;
}
}
}
- if (!hasFavicon) {
- // If an HTTP(S)? webpage does not reference a "favicon" then search
- // for a file named "favicon.ico" at the root of the website (legacy).
- // http://en.wikipedia.org/wiki/Favicon
- var location = document.location;
- if (location.protocol == 'http:' || location.protocol == 'https:') {
- var favicon = {
- rel: 'icon',
- href: location.origin + '/favicon.ico'
- };
- favicons.push(favicon);
- }
- }
return favicons;
};
« no previous file with comments | « ios/web/web_state/favicon_callbacks_inttest.mm ('k') | ios/web/web_state/ui/crw_web_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698