[ntp4] Add a 'Manage Bookmarks' link at the top of the Bookmarks page. Also right-align links.

chrome/browser/resources/ntp4/bookmarks_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';
 
   var localStrings = new LocalStrings;
 
   /**
@@ skipping 215 matching lines @@
 
     /**
      * Initialize the bookmarks page object.
      */
     initialize: function() {
       this.classList.add('bookmarks-page');
 
       // insert the bookmark titles header which is unique to bookmark pages.
       this.insertBefore($('bookmarks-title-wrapper'), this.firstChild);
 
-      // insert a container for a link to a Bookmarks Manager page.
-      var link = document.createElement('a');
-      link.className = 'bookmarks-manager-link';
-      link.textContent = localStrings.getString('bookmarksManagerLinkTitle');
-      var container = document.createElement('div');
-      container.className = 'bookmarks-manager-link-container';
-      container.hidden = true;
-      container.appendChild(link);
-      this.querySelector('.tile-page-content').appendChild(container);
+      // insert the top & bottom links for the Bookmarks Manager page.

[Evan Stade, 2011/09/05 01:59:20]

Insert

[csilv, 2011/09/06 05:33:33]

Done.

+      var pageContent = this.querySelector('.tile-page-content');
+      var topWrapper = $('bookmarks-top-link-wrapper');
+      pageContent.insertBefore(topWrapper, pageContent.firstChild);
+      topWrapper.hidden = false;
+      pageContent.appendChild($('bookmarks-bottom-link-wrapper'));
     },
 
     /**
      * Build the bookmark titles bar (ie, navigation hiearchy).
      * @param {Array} items The parent hiearchy of the current folder.
      * @private
      */
     updateBookmarkTitles_: function(items) {
       var wrapper = $('bookmarks-title-wrapper');
       var title = wrapper.querySelector('.section-title');
@@ skipping 16 matching lines @@
      * Build the bookmark tiles.
      * @param {Array} items The contents of the current folder.
      * @private
      */
     updateBookmarkTiles_: function(items) {
       this.removeAllTiles();
       var tile_count = Math.min(items.length, MAX_BOOKMARK_TILES);
       for (var i = 0; i < tile_count; i++)
         this.appendTile(new Bookmark(items[i]), false);
 
-      var container = this.querySelector('.bookmarks-manager-link-container');
+      var wrapper = $('bookmarks-bottom-link-wrapper');
       if (items.length > MAX_BOOKMARK_TILES) {
-        var link = container.querySelector('.bookmarks-manager-link');
+        var link = wrapper.querySelector('a');
         link.href = 'chrome://bookmarks/#' + this.id;
-        container.hidden = false;
+        wrapper.hidden = false;
       } else {
-        container.hidden = true;
+        wrapper.hidden = true;
       }
     },
 
     /**
      * Determine whether a bookmark ID matches a folder in the current
      * hierarchy.
      * @param {string} id The bookmark ID to search for.
      * @private
      */
     isBookmarkInParentHierarchy_: function(id) {
@@ skipping 182 matching lines @@
       tile.stripeColor = color;
   };
 
   return {
     BookmarksPage: BookmarksPage,
     initBookmarkChevron: initBookmarkChevron,
   };
 });
 
 window.addEventListener('load', ntp4.initBookmarkChevron);