Refactoring NTP5: new implementation of TilePage and MostVisitedPage (which now inherits from Thumb…

chrome/browser/resources/ntp_search/new_tab.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.
 
 /**
  * @fileoverview New tab page
  * This is the main code for the new tab page used by touch-enabled Chrome
  * browsers.  For now this is still a prototype.
  */
 
@@ skipping 69 matching lines @@
   function NewTabView() {
     var pageSwitcherStart = null;
     var pageSwitcherEnd = null;
     if (loadTimeData.getValue('showApps')) {
       pageSwitcherStart = getRequiredElement('page-switcher-start');
       pageSwitcherEnd = getRequiredElement('page-switcher-end');
     }
     this.initialize(getRequiredElement('page-list'),
                     getRequiredElement('dot-list'),
                     getRequiredElement('card-slider-frame'),
+                    // TODO(xci): remove the Trash component.
                     getRequiredElement('trash'),
+                    // TODO(xci): remove page switchers.
                     pageSwitcherStart, pageSwitcherEnd);
   }
 
   NewTabView.prototype = {
     __proto__: ntp.PageListView.prototype,
 
     /** @inheritDoc */
     appendTilePage: function(page, title, titleIsEditable, opt_refNode) {
       ntp.PageListView.prototype.appendTilePage.apply(this, arguments);
 
@@ skipping 337 matching lines @@
 
   function setMostVisitedPages(data, hasBlacklistedUrls) {
     newTabView.mostVisitedPage.data = data;
     cr.dispatchSimpleEvent(document, 'sectionready', true, true);
   }
 
   function setSuggestionsPages(data, hasBlacklistedUrls) {
     newTabView.suggestionsPage.data = data;
   }
 
-  /**
-   * Set the dominant color for a node. This will be called in response to
-   * getFaviconDominantColor. The node represented by |id| better have a setter
-   * for stripeColor.
-   * @param {string} id The ID of a node.
-   * @param {string} color The color represented as a CSS string.
-   */
-  function setStripeColor(id, color) {
-    var node = $(id);
-    if (node)
-      node.stripeColor = color;
+  function getThumbnailUrl(url) {
+    return 'chrome://thumb/' + url;
   }
 
   /**
    * Updates the text displayed in the login container. If there is no text then
    * the login container is hidden.
    * @param {string} loginHeader The first line of text.
    * @param {string} loginSubHeader The second line of text.
    * @param {string} iconURL The url for the login status icon. If this is null
         then the login status icon is hidden.
    * @param {boolean} isUserSignedIn Indicates if the user is signed in or not.
@@ skipping 97 matching lines @@
   // Return an object with all the exports
   return {
     appAdded: appAdded,
     appMoved: appMoved,
     appRemoved: appRemoved,
     appsPrefChangeCallback: appsPrefChangeCallback,
     enterRearrangeMode: enterRearrangeMode,
     getAppsCallback: getAppsCallback,
     getAppsPageIndex: getAppsPageIndex,
     getCardSlider: getCardSlider,
+    getThumbnailUrl: getThumbnailUrl,
     onLoad: onLoad,
     leaveRearrangeMode: leaveRearrangeMode,
     NtpFollowAction: NtpFollowAction,
     saveAppPageName: saveAppPageName,
     setAppToBeHighlighted: setAppToBeHighlighted,
     setBookmarkBarAttached: setBookmarkBarAttached,
     setForeignSessions: setForeignSessions,
     setMostVisitedPages: setMostVisitedPages,
     setSuggestionsPages: setSuggestionsPages,
     setRecentlyClosedTabs: setRecentlyClosedTabs,
-    setStripeColor: setStripeColor,
     showNotification: showNotification,
     themeChanged: themeChanged,
     updateLogin: updateLogin
   };
 });
 
 document.addEventListener('DOMContentLoaded', ntp.onLoad);
 
 var toCssPx = cr.ui.toCssPx;