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

Issue 171673004: Remove pp_ifdef from resource files in chrome/app/. (Closed)

Created:
6 years, 10 months ago by newt (away)
Modified:
6 years, 10 months ago
Reviewers:
oshima
CC:
chromium-reviews, oshima+watch_chromium.org, jshin+watch_chromium.org
Visibility:
Public.

Description

Remove pp_ifdef from resource files in chrome/app/. Generated using: > cd src/chrome/app > git grep -l pp_ifdef | xargs sed -i "s:pp_ifdef('\([^']\+\)'):\1:g" BUG=343235 R=oshima@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252072

Patch Set 1 #

Patch Set 2 : rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+408 lines, -408 lines) Patch
M chrome/app/bookmarks_strings.grdp View 11 chunks +18 lines, -18 lines 0 comments Download
M chrome/app/chromeos_strings.grdp View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/app/chromium_strings.grd View 23 chunks +43 lines, -43 lines 0 comments Download
M chrome/app/generated_resources.grd View 119 chunks +166 lines, -166 lines 0 comments Download
M chrome/app/google_chrome_strings.grd View 16 chunks +30 lines, -30 lines 0 comments Download
M chrome/app/resources/locale_settings.grd View 8 chunks +16 lines, -16 lines 0 comments Download
M chrome/app/resources/locale_settings_chromiumos.grd View 4 chunks +11 lines, -11 lines 0 comments Download
M chrome/app/resources/locale_settings_google_chromeos.grd View 4 chunks +11 lines, -11 lines 0 comments Download
M chrome/app/resources/locale_settings_linux.grd View 4 chunks +11 lines, -11 lines 0 comments Download
M chrome/app/resources/platform_locale_settings/locale_settings_cros_ja.xtb View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/app/resources/platform_locale_settings/locale_settings_cros_zh-CN.xtb View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/app/resources/platform_locale_settings/locale_settings_cros_zh-TW.xtb View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/app/theme/chrome_unscaled_resources.grd View 4 chunks +8 lines, -8 lines 0 comments Download
M chrome/app/theme/theme_resources.grd View 1 34 chunks +87 lines, -87 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
newt (away)
more of the same :) PTAL, thanks!
6 years, 10 months ago (2014-02-19 01:15:35 UTC) #1
oshima
lgtm
6 years, 10 months ago (2014-02-19 18:40:26 UTC) #2
newt (away)
The CQ bit was checked by newt@chromium.org
6 years, 10 months ago (2014-02-19 19:26:57 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/newt@chromium.org/171673004/1
6 years, 10 months ago (2014-02-19 19:39:24 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 10 months ago (2014-02-19 19:39:28 UTC) #5
commit-bot: I haz the power
Failed to apply patch for chrome/app/theme/theme_resources.grd: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 10 months ago (2014-02-19 19:39:29 UTC) #6
newt (away)
6 years, 10 months ago (2014-02-19 20:48:37 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 manually as r252072 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698