[Local NTP] Cleanup: Remove support for "icons" layout

chrome/browser/resources/local_ntp/local_ntp.js

 // Copyright 2015 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 The local InstantExtended NTP.
  */
 
 
@@ skipping 15 matching lines @@
 }
 
 
 /**
  * Specifications for an NTP design (not comprehensive).
  *
  * fakeboxWingSize: Extra distance for fakebox to extend beyond beyond the list
  *   of tiles.
  * fontFamily: Font family to use for title and thumbnail iframes.
  * fontSize: Font size to use for the iframes, in px.
- * mainClass: Class applied to #ntp-contents to control CSS.
  * numTitleLines: Number of lines to display in titles.
  * showFavicon: Whether to show favicon.
  * thumbnailTextColor: The 4-component color that thumbnail iframe may use to
  *   display text message in place of missing thumbnail.
  * thumbnailFallback: (Optional) A value in THUMBNAIL_FALLBACK to specify the
  *   thumbnail fallback strategy. If unassigned, then the thumbnail.html
  *   iframe would handle the fallback.
  * tileWidth: The width of each suggestion tile, in px.
  * tileMargin: Spacing between successive tiles, in px.
  * titleColor: The 4-component color of title text.
  * titleColorAgainstDark: The 4-component color of title text against a dark
  *   theme.
  * titleTextAlign: (Optional) The alignment of title text. If unspecified, the
  *   default value is 'center'.
  * titleTextFade: (Optional) The number of pixels beyond which title
  *   text begins to fade. This overrides the default ellipsis style.
  *
  * @type {{
  *   fakeboxWingSize: number,
  *   fontFamily: string,
  *   fontSize: number,
- *   mainClass: string,
  *   numTitleLines: number,
  *   showFavicon: boolean,
  *   thumbnailTextColor: string,
  *   thumbnailFallback: string|null|undefined,
  *   tileWidth: number,
  *   tileMargin: number,
  *   titleColor: string,
  *   titleColorAgainstDark: string,
  *   titleTextAlign: string|null|undefined,
  *   titleTextFade: number|null|undefined
  * }}
  */
 var NTP_DESIGN = {
   fakeboxWingSize: 0,
   fontFamily: 'arial, sans-serif',
   fontSize: 12,
-  mainClass: 'thumb-ntp',
   numTitleLines: 1,
   showFavicon: true,
   thumbnailTextColor: [50, 50, 50, 255],
   thumbnailFallback: 'dot',  // Draw single dot.
   tileWidth: 154,
   tileMargin: 16,
   titleColor: [50, 50, 50, 255],
   titleColorAgainstDark: [210, 210, 210, 255],
   titleTextAlign: 'inherit',
   titleTextFade: 122 - 36  // 112px wide title with 32 pixel fade at end.
 };
 
 
 /**
- * Modifies NTP_DESIGN parameters for icon NTP.
- */
-function modifyNtpDesignForIcons() {
-  NTP_DESIGN.fakeboxWingSize = 132;
-  NTP_DESIGN.mainClass = 'icon-ntp';
-  NTP_DESIGN.numTitleLines = 2;
-  NTP_DESIGN.showFavicon = false;
-  NTP_DESIGN.thumbnailFallback = null;
-  NTP_DESIGN.tileWidth = 48 + 2 * 18;
-  NTP_DESIGN.tileMargin = 60 - 18 * 2;
-  NTP_DESIGN.titleColor = [120, 120, 120, 255];
-  NTP_DESIGN.titleColorAgainstDark = [210, 210, 210, 255];
-  NTP_DESIGN.titleTextAlign = 'center';
-  delete NTP_DESIGN.titleTextFade;
-}
-
-
-/**
  * Enum for classnames.
  * @enum {string}
  * @const
  */
 var CLASSES = {
   ALTERNATE_LOGO: 'alternate-logo', // Shows white logo if required by theme
   DARK: 'dark',
   DEFAULT_THEME: 'default-theme',
   DELAYED_HIDE_NOTIFICATION: 'mv-notice-delayed-hide',
   FAKEBOX_DISABLE: 'fakebox-disable', // Makes fakebox non-interactive
@@ skipping 606 matching lines @@
  */
 function handlePostMessage(event) {
   var cmd = event.data.cmd;
   var args = event.data;
   if (cmd == 'tileBlacklisted') {
     showNotification();
     lastBlacklistedTile = args.tid;
 
     ntpApiHandle.deleteMostVisitedItem(args.tid);
   }
+  // TODO(treib): Should we also handle the 'loaded' message from the iframe
+  // here? We could hide the page until it arrives, to avoid flicker.
 }
 
 
 /**
  * Prepares the New Tab Page by adding listeners, rendering the current
  * theme, the most visited pages section, and Google-specific elements for a
  * Google-provided page.
  */
 function init() {
   notification = $(IDS.NOTIFICATION);
@@ skipping 13 matching lines @@
         '" autocomplete="off" tabindex="-1" type="url" aria-hidden="true">');
     fakeboxHtml.push('<div id="cursor"></div>');
     fakebox.innerHTML = fakeboxHtml.join('');
 
     ntpContents.insertBefore(fakebox, ntpContents.firstChild);
     ntpContents.insertBefore(logo, ntpContents.firstChild);
   } else {
     document.body.classList.add(CLASSES.NON_GOOGLE_PAGE);
   }
 
-  // Modify design for experimental icon NTP, if specified.
-  if (configData.useIcons)
-    modifyNtpDesignForIcons();
-  document.querySelector('#ntp-contents').classList.add(NTP_DESIGN.mainClass);
-
   // Hide notifications after fade out, so we can't focus on links via keyboard.
   notification.addEventListener('webkitTransitionEnd', hideNotification);
 
   var notificationMessage = $(IDS.NOTIFICATION_MESSAGE);
   notificationMessage.textContent =
       configData.translatedStrings.thumbnailRemovedNotification;
 
   var undoLink = $(IDS.UNDO_LINK);
   undoLink.addEventListener('click', onUndo);
   registerKeyHandler(undoLink, KEYCODE.ENTER, onUndo);
@@ skipping 80 matching lines @@
 
   var iframe = document.createElement('iframe');
   // Change the order of tabbing the page to start with NTP tiles.
   iframe.setAttribute('tabindex', '1');
   iframe.id = 'mv-single';
 
   var args = [];
 
   if (searchboxApiHandle.rtl)
     args.push('rtl=1');
-  if (window.configData.useIcons)
-    args.push('icons=1');
   if (NTP_DESIGN.numTitleLines > 1)
     args.push('ntl=' + NTP_DESIGN.numTitleLines);
 
   args.push('removeTooltip=' +
       encodeURIComponent(configData.translatedStrings.removeThumbnailTooltip));
 
   iframe.src = '//most-visited/single.html?' + args.join('&');
   $(IDS.TILES).appendChild(iframe);
 
   iframe.onload = function() {
@@ skipping 14 matching lines @@
 
 return {
   init: init,
   listen: listen
 };
 }
 
 if (!window.localNTPUnitTest) {
   LocalNTP().listen();
 }