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

chrome/browser/resources/ntp_search/tile_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';
 
   //----------------------------------------------------------------------------
   // Constants
   //----------------------------------------------------------------------------
@@ skipping 287 matching lines @@
     /**
      * Whether or not this TilePage is selected.
      * @type {boolean}
      */
     get selected() {
       return Array.prototype.indexOf.call(this.parentNode.children, this) ==
           ntp.getCardSlider().currentCard;
     },
 
     /**
-     * The notification content of this tile (if any, otherwise null).
-     * @type {!HTMLElement}
-     */
-    get notification() {
-      return this.content_.firstChild.id == 'notification-container' ?
-          this.content_.firstChild : null;
-    },
-    /**
-     * The notification content of this tile (if any, otherwise null).
-     * @type {!HTMLElement}
-     */
-    set notification(node) {
-      assert(node instanceof HTMLElement, '|node| isn\'t an HTMLElement!');
-      // NOTE: Implicitly removes from DOM if |node| is inside it.
-      this.content_.insertBefore(node, this.content_.firstChild);
-      this.positionNotification_();
-    },
-
-    /**
      * Removes the tilePage from the DOM and cleans up event handlers.
      */
     remove: function() {
       // This checks arguments.length as most remove functions have a boolean
       // |opt_animate| argument, but that's not necesarilly applicable to
       // removing a tilePage. Selecting a different card in an animated way and
       // deleting the card afterward is probably a better choice.
       assert(typeof arguments[0] != 'boolean',
              'This function takes no |opt_animate| argument.');
       this.tearDown_();
@@ skipping 78 matching lines @@
 
     /**
      * Called when the page loses selection (in the card selector).
      * @param {Event} e A custom carddeselected event.
      * @private
      */
     handleCardDeselection_: function(e) {
     },
 
     /**
-     * Position the notification if there's one showing.
-     * TODO(pedrosimonetti): Fix the position of the notification.
-     */
-    positionNotification_: function() {
-    },
-
-    /**
      * Scrolls the page in response to an mousewheel event, although the event
      * may have been triggered on a different element. Return true if the
      * event triggered scrolling, and false otherwise.
      * This is called explicitly, which allows a consistent experience whether
      * the user scrolls on the page or on the page switcher, because this
      * function provides a common conversion factor between wheel delta and
      * scroll delta.
      * @param {Event} e The mousewheel event.
      */
     handleMouseWheel: function(e) {
@@ skipping 474 matching lines @@
   }
 
   return {
     // TODO(pedrosimonetti): Drag. Delete after porting the rest of the code.
     getCurrentlyDraggingTile2: deprecate,
     setCurrentDropEffect2: deprecate,
     Tile2: Tile,
     TilePage2: TilePage
   };
 });