Fix apps page footer shrinking/overlap for small window sizes.

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 110 matching lines @@
       sectionsToWaitFor++;
       if (loadTimeData.getBoolean('showAppLauncherPromo')) {
         $('app-launcher-promo-close-button').addEventListener('click',
             function() { chrome.send('stopShowingAppLauncherPromo'); });
         $('apps-promo-learn-more').addEventListener('click',
             function() { chrome.send('onLearnMore'); });
       }
     }
     if (loadTimeData.getBoolean('isDiscoveryInNTPEnabled'))
       sectionsToWaitFor++;
     measureNavDots();

[Dan Beam, 2014/01/09 18:58:19]

^ you could do it here

 
     // Load the current theme colors.
     themeChanged();
 
     newTabView = new NewTabView();
 
     notificationContainer = getRequiredElement('notification-container');
     notificationContainer.addEventListener(
         'webkitTransitionEnd', onNotificationTransitionEnd);
 
@@ skipping 207 matching lines @@
     var measuringDiv = $('fontMeasuringDiv');
     measuringDiv.textContent = loadTimeData.getString(id);
     // The 4 is for border and padding.
     return Math.max(measuringDiv.clientWidth * 1.15 + 4, 80);
   }
 
   /**
    * Fills in an invisible div with the longest dot title string so that
    * its length may be measured and the nav dots sized accordingly.
    */
   function measureNavDots() {

[Dan Beam, 2014/01/09 18:58:19]

or here

     var pxWidth = measureNavDot('appDefaultPageName');
     if (loadTimeData.getBoolean('showMostvisited'))
       pxWidth = Math.max(measureNavDot('mostvisited'), pxWidth);
 
     var styleElement = document.createElement('style');
     styleElement.type = 'text/css';
     // max-width is used because if we run out of space, the nav dots will be
     // shrunk.
     styleElement.textContent = '.dot { max-width: ' + pxWidth + 'px; }';
     document.querySelector('head').appendChild(styleElement);
@@ skipping 332 matching lines @@
     setMostVisitedPages: setMostVisitedPages,
     setSuggestionsPages: setSuggestionsPages,
     setRecentlyClosedTabs: setRecentlyClosedTabs,
     setFaviconDominantColor: setFaviconDominantColor,
     showNotification: showNotification,
     themeChanged: themeChanged,
     updateLogin: updateLogin
   };
 });
 
-document.addEventListener('DOMContentLoaded', ntp.onLoad);
+document.addEventListener('DOMContentLoaded', function() {
+  var logo = $('logo-img').querySelector('img');
+  if (logo.complete)
+    ntp.onLoad();
+  else
+    logo.addEventListener('load', ntp.onLoad);
+});

[Dan Beam, 2014/01/09 18:58:19]

^ you should not block the whole NTP on this.

[dconnelly, 2014/01/10 10:46:57]

Done.

 
 var toCssPx = cr.ui.toCssPx;