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

Unified Diff: ios/web/webui/resources/web_ui_favicons.js

Issue 2583963002: Remove WebUI favicon support on iOS (Closed)
Patch Set: Remove web_ui_favicons.js from BUILD.gn Created 4 years 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/webui/resources/web_ui_bundle.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/webui/resources/web_ui_favicons.js
diff --git a/ios/web/webui/resources/web_ui_favicons.js b/ios/web/webui/resources/web_ui_favicons.js
deleted file mode 100644
index ff18cc24e95baf533f98bf2c906cc14e1775ab5e..0000000000000000000000000000000000000000
--- a/ios/web/webui/resources/web_ui_favicons.js
+++ /dev/null
@@ -1,56 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-// This file adheres to closure-compiler conventions in order to enable
-// compilation with ADVANCED_OPTIMIZATIONS. In particular, members that are to
-// be accessed externally should be specified in this['style'] as opposed to
-// this.style because member identifiers are minified by default.
-// See http://goo.gl/FwOgy
-
-goog.require('__crWeb.webUIBase');
-
-goog.provide('__crWeb.webUIFavicons');
-
-/**
- * Sends message requesting favicon at the URL from imageSet for el. Sets
- * favicon-url attribute on el to the favicon URL.
- * @param {Element} el The DOM element to request the favicon for.
- * @param {string} imageSet The CSS -webkit-image-set.
- */
-window['chrome']['requestFavicon'] = function(el, imageSet) {
- var cssUrls = imageSet.match(/url\([^\)]+\) \dx/g);
- // TODO(jyquinn): Review match above (crbug.com/528080).
- if (!cssUrls) {
- return;
- }
- // Extract url from CSS -webkit-image-set.
- var faviconUrl = '';
- for (var i = 0; i < cssUrls.length; ++i) {
- var scaleFactorExp = /(\d)x$/;
- var scaleFactor = window.devicePixelRatio;
- if (parseInt(scaleFactorExp.exec(cssUrls[i])[1], 10) === scaleFactor) {
- var urlExp = /url\(\"(.+)\"\)/;
- faviconUrl = urlExp.exec(cssUrls[i])[1];
- break;
- }
- }
- el.setAttribute('favicon-url', url(faviconUrl));
- chrome.send('webui.requestFavicon', [faviconUrl]);
-};
-
-/**
- * Called to set elements with favicon-url attribute equal to faviconUrl to
- * provided dataUrl.
- * @param {string} faviconUrl Favicon URL used to locate favicon element
- * via the favicon-url attribute.
- * @param {string} dataUrl The data URL to assign to the favicon element's
- * backgroundImage.
- */
-window['chrome']['setFaviconBackground'] = function(faviconUrl, dataUrl) {
- var selector = "[favicon-url='" + url(faviconUrl) + "']";
- var elements = document.querySelectorAll(selector);
- for (var i = 0; i < elements.length; ++i) {
- elements[i].style.backgroundImage = url(dataUrl);
- }
-};
« no previous file with comments | « ios/web/webui/resources/web_ui_bundle.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698