NTP5: Fix page blacklisting and remove recently closed tabs when they're clicked. Fix the styling …

chrome/browser/resources/ntp_search/most_visited_page.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.
 
 cr.define('ntp', function() {
   'use strict';
 
   var Thumbnail = ntp.Thumbnail;
   var ThumbnailPage = ntp.ThumbnailPage;
 
@@ skipping 192 matching lines @@
     handleCardSelected_: function(e) {
       if (!document.documentElement.classList.contains('starting-up'))
         chrome.send('mostVisitedSelected');
     },
 
     /**
      * Sets the data that will be used to create Thumbnails.
      * TODO(pedrosimonetti): Move data handling related code to TilePage. Make
      * sure the new logic works with Apps without requiring duplicating code.
      * @param {Array} data The array of data.
-     * @type (Array}
      */
-    set data(data) {
+    setData: function(data) {
       var startTime = Date.now();
-      var maxTileCount = this.config_.maxTileCount;
+      ThumbnailPage.prototype.setData.apply(this, arguments);
+      logEvent('mostVisited.layout: ' + (Date.now() - startTime));
+    },
 
-      // The first time data is set, create the tiles.
-      if (!this.data_) {
-        this.data_ = data.slice(0, maxTileCount);
-        this.createTiles_(this.data_.length);
-      } else {
-        this.data_ = refreshData(this.data_, data, maxTileCount);
+    /**
+     * Merges new Most Visited data into the old while minimizing re-ordering.
+     * @param {Array} oldData The current page list.
+     * @param {Array} newData The new page list.
+     * @param {number} maxTileCount The maximum number of tiles to render.
+     * @return {Array} The merged page list that should replace the current page
+     *     list.
+     * @private
+     */
+    refreshData_: function(oldData, newData, maxTileCount) {
+      oldData = oldData.slice(0, maxTileCount);
+      newData = newData.slice(0, maxTileCount);
+
+      // Look through old pages; if they exist in the newData list, keep them
+      // where they are.
+      for (var i = 0; i < oldData.length; i++) {
+        if (!oldData[i] || oldData[i].updated)
+          continue;
+
+        for (var j = 0; j < newData.length; j++) {
+          if (newData[j].used)
+            continue;
+
+          if (newData[j].url == oldData[i].url) {
+            // The background image and other data may have changed.
+            oldData[i] = newData[j];
+            oldData[i].updated = true;
+            newData[j].used = true;
+            break;
+          }
+        }
       }
 
-      this.updateTiles_();
-      logEvent('mostVisited.layout: ' + (Date.now() - startTime));
+      // Look through old pages that haven't been updated yet; replace them.
+      for (var i = 0; i < oldData.length; i++) {
+        if (oldData[i] && oldData[i].updated)
+          continue;
+
+        for (var j = 0; j < newData.length; j++) {
+          if (newData[j].used)
+            continue;
+
+          oldData[i] = newData[j];
+          oldData[i].updated = true;
+          newData[j].used = true;
+          break;
+        }
+
+        if (oldData[i] && !oldData[i].updated)
+          oldData[i] = null;
+      }
+
+      // Clear 'updated' flags so this function will work next time it's called.
+      for (var i = 0; i < maxTileCount; i++) {
+        if (oldData[i])
+          oldData[i].updated = false;
+      }
+
+      return oldData;
+    },
+
+    /**
+     * Returns all non-null data entries.  Null is used by Most Visited Page as
+     * placeholders for blacklisted entries.
+     * @private
+     * @return {Array} Non-null data entries.
+     */
+    getValidData_: function() {
+      var validData = [];
+      var data = this.data_;
+      for (var i = 0, length = data.length; i < length; i++) {
+        if (data[i])
+          validData.push(data[i]);
+      }
+      return validData;
     },
   };
 
   /**
    * Executed once the NTP has loaded. Checks if the Most Visited pane is
    * shown or not. If it is shown, the 'mostVisitedSelected' message is sent
    * to the C++ code, to record the fact that the user has seen this pane.
    */
   MostVisitedPage.onLoaded = function() {
     if (ntp.getCardSlider() &&
         ntp.getCardSlider().currentCardValue &&
         ntp.getCardSlider().currentCardValue.classList
         .contains('most-visited-page')) {
       chrome.send('mostVisitedSelected');
     }
   };
 
-  /**
-   * We've gotten additional Most Visited data. Update our old data with the
-   * new data. The ordering of the new data is not important, except when a
-   * page is pinned. Thus we try to minimize re-ordering.
-   * @param {Array} oldData The current Most Visited page list.
-   * @param {Array} newData The new Most Visited page list.
-   * @return {Array} The merged page list that should replace the current page
-   *     list.
-   */
-  function refreshData(oldData, newData, maxTileCount) {
-    oldData = oldData.slice(0, maxTileCount);
-    newData = newData.slice(0, maxTileCount);
-
-    // Copy over pinned sites directly.
-    for (var j = 0; j < newData.length; j++) {
-      if (newData[j].pinned) {
-        oldData[j] = newData[j];
-        // Mark the entry as 'updated' so we don't try to update again.
-        oldData[j].updated = true;
-        // Mark the newData page as 'used' so we don't try to re-use it.
-        newData[j].used = true;
-      }
-    }
-
-    // Look through old pages; if they exist in the newData list, keep them
-    // where they are.
-    for (var i = 0; i < oldData.length; i++) {
-      if (!oldData[i] || oldData[i].updated)
-        continue;
-
-      for (var j = 0; j < newData.length; j++) {
-        if (newData[j].used)
-          continue;
-
-        if (newData[j].url == oldData[i].url) {
-          // The background image and other data may have changed.
-          oldData[i] = newData[j];
-          oldData[i].updated = true;
-          newData[j].used = true;
-          break;
-        }
-      }
-    }
-
-    // Look through old pages that haven't been updated yet; replace them.
-    for (var i = 0; i < oldData.length; i++) {
-      if (oldData[i] && oldData[i].updated)
-        continue;
-
-      for (var j = 0; j < newData.length; j++) {
-        if (newData[j].used)
-          continue;
-
-        oldData[i] = newData[j];
-        oldData[i].updated = true;
-        newData[j].used = true;
-        break;
-      }
-
-      if (oldData[i] && !oldData[i].updated)
-        oldData[i] = null;
-    }
-
-    // Clear 'updated' flags so this function will work next time it's called.
-    for (var i = 0; i < maxTileCount; i++) {
-      if (oldData[i])
-        oldData[i].updated = false;
-    }
-
-    return oldData;
-  }
-
   return {
     MostVisitedPage: MostVisitedPage,
-    refreshData: refreshData,
   };
 });
 
 document.addEventListener('ntpLoaded', ntp.MostVisitedPage.onLoaded);