| Index: chrome/browser/resources/ntp4/apps_page.js
|
| diff --git a/chrome/browser/resources/ntp4/apps_page.js b/chrome/browser/resources/ntp4/apps_page.js
|
| index 9e7b71e6d4044d1005beb4f00e9e66d60da39b23..fd983250373cc575a49aa7e6cf95bfe5ee517e88 100644
|
| --- a/chrome/browser/resources/ntp4/apps_page.js
|
| +++ b/chrome/browser/resources/ntp4/apps_page.js
|
| @@ -245,12 +245,7 @@ cr.define('ntp', function() {
|
|
|
| this.className = 'app focusable';
|
|
|
| - if (!this.appData_.icon_big_exists && this.appData_.icon_small_exists)
|
| - this.useSmallIcon_ = true;
|
| -
|
| - this.appContents_ = this.useSmallIcon_ ?
|
| - $('app-small-icon-template').cloneNode(true) :
|
| - $('app-large-icon-template').cloneNode(true);
|
| + this.appContents_ = $('app-icon-template').cloneNode(true);
|
| this.appContents_.id = '';
|
| this.appendChild(this.appContents_);
|
|
|
| @@ -259,16 +254,8 @@ cr.define('ntp', function() {
|
| this.appImg_ = this.appImgContainer_.querySelector('img');
|
| this.setIcon();
|
|
|
| - if (this.useSmallIcon_) {
|
| - this.imgDiv_ = /** @type {HTMLElement} */(
|
| - this.querySelector('.app-icon-div'));
|
| - this.addLaunchClickTarget_(this.imgDiv_);
|
| - this.imgDiv_.title = this.appData_.full_name;
|
| - chrome.send('getAppIconDominantColor', [this.id]);
|
| - } else {
|
| - this.addLaunchClickTarget_(this.appImgContainer_);
|
| - this.appImgContainer_.title = this.appData_.full_name;
|
| - }
|
| + this.addLaunchClickTarget_(this.appImgContainer_);
|
| + this.appImgContainer_.title = this.appData_.full_name;
|
|
|
| // The app's full name is shown in the tooltip, whereas the short name
|
| // is used for the label.
|
| @@ -299,14 +286,6 @@ cr.define('ntp', function() {
|
| },
|
|
|
| /**
|
| - * Sets the color of the favicon dominant color bar.
|
| - * @param {string} color The css-parsable value for the color.
|
| - */
|
| - set stripeColor(color) {
|
| - this.querySelector('.color-stripe').style.backgroundColor = color;
|
| - },
|
| -
|
| - /**
|
| * Removes the app tile from the page. Should be called after the app has
|
| * been uninstalled.
|
| *
|
| @@ -328,8 +307,7 @@ cr.define('ntp', function() {
|
| * to load icons until we have to).
|
| */
|
| setIcon: function() {
|
| - var src = this.useSmallIcon_ ? this.appData_.icon_small :
|
| - this.appData_.icon_big;
|
| + var src = this.appData_.icon;
|
| if (!this.appData_.enabled ||
|
| (!this.appData_.offlineEnabled && !navigator.onLine)) {
|
| src += '?grayscale=true';
|
| @@ -363,20 +341,7 @@ cr.define('ntp', function() {
|
| setBounds: function(size, x, y) {
|
| var imgSize = size * APP_IMG_SIZE_FRACTION;
|
| this.appImgContainer_.style.width = this.appImgContainer_.style.height =
|
| - toCssPx(this.useSmallIcon_ ? 16 : imgSize);
|
| - if (this.useSmallIcon_) {
|
| - // 3/4 is the ratio of 96px to 128px (the used height and full height
|
| - // of icons in apps).
|
| - var iconSize = imgSize * 3 / 4;
|
| - // The -2 is for the div border to improve the visual alignment for the
|
| - // icon div.
|
| - this.imgDiv_.style.width = this.imgDiv_.style.height =
|
| - toCssPx(iconSize - 2);
|
| - // Margins set to get the icon placement right and the text to line up.
|
| - this.imgDiv_.style.marginTop = this.imgDiv_.style.marginBottom =
|
| - toCssPx((imgSize - iconSize) / 2);
|
| - }
|
| -
|
| + toCssPx(imgSize);
|
| this.style.width = this.style.height = toCssPx(size);
|
| this.style.left = toCssPx(x);
|
| this.style.right = toCssPx(x);
|
| @@ -394,7 +359,8 @@ cr.define('ntp', function() {
|
| * @private
|
| */
|
| onClick_: function(e) {
|
| - if (/** @type {MouseEvent} */(e).button > 1) return;
|
| + if (/** @type {MouseEvent} */(e).button > 1)
|
| + return;
|
|
|
| chrome.send('launchApp',
|
| [this.appId, APP_LAUNCH.NTP_APPS_MAXIMIZED, 'chrome-ntp-icon',
|
| @@ -473,16 +439,10 @@ cr.define('ntp', function() {
|
| * The data and preferences for this app.
|
| * @type {Object}
|
| */
|
| - set appData(data) {
|
| - this.appData_ = data;
|
| - },
|
| - get appData() {
|
| - return this.appData_;
|
| - },
|
| + set appData(data) { this.appData_ = data; },
|
| + get appData() { return this.appData_; },
|
|
|
| - get appId() {
|
| - return this.appData_.id;
|
| - },
|
| + get appId() { return this.appData_.id; },
|
|
|
| /**
|
| * Returns a pointer to the context menu for this app. All apps share the
|
| @@ -501,9 +461,7 @@ cr.define('ntp', function() {
|
| * the user can drag it onto the trash and expect something to happen).
|
| * @return {boolean} True if the app can be uninstalled.
|
| */
|
| - canBeRemoved: function() {
|
| - return this.appData_.mayDisable;
|
| - },
|
| + canBeRemoved: function() { return this.appData_.mayDisable; },
|
|
|
| /**
|
| * Uninstalls the app after it's been dropped on the trash.
|
|
|