[ntp4] Bookmarks page implementation, first-pass.

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 482 matching lines @@
       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);
       }
     },
 
     /**
+     * Removes all tiles from the page.
+     */
+    removeAllTiles: function() {
+      while (this.tileElements_.length) {

[arv (Not doing code reviews), 2011/08/13 01:02:07]

why not use?

this.tileGrid_.innerHTML = '';

[csilv, 2011/08/13 01:48:17]

I've changed this since this is going to be more efficient.  This assumes that
we'll only put tile elements into the grid, Evan, any concerns about this?

+        this.tileGrid_.removeChild(this.tileElements_[0]);
+      }
+    },
+
+    /**
      * Makes some calculations for tile layout. These change depending on
      * height, width, and the number of tiles.
      * TODO(estade): optimize calls to this function. Do nothing if the page is
      * hidden, but call before being shown.
      * @private
      */
     calculateLayoutValues_: function() {
       var grid = this.gridValues_;
       var availableSpace = this.tileGrid_.clientWidth - 2 * MIN_WIDE_MARGIN;
       var wide = availableSpace >= grid.minWideWidth;
@@ skipping 482 matching lines @@
      */
     tileMoved: function(draggedTile) {
     },
   };
 
   return {
     getCurrentlyDraggingTile: getCurrentlyDraggingTile,
     TilePage: TilePage,
   };
 });