| Index: chrome/browser/resources/ntp4/bookmarks_page.js
|
| ===================================================================
|
| --- chrome/browser/resources/ntp4/bookmarks_page.js (revision 98871)
|
| +++ chrome/browser/resources/ntp4/bookmarks_page.js (working copy)
|
| @@ -59,10 +59,11 @@
|
| var faviconDiv = this.querySelector('.favicon');
|
| var faviconUrl;
|
| if (this.data.url) {
|
| - faviconUrl = 'chrome://favicon/size/16/' + this.data.url;
|
| + faviconUrl = 'chrome://favicon/size/32/' + this.data.url;
|
| chrome.send('getFaviconDominantColor',
|
| [faviconUrl, id, 'ntp4.setBookmarksFaviconDominantColor']);
|
| } else {
|
| + // TODO(csilv): We need a large (32px) icon for this URL.
|
| faviconUrl = 'chrome://theme/IDR_BOOKMARK_BAR_FOLDER';
|
| // TODO(csilv): Should we vary this color by platform?
|
| this.stripeColor = '#919191';
|
| @@ -88,11 +89,8 @@
|
| * animate.
|
| */
|
| setBounds: function(size, x, y) {
|
| - this.style.width = size + 'px';
|
| - this.style.height = heightForWidth(size) + 'px';
|
| -
|
| + this.style.width = this.style.height = size + 'px';
|
| this.style.left = x + 'px';
|
| - this.style.right = x + 'px';
|
| this.style.top = y + 'px';
|
| },
|
|
|
| @@ -175,16 +173,6 @@
|
| TilePage.initGridValues(bookmarksPageGridValues);
|
|
|
| /**
|
| - * Calculates the height for a bookmarks tile for a given width. The size
|
| - * is based on a desired size of 96x72 ratio.
|
| - * @return {number} The height.
|
| - */
|
| - function heightForWidth(width) {
|
| - // The 2s are for borders, the 31 is for the title.
|
| - return (width - 2) * 72 / 96 + 2 + 31;
|
| - }
|
| -
|
| - /**
|
| * Creates a new BookmarksPage object.
|
| * @constructor
|
| * @extends {TilePage}
|
| @@ -287,9 +275,6 @@
|
| return false;
|
| },
|
|
|
| - /** @inheritDoc */
|
| - heightForWidth: heightForWidth,
|
| -
|
| /**
|
| * Invoked before a batch import begins. We will ignore added/changed
|
| * notifications while the operation is in progress.
|
|
|