Chromium Code Reviews| 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 bb8ee68057409d99f9f93c0b509ed65763de0c36..ecb7b8baee387c14ff00f06c908f90d752cac1c8 100644 |
| --- a/chrome/browser/resources/ntp_android/ntp_android.js |
| +++ b/chrome/browser/resources/ntp_android/ntp_android.js |
| @@ -592,8 +592,8 @@ cr.define('ntp', function() { |
| cell.setAttribute(CONTEXT_MENU_URL_KEY, item.url); |
| - var iconUrl = MOCK ? |
| - touchIconURI : 'chrome://touch-icon/size/64/' + item.url; |
| + var iconUrl = |
| + item.icon ? item.icon : 'chrome://touch-icon/size/64/' + item.url; |
|
Evan Stade
2012/09/10 09:16:08
reverseditto
Ted C
2012/09/10 22:35:08
Done.
|
| var icon = createDiv('icon', iconUrl); |
| trackImageLoad(iconUrl); |
| cell.appendChild(icon); |
| @@ -629,8 +629,6 @@ cr.define('ntp', function() { |
| var thumbnailCell = createDiv('thumbnail-cell'); |
| var thumbnailContainer = createDiv('thumbnail-container'); |
| var backgroundUrl = item.thumbnailUrl || 'chrome://thumb/' + item.url; |
| - if (MOCK) |
| - backgroundUrl = thumbnailURI; |
| if (backgroundUrl == 'chrome://thumb/chrome://welcome/') { |
| // Ideally, it would be nice to use the URL as is. However, as of now |
| // theme support has been removed from Chrome. Instead, load the image |
| @@ -688,7 +686,7 @@ cr.define('ntp', function() { |
| if (item.folder) { |
| faviconBox.classList.add('folder'); |
| } else { |
| - var iconUrl = MOCK ? item.icon : 'chrome://touch-icon/' + item.url; |
| + var iconUrl = item.icon ? item.icon : 'chrome://touch-icon/' + item.url; |
|
Evan Stade
2012/09/10 09:16:08
reverse ditto
Ted C
2012/09/10 22:35:08
Done.
|
| var faviconIcon = createDiv('favicon-icon'); |
| faviconIcon.style.backgroundImage = 'url(' + iconUrl + ')'; |
| trackImageLoad(iconUrl); |
| @@ -811,7 +809,8 @@ cr.define('ntp', function() { |
| var listItem = createDiv('list-item'); |
| addActiveTouchListener(listItem, ACTIVE_LIST_ITEM_CSS_CLASS); |
| listItem.setAttribute(CONTEXT_MENU_URL_KEY, item.url); |
| - var iconUrl = MOCK ? item.icon : 'chrome://touch-icon/size/64/' + item.url; |
| + var iconUrl = |
| + item.icon ? item.icon : 'chrome://touch-icon/size/64/' + item.url; |
|
Evan Stade
2012/09/10 09:16:08
in javascript you do this like so:
var iconUrl =
Ted C
2012/09/10 22:35:08
Done.
|
| listItem.appendChild(createDiv('icon', iconUrl)); |
| trackImageLoad(iconUrl); |
| var title = createElement('span', { |