Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(299)

Issue 6293012: Delete unused strings, round 2. (Closed)

Created:
9 years, 11 months ago by Evan Stade
Modified:
9 years, 6 months ago
Reviewers:
tony, TVL
CC:
chromium-reviews, finnur+watch_chromium.org, jshin+watch_chromium.org
Visibility:
Public.

Description

Delete unused strings, round 2. Delete strings from .grd files that are at least 6 months old and aren't used anywhere. BUG=none TEST=compile Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71903

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : . #

Total comments: 1

Patch Set 4 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -454 lines) Patch
M chrome/app/generated_resources.grd View 1 2 3 52 chunks +85 lines, -445 lines 0 comments Download
M chrome/app/resources/locale_settings.grd View 1 chunk +0 lines, -9 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Stade
more of the same
9 years, 11 months ago (2011-01-20 00:14:39 UTC) #1
tony
LGTM once the try bots go green. http://codereview.chromium.org/6293012/diff/5001/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): http://codereview.chromium.org/6293012/diff/5001/chrome/app/generated_resources.grd#newcode9735 chrome/app/generated_resources.grd:9735: asdfjkl789 These ...
9 years, 11 months ago (2011-01-20 00:21:11 UTC) #2
tony
On 2011/01/20 00:21:11, tony wrote: > LGTM once the try bots go green. > > ...
9 years, 11 months ago (2011-01-20 00:28:26 UTC) #3
TVL
9 years, 11 months ago (2011-01-20 00:56:32 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698