| Index: chrome/browser/resources/ntp_android/ntp_android.js
|
| diff --git a/chrome/browser/resources/ntp_android/ntp_android.js b/chrome/browser/resources/ntp_android/ntp_android.js
|
| index 4e61603ccfa5c751f5d1af18c30905f7049d28b6..517deadce3789e94f2b182e423fb40924626917e 100644
|
| --- a/chrome/browser/resources/ntp_android/ntp_android.js
|
| +++ b/chrome/browser/resources/ntp_android/ntp_android.js
|
| @@ -707,9 +707,7 @@ cr.define('ntp', function() {
|
| image.onload = function() {
|
| var w = image.width;
|
| var h = image.height;
|
| - var wDip = w / window.devicePixelRatio;
|
| - var hDip = h / window.devicePixelRatio;
|
| - if (Math.floor(wDip) <= 16 || Math.floor(hDip) <= 16) {
|
| + if (Math.floor(w) <= 16 || Math.floor(h) <= 16) {
|
| // it's a standard favicon (or at least it's small).
|
| faviconBox.classList.add('document');
|
|
|
| @@ -740,13 +738,13 @@ cr.define('ntp', function() {
|
| } else {
|
| // It's an html5 icon (or at least it's larger).
|
| // Rescale it to be no bigger than 64x64 dip.
|
| - var maxDip = 64; // DIP
|
| - if (wDip > maxDip || hDip > maxDip) {
|
| - var scale = (wDip > hDip) ? (maxDip / wDip) : (maxDip / hDip);
|
| - wDip *= scale;
|
| - hDip *= scale;
|
| + var max = 64;
|
| + if (w > max || h > max) {
|
| + var scale = (w > h) ? (max / w) : (max / h);
|
| + w *= scale;
|
| + h *= scale;
|
| }
|
| - faviconIcon.style.backgroundSize = wDip + 'px ' + hDip + 'px';
|
| + faviconIcon.style.backgroundSize = w + 'px ' + h + 'px';
|
| }
|
| };
|
| faviconBox.appendChild(faviconIcon);
|
| @@ -2214,9 +2212,6 @@ cr.define('ntp', function() {
|
| var itemMarginRight = 20;
|
|
|
| var listHeight = 0;
|
| - // set it to the unscaled size so centerGrid works correctly
|
| - modifyCssRule('body[device="phone"] .thumbnail-cell',
|
| - 'width', thumbWidth + 'px');
|
|
|
| var screenHeight =
|
| document.documentElement.offsetHeight -
|
| @@ -2224,8 +2219,8 @@ cr.define('ntp', function() {
|
|
|
| if (isPortrait()) {
|
| mostVisitedList.setAttribute(GRID_COLUMNS, '2');
|
| - listHeight = screenHeight * .85;
|
| - listHeight = listHeight >= 420 ? 420 : listHeight;
|
| + listHeight = screenHeight;
|
| + listHeight = listHeight >= 400 ? 400 : listHeight;
|
| // Size for 3 rows (4 gutters)
|
| itemMarginTop = (listHeight - (itemHeight * 3)) / 4;
|
| } else {
|
| @@ -2241,8 +2236,8 @@ cr.define('ntp', function() {
|
| var scale = (screenHeight - 2 * labelHeight -
|
| targetRemainder) / (2 * thumbHeight);
|
| // update values based on scale
|
| - thumbWidth *= scale;
|
| - thumbHeight *= scale;
|
| + thumbWidth = Math.round(thumbWidth * scale);
|
| + thumbHeight = Math.round(thumbHeight * scale);
|
| labelWidth = thumbWidth + 20;
|
| itemHeight = thumbHeight + labelHeight;
|
| }
|
|
|