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

Issue 2339783002: Reland of 🍵 Refactor the various locale_paks() templates to be more shared (Closed)

Created:
4 years, 3 months ago by agrieve
Modified:
4 years, 3 months ago
CC:
chromium-reviews, extensions-reviews_chromium.org, chromium-apps-reviews_chromium.org, android-webview-reviews_chromium.org, sdefresne+watch_chromium.org, brettw
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland of Refactor the various locale_paks() templates to be more shared Reverted by: https://codereview.chromium.org/2340673002/ Reason for reland: Now gn gens for enable_resource_whitelist_generation = true Main motivation for this is to more easily add per-target repack_whitelist. TBR=michaelbai@chromium.org,thakis@chromium.org,sdefresne@chromium.org,estevenson@chromium.org,rsesek@chromium.org,brettw@chromium.org BUG=645716 Committed: https://crrev.com/d83512c644c7c61331a32c97ba549fcb1d8793cb Cr-Commit-Position: refs/heads/master@{#418450}

Patch Set 1 #

Patch Set 2 : deps += -> deps = #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+218 lines, -532 lines) Patch
M android_webview/webview_repack_locales.gni View 1 1 chunk +11 lines, -89 lines 0 comments Download
M android_webview/webview_repack_locales_list.gni View 1 chunk +1 line, -1 line 0 comments Download
M chrome/BUILD.gn View 1 4 chunks +16 lines, -2 lines 0 comments Download
M chrome/android/monochrome_repack_locales.gni View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_repack_locales.gni View 1 5 chunks +44 lines, -143 lines 0 comments Download
M ios/chrome/app/resources/BUILD.gn View 1 2 chunks +2 lines, -2 lines 0 comments Download
M ios/chrome/app/resources/ios_chrome_repack.gni View 1 4 chunks +17 lines, -143 lines 0 comments Download
M ios/chrome/extension_repack.gni View 1 1 chunk +9 lines, -107 lines 1 comment Download
M ios/chrome/share_extension/BUILD.gn View 1 1 chunk +0 lines, -1 line 0 comments Download
M ios/chrome/today_extension/BUILD.gn View 1 1 chunk +1 line, -0 lines 0 comments Download
M ios/web/test/BUILD.gn View 1 2 chunks +2 lines, -2 lines 0 comments Download
M tools/grit/repack.gni View 1 3 chunks +114 lines, -41 lines 0 comments Download

Messages

Total messages: 10 (4 generated)
agrieve
Created Reland of 🍵 Refactor the various locale_paks() templates to be more shared
4 years, 3 months ago (2016-09-14 00:02:06 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2339783002/260001
4 years, 3 months ago (2016-09-14 00:08:33 UTC) #4
commit-bot: I haz the power
Committed patchset #2 (id:260001)
4 years, 3 months ago (2016-09-14 01:39:05 UTC) #6
commit-bot: I haz the power
Patchset 2 (id:??) landed as https://crrev.com/d83512c644c7c61331a32c97ba549fcb1d8793cb Cr-Commit-Position: refs/heads/master@{#418450}
4 years, 3 months ago (2016-09-14 01:42:36 UTC) #8
sdefresne
https://codereview.chromium.org/2339783002/diff/260001/ios/chrome/extension_repack.gni File ios/chrome/extension_repack.gni (left): https://codereview.chromium.org/2339783002/diff/260001/ios/chrome/extension_repack.gni#oldcode52 ios/chrome/extension_repack.gni:52: output = "$target_gen_dir/${invoker.output_locale}.lproj/locale.pak" This change is breaking our downstream ...
4 years, 3 months ago (2016-09-14 08:02:53 UTC) #9
sdefresne
4 years, 3 months ago (2016-09-14 08:05:43 UTC) #10
Message was sent while issue was closed.
A revert of this CL (patchset #2 id:260001) has been created in
https://codereview.chromium.org/2338213002/ by sdefresne@chromium.org.

The reason for reverting is: The refactoring of the ios file is likely incorrect
as it breaks the autoroller (as some files are no longer generated or have
different name).

Reverting as half of the touched files are in ios/..

Powered by Google App Engine
This is Rietveld 408576698