ntp string cleanse

ntp string cleanse

remove unused strings, and in one case start using an unused string (most visited delete tooltip)

a couple of the strings may still used by touch ntp and in the bookmarks pane, but both of these are getting removed quite soon

BUG=none
TEST=compile and visual inspection


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112366

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

LGTM with nit

Note that I didn't actually verify each of the strings you removed isn't
actually used outside of the code that's getting deleted.

http://codereview.chromium.org/8746007/diff/1/chrome/browser/resources/ntp4/m...
File chrome/browser/resources/ntp4/most_visited_page.js (right):

http://codereview.chromium.org/8746007/diff/1/chrome/browser/resources/ntp4/m...
chrome/browser/resources/ntp4/most_visited_page.js:66:
templateData.removethumbnailtooltip;
Nit: This seems mostly unrelated to the string removal.  Please mention it in
the CL description as an addition fix/improvement.

[csilv, 2011-11-30 18:30:27]

lgtm

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

http://codereview.chromium.org/8746007/diff/1/chrome/browser/resources/ntp4/m...
File chrome/browser/resources/ntp4/most_visited_page.js (right):

http://codereview.chromium.org/8746007/diff/1/chrome/browser/resources/ntp4/m...
chrome/browser/resources/ntp4/most_visited_page.js:66:
templateData.removethumbnailtooltip;
On 2011/11/30 15:16:07, Rick Byers wrote:
> Nit: This seems mostly unrelated to the string removal.  Please mention it in
> the CL description as an addition fix/improvement.

Done.

[commit-bot: I haz the power, 2011-11-30 22:47:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/8746007/1

[commit-bot: I haz the power, 2011-12-01 01:14:35]

Change committed as 112366