ntp4: re-enable bookmark pane

chrome/browser/resources/ntp4/new_tab.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.
 
 /**
  * @fileoverview New tab page
  * This is the main code for the new tab page used by touch-enabled Chrome
  * browsers.  For now this is still a prototype.
  */
 
@@ skipping 190 matching lines @@
       bubbleContent.hidden = false;
 
       var learnMoreLink = infoBubble.querySelector('a');
       learnMoreLink.href = localStrings.getString('ntp4_intro_url');
       learnMoreLink.onclick = infoBubble.hide.bind(infoBubble);
 
       infoBubble.show();
       chrome.send('introMessageSeen');
     }
 
-    /*
     bookmarksPage = new ntp4.BookmarksPage();
     appendTilePage(bookmarksPage, localStrings.getString('bookmarksPage'));
     chrome.send('getBookmarksData');
-    */
 
     var serverpromo = localStrings.getString('serverpromo');
     if (serverpromo) {
       showNotification(parseHtmlSubset(serverpromo), [], function() {
         chrome.send('closePromo');
       }, 60000);
     }
   }
 
   /**
@@ skipping 185 matching lines @@
    *     applications.
    */
   function appsPrefChangeCallback(data) {
     for (var i = 0; i < data.apps.length; ++i) {
       $(data.apps[i].id).appData = data.apps[i];
     }
 
     // Set the App dot names. Skip the first and last dots (Most Visited and
     // Bookmarks).
     var dots = dotList.getElementsByClassName('dot');
-    for (var i = 1; i < dots.length - 1; ++i) {
+    for (var i = 1; i < dots.length - 2; ++i) {
       dots[i].displayTitle = data.appPageNames[i - 1] || '';
     }
   }
 
   /**
    * Listener for offline status change events. Updates apps that are
    * not offline-enabled to be grayscale if the browser is offline.
    */
   function updateOfflineEnabledApps() {
     var apps = document.querySelectorAll('.app');
@@ skipping 10 matching lines @@
   }
 
   /**
    * Invoked whenever the pages in apps-page-list have changed so that
    * the Slider knows about the new elements.
    */
   function updateSliderCards() {
     var pageNo = Math.min(cardSlider.currentCard, tilePages.length - 1);
     cardSlider.setCards(Array.prototype.slice.call(tilePages), pageNo);
     switch (shownPage) {
-      case templateData['bookmarks_page_id']:
       case templateData['apps_page_id']:
         cardSlider.selectCardByValue(
             appsPages[Math.min(shownPageIndex, appsPages.length - 1)]);
         break;
-/*
+      case templateData['bookmarks_page_id']:
         cardSlider.selectCardByValue(bookmarksPage);
         break;
-*/
       case templateData['most_visited_page_id']:
         cardSlider.selectCardByValue(mostVisitedPage);
         break;
     }
   }
 
   /**
    * Appends a tile page (for bookmarks or most visited).
    *
    * @param {TilePage} page The page element.
@@ skipping 395 matching lines @@
 var assert = ntp4.assert;
 var getAppsCallback = ntp4.getAppsCallback;
 var appsPrefChangeCallback = ntp4.appsPrefChangeCallback;
 var themeChanged = ntp4.themeChanged;
 var recentlyClosedTabs = ntp4.setRecentlyClosedTabs;
 var setMostVisitedPages = ntp4.setMostVisitedPages;
 
 document.addEventListener('DOMContentLoaded', ntp4.initialize);
 window.addEventListener('online', ntp4.updateOfflineEnabledApps);
 window.addEventListener('offline', ntp4.updateOfflineEnabledApps);