ntp string cleanse

chrome/browser/resources/ntp4/most_visited_page.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.
 
 cr.define('ntp4', function() {
   'use strict';
 
   var TilePage = ntp4.TilePage;
 
   /**
+   * A counter for generating unique tile IDs.
    */
   var tileID = 0;
 
   /**
    * Creates a new Most Visited object for tiling.
    * @constructor
    * @extends {HTMLAnchorElement}
    */
   function MostVisited() {
     var el = cr.doc.createElement('a');
@@ skipping 33 matching lines @@
             '<div class="close-button"></div>' +
             '<span class="thumbnail fills-parent">' +
               // thumbnail-shield provides a gradient fade effect.
               '<div class="thumbnail-shield fills-parent"></div>' +
             '</span>' +
             '<span class="favicon"></span>' +
           '</span>' +
           '<div class="color-stripe"></div>' +
           '<span class="title"></span>';
 
+      this.querySelector('.close-button').title =
+          templateData.removethumbnailtooltip;

[Rick Byers, 2011/11/30 15:16:07]

Nit: This seems mostly unrelated to the string removal.  Please mention it in
the CL description as an addition fix/improvement.

[Evan Stade, 2011/11/30 22:47:25]

Done.

+
       this.tabIndex = -1;
       this.data_ = null;
       this.removeAttribute('id');
       this.title = '';
     },
 
     /**
      * Update the appearance of this tile according to |data|.
      * @param {Object} data A dictionary of relevant data for the page.
      */
@@ skipping 339 matching lines @@
     }
 
     return oldData;
   };
 
   return {
     MostVisitedPage: MostVisitedPage,
     refreshData: refreshData,
   };
 });