ntp4: small followup to r91606

chrome/browser/resources/ntp4/tile_page.js

 // Copyright (c) 2011 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('ntp4', function() {
   'use strict';
 
   // We can't pass the currently dragging tile via dataTransfer because of
   // http://crbug.com/31037
   var currentlyDraggingTile = null;
@@ skipping 239 matching lines @@
     },
 
     /**
      * Callback for the webkitAnimationEnd event on the tile's contents.
      * @param {Event} e The event object.
      */
     onContentsAnimationEnd_: function(e) {
       if (this.firstChild.classList.contains('new-tile-contents'))
         this.firstChild.classList.remove('new-tile-contents');
       else if (this.firstChild.classList.contains('removing-tile-contents'))
-        this.parentNode.removeChild(this);
+        this.tilePage.removeTile(this);
     },
   };
 
   /**
    * Gives the proportion of the row width that is devoted to a single icon.
    * @param {number} rowTileCount The number of tiles in a row.
    * @return {number} The ratio between icon width and row width.
    */
   function tileWidthFraction(rowTileCount) {
     return rowTileCount +
@@ skipping 173 matching lines @@
      * @protected
      */
     appendTile: function(tileElement, animate) {
       this.addTileAt(tileElement, this.tileElements_.length, animate);
     },
 
     /**
      * Adds the given element to the tile grid.
      * @param {Node} tileElement The tile object/node to insert.
      * @param {number} index The location in the tile grid to insert it at.
-     * @param {?boolean} animate If true, the add will be animated.
+     * @param {?boolean} animate If true, the tile in question will be animated
+     *     (other tiles, if they must reposition, do not animate).
      * @protected
      */
     addTileAt: function(tileElement, index, animate) {
       this.classList.remove('animating-tile-page');
       if (animate)
         tileElement.classList.add('new-tile-contents');
       var wrapperDiv = new Tile(tileElement);
       if (index == this.tileElements_.length) {
         this.tileGrid_.appendChild(wrapperDiv);
       } else {
         this.tileGrid_.insertBefore(wrapperDiv,
                                     this.tileElements_[index]);
       }
       this.calculateLayoutValues_();
 
       this.positionTile_(index);
     },
 
     /**
+     * Removes the given tile and animates the respositioning of the other
+     * tiles.
+     * @param {HTMLElement} tile The tile to remove from |tileGrid_|.
+     */
+    removeTile: function(tile) {
+      this.classList.add('animating-tile-page');
+      var index = Array.prototype.indexOf.call(this.tileElements_, tile);
+      tile.parentNode.removeChild(tile);
+      this.calculateLayoutValues_();
+      for (var i = index; i < this.tileElements_.length; i++) {
+        this.positionTile_(i);
+      }
+    },
+
+    /**
      * Makes some calculations for tile layout. These change depending on
      * height, width, and the number of tiles.
      * @private
      */
     calculateLayoutValues_: function() {
       var grid = this.gridValues_;
       var availableSpace = this.tileGrid_.clientWidth - 2 * MIN_WIDE_MARGIN;
       var wide = availableSpace >= grid.minWideWidth;
       var numRowTiles = wide ? grid.maxColCount : grid.minColCount;
 
@@ skipping 518 matching lines @@
      */
     tileMoved: function(draggedTile) {
     },
   };
 
   return {
     getCurrentlyDraggingTile: getCurrentlyDraggingTile,
     TilePage: TilePage,
   };
 });