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

chrome/browser/resources/ntp_search/recently_closed_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 121 matching lines @@
      * @param {Object} config TilePage configuration object.
      */
     initialize: function(config) {
       ThumbnailPage.prototype.initialize.apply(this, arguments);
       this.classList.add('recently-closed-page');
     },
 
     /**
      * Sets the data that will be used to create Thumbnails.
      * @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);
 
-      this.data_ = data.slice(0, maxTileCount);
-      var dataLength = data.length;
-      var tileCount = this.tileCount;
-      // Create new tiles if necessary.
-      // TODO(jeremycho): Verify this handles the removal of tiles, once that
-      // functionality is supported.
-      if (tileCount < dataLength)
-        this.createTiles_(dataLength - tileCount);
-
-      this.updateTiles_();
       // Only show the dot if we have recently closed tabs to display.
       var dot = this.navigationDot;
-      if (dot)
-        dot.hidden = dataLength == 0;
+      if (this.data_.length == 0) {
+        dot.hidden = true;
+        // If the last tab has been removed (by a user click) and we're still on
+        // the Recently Closed page, fall back to the first page in the dot
+        // list.
+        var cardSlider = ntp.getCardSlider();
+        if (cardSlider.currentCardValue == this)
+          cardSlider.selectCard(0, true);
+      } else {
+        dot.hidden = false;
+      }
 
       logEvent('recentlyClosed.layout: ' + (Date.now() - startTime));
     }
   };
 
   /**
    * Returns the text used for a recently closed window.
    * @param {number} numTabs Number of tabs in the window.
    * @return {string} The text to use.
    */
   function formatTabsText(numTabs) {
     if (numTabs == 1)
       return loadTimeData.getString('closedwindowsingle');
     return loadTimeData.getStringF('closedwindowmultiple', numTabs);
   }
 
   return {
     RecentlyClosedPage: RecentlyClosedPage,
   };
 });