Make favicons for about:history and about:bookmarks switch from lodpi to hidpi when dragging browser

chrome/browser/resources/shared/js/util.js

 // Copyright (c) 2012 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.
 
 /**
  * The global object.
  * @type {!Object}
  * @const
  */
 var global = this;
@@ skipping 20 matching lines @@
     // restore
     global[callbackName] = old;
 
     var args = Array.prototype.slice.call(arguments);
     return callback.apply(global, args);
   };
   chrome.send(name, params);
 }
 
 /**
+ * Returns the scale factors supported by this platform.
+ * @return {array} The supported scale factors.
+ */
+function getSupportedScaleFactors() {
+  var supported_scale_factors = new Array();

[kevers, 2013/01/04 20:21:10]

Using C++ naming convention here and below.  Should use camel-case naming in
Javascript.  Underscores appear in optional parameters that start with the
"opt_" prefix and at the end of private data members.

Also prefer "[]" to "new Array()", but this just a style preference.

[pkotwicz, 2013/01/04 21:21:39]

Done.

[Dan Beam, 2013/01/04 21:34:28]

This is not just style, [] always invokes the builtin Array constructor whereas
if somebody redefines window.Array new Array() invokes that code.  This was used
in a famous Gmail contacts hack. 
http://jeremiahgrossman.blogspot.com/2006/01/advanced-web-attack-techniques-u...

+  if (cr.isMac || cr.isChromeOS) {
+    supported_scale_factors.push(1);
+    supported_scale_factors.push(2);
+  } else {
+    // Windows must be restarted to display at a different scale factor.
+    supported_scale_factors.push(window.devicePixelRatio);
+  }
+  return supported_scale_factors;
+}
+
+/**
  * Generates a CSS url string.
  * @param {string} s The URL to generate the CSS url for.
  * @return {string} The CSS url string.
  */
 function url(s) {
   // http://www.w3.org/TR/css3-values/#uris
   // Parentheses, commas, whitespace characters, single quotes (') and double
   // quotes (") appearing in a URI must be escaped with a backslash
   var s2 = s.replace(/(\(|\)|\,|\s|\'|\"|\\)/g, '\\$1');
   // WebKit has a bug when it comes to URLs that end with \
   // https://bugs.webkit.org/show_bug.cgi?id=28885
   if (/\\\\$/.test(s2)) {
     // Add a space to work around the WebKit bug.
     s2 += ' ';
   }
   return 'url("' + s2 + '")';
 }
 
 /**
+ * Generates a CSS -webkit-image-set for a chrome:// url.
+ * An entry in the image set is added for each of getSupportedScaleFactors().
+ * The scale-factor-specific url is generated by replacing the first instance of
+ * 'scalefactor' in |path| with the numeric scale factor.
+ * @param {string} path The URL to generate an image set for.
+ *     'scalefactor' should be a substring of |path|.
+ * @return {string} The CSS -webkit-image-set.
+ */
+function imageset(path) {
+  var supported_scale_factors = getSupportedScaleFactors();
+
+  var replace_start_index = path.indexOf('scalefactor');
+  if (replace_start_index < 0)
+    return;
+
+  var s = '';
+  for (var i = 0; i < supported_scale_factors.length; ++i) {
+    var scale_factor = supported_scale_factors[i];
+    var path_with_scale_factor =
+        path.substr(0, replace_start_index) + scale_factor +
+        path.substr(replace_start_index + 11);
+
+   s += url(path_with_scale_factor) + ' ' + scale_factor + 'x';
+
+   if (i != supported_scale_factors.length - 1)
+     s += ', ';
+  }
+  return '-webkit-image-set(' + s + ')';
+}
+
+/**
  * Parses query parameters from Location.
  * @param {string} location The URL to generate the CSS url for.
  * @return {object} Dictionary containing name value pairs for URL
  */
 function parseQueryParams(location) {
   var params = {};
   var query = unescape(location.search.substring(1));
   var vars = query.split('&');
   for (var i = 0; i < vars.length; i++) {
     var pair = vars[i].split('=');
@@ skipping 151 matching lines @@
  */
 function appendParam(url, key, value) {
   var param = encodeURIComponent(key) + '=' + encodeURIComponent(value);
 
   if (url.indexOf('?') == -1)
     return url + '?' + param;
   return url + '&' + param;
 }
 
 /**
- * Creates a new URL for a favicon request.
+ * Creates a CSS -webkit-image-set for a favicon request.
  * @param {string} url The url for the favicon.
  * @param {number=} opt_size Optional preferred size of the favicon.
- * @return {string} Updated URL for the favicon.
+ * @param {boolean=} opt_sessionFavicon Optional flag to indicate if
+ *     requesting a session favicon.
+ * @return {string} -webkit-image-set for the favicon.
  */
-function getFaviconUrl(url, opt_size) {
+function getFaviconImageSet(url, opt_size, opt_sessionFavicon) {
   var size = opt_size || 16;
-  return 'chrome://favicon/size/' + size + '@' +
-      window.devicePixelRatio + 'x/' + url;
+  var type = opt_sessionFavicon ? 'session-favicon' : 'favicon';
+  return imageset(
+      'chrome://' + type + '/size/' + size + '@scalefactorx/' + url);
 }
 
 /**
+ * Creates a new URL for a request of the 1x favicon.
+ * @param {string} url The url for the favicon.
+ * @param {number=} opt_size Optional preferred size of the favicon.
+ * @param {boolean=} opt_sessionFavicon Optional flag to indicate if
+ *     requesting a session favicon.
+ * @return {string} Updated URL for the favicon.
+ */
+function get1xFaviconUrl(url, opt_size, opt_sessionFavicon) {
+  var size = opt_size || 16;
+  var type = opt_sessionFavicon ? 'session-favicon' : 'favicon';
+  return 'chrome://' + type + '/size/' + size + '@1x/' + url;
+}
+
+/**
  * Creates an element of a specified type with a specified class name.
  * @param {string} type The node type.
  * @param {string} className The class name to use.
  * @return {Element} The created element.
  */
 function createElementWithClassName(type, className) {
   var elm = document.createElement(type);
   elm.className = className;
   return elm;
 }