Added new adaptive "Suggest" tab on the New Tab Page, behing the flag, for the experiments.

chrome/browser/resources/ntp4/new_tab.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.
 
 /**
  * @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 95 matching lines @@
         'webkitTransitionEnd', onNotificationTransitionEnd);
 
     cr.ui.decorate($('recently-closed-menu-button'), ntp4.RecentMenuButton);
     chrome.send('getRecentlyClosedTabs');
 
     newTabView.appendTilePage(new ntp4.MostVisitedPage(),
                               localStrings.getString('mostvisited'),
                               false);
     chrome.send('getMostVisited');
 
+    if (templateData.isSuggestionsPageEnabled) {
+      var suggestions_script = document.createElement('script');
+      suggestions_script.src = 'suggestions_page.js';
+      document.querySelector('head').appendChild(suggestions_script);
+    }

[Dan Beam, 2012/02/28 20:14:24]

Seems easier just to do this in i18n-values unless there's timing issues.

[GeorgeY, 2012/02/28 21:17:42]

?
This is the way it is done elsewhere. Not sure it can be easier

+
     var webstoreLink = localStrings.getString('webStoreLink');
     if (templateData.isWebStoreExperimentEnabled) {
       var url = appendParam(webstoreLink, 'utm_source', 'chrome-ntp-launcher');
       $('chrome-web-store-href').href = url;
       $('chrome-web-store-href').addEventListener('click',
           onChromeWebStoreButtonClick);
 
       $('footer-content').classList.add('enable-cws-experiment');
     }
 
@@ skipping 292 matching lines @@
 
   function setRecentlyClosedTabs(dataItems) {
     $('recently-closed-menu-button').dataItems = dataItems;
   }
 
   function setMostVisitedPages(data, hasBlacklistedUrls) {
     newTabView.mostVisitedPage.data = data;
     cr.dispatchSimpleEvent(document, 'sectionready', true, true);
   }
 
+  function setSuggestionsPages(data, hasBlacklistedUrls) {
+    newTabView.suggestionsPage.data = data;
+  }
+
   /**
    * Set the dominant color for a node. This will be called in response to
    * getFaviconDominantColor. The node represented by |id| better have a setter
    * for stripeColor.
    * @param {string} id The ID of a node.
    * @param {string} color The color represented as a CSS string.
    */
   function setStripeColor(id, color) {
     var node = $(id);
     if (node)
@@ skipping 68 matching lines @@
   }
 
   function getAppsPageIndex() {
     return newTabView.getAppsPageIndex.apply(newTabView, arguments);
   }
 
   function getCardSlider() {
     return newTabView.cardSlider;
   }
 
+  function getNewTabView() {
+    return newTabView;
+  }

[Dan Beam, 2012/02/28 20:14:24]

this should not be required

[GeorgeY, 2012/02/28 21:17:42]

It is. Either newTabView or getNewTabView needs to be exported. This way is
clearer

[Dan Beam, 2012/02/28 22:30:12]

It doesn't need to be exported if you just use onload of the script and closure.

[GeorgeY, 2012/02/29 00:15:45]

done.

+
   function leaveRearrangeMode() {
     return newTabView.leaveRearrangeMode.apply(newTabView, arguments);
   }
 
   function saveAppPageName() {
     return newTabView.saveAppPageName.apply(newTabView, arguments);
   }
 
   function setAppToBeHighlighted(appId) {
     newTabView.highlightAppId = appId;
   }
 
   // Return an object with all the exports
   return {
     appAdded: appAdded,
     appMoved: appMoved,
     appRemoved: appRemoved,
     appsPrefChangeCallback: appsPrefChangeCallback,
     enterRearrangeMode: enterRearrangeMode,
     getAppsCallback: getAppsCallback,
     getAppsPageIndex: getAppsPageIndex,
     getCardSlider: getCardSlider,
+    getNewTabView: getNewTabView,
     onLoad: onLoad,
     leaveRearrangeMode: leaveRearrangeMode,
     saveAppPageName: saveAppPageName,
     setAppToBeHighlighted: setAppToBeHighlighted,
     setMostVisitedPages: setMostVisitedPages,
+    setSuggestionsPages: setSuggestionsPages,
     setRecentlyClosedTabs: setRecentlyClosedTabs,
     setStripeColor: setStripeColor,
     showNotification: showNotification,
     themeChanged: themeChanged,
     updateLogin: updateLogin
   };
 });
 
 // publish ntp globals
 // TODO(estade): update the content handlers to use ntp namespace instead of
 // making these global.
 var getAppsCallback = ntp4.getAppsCallback;
 var appsPrefChangeCallback = ntp4.appsPrefChangeCallback;
 var themeChanged = ntp4.themeChanged;
 var recentlyClosedTabs = ntp4.setRecentlyClosedTabs;
 var setMostVisitedPages = ntp4.setMostVisitedPages;
 var updateLogin = ntp4.updateLogin;
 
 document.addEventListener('DOMContentLoaded', ntp4.onLoad);