Don't show the apps page on the NTP if the app launcher is installed.

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 92 matching lines @@
 
     /** @override */
     appendTilePage: function(page, title, titleIsEditable, opt_refNode) {
       ntp.PageListView.prototype.appendTilePage.apply(this, arguments);
 
       if (promoBubble)
         window.setTimeout(promoBubble.reposition.bind(promoBubble), 0);
     }
   };
 
+  function gotShouldShowApps(shouldShowApps) {
+    if (shouldShowApps != loadTimeData.getBoolean('showApps')) {
+      // TODO(jeremya): update the UI in-place instead of reloading.
+      window.location.reload();
+      return;
+    }
+  }
+
   /**
    * Invoked at startup once the DOM is available to initialize the app.
    */
   function onLoad() {
     sectionsToWaitFor = loadTimeData.getBoolean('showApps') ? 2 : 1;
     if (loadTimeData.getBoolean('isDiscoveryInNTPEnabled'))
       sectionsToWaitFor++;
     measureNavDots();
 
     // Load the current theme colors.
@@ skipping 498 matching lines @@
   return {
     appAdded: appAdded,
     appMoved: appMoved,
     appRemoved: appRemoved,
     appsPrefChangeCallback: appsPrefChangeCallback,
     enterRearrangeMode: enterRearrangeMode,
     getAppsCallback: getAppsCallback,
     getAppsPageIndex: getAppsPageIndex,
     getCardSlider: getCardSlider,
     onLoad: onLoad,
+    gotShouldShowApps: gotShouldShowApps,
     leaveRearrangeMode: leaveRearrangeMode,
     logTimeToClick: logTimeToClick,
     NtpFollowAction: NtpFollowAction,
     saveAppPageName: saveAppPageName,
     setAppToBeHighlighted: setAppToBeHighlighted,
     setBookmarkBarAttached: setBookmarkBarAttached,
     setForeignSessions: setForeignSessions,
     setMostVisitedPages: setMostVisitedPages,
     setSuggestionsPages: setSuggestionsPages,
     setRecentlyClosedTabs: setRecentlyClosedTabs,
     setFaviconDominantColor: setFaviconDominantColor,
     showNotification: showNotification,
     themeChanged: themeChanged,
     updateLogin: updateLogin
   };
 });
 
+// This will end up calling ntp.gotShouldShowApps.
+chrome.send('getShouldShowApps');

[Evan Stade, 2013/01/23 23:34:24]

this should probably be a part of onLoad

[jeremya, 2013/01/24 02:40:04]

Why's that?

[Evan Stade, 2013/01/24 18:40:16]

well, my thinking was that if gotShouldShowApps were called before onLoad, then
you'd have a hard time updating the DOM (your TODO). But I suppose in that case
you could just build the DOM correctly in the first place instead of needing to
update it.

[jeremya, 2013/01/25 00:06:19]

Right, that makes sense -- if we get the gotShouldShowApps message before
onLoad, we could just update the load time data and carry on as normal, without
having to reload. I'll save that for a follow-up patch though.

 document.addEventListener('DOMContentLoaded', ntp.onLoad);
 
 var toCssPx = cr.ui.toCssPx;