GN WebView: add resources

GN WebView: add resources

BUG=532905
Committed: https://crrev.com/ef836d865dbbea4ebbb1c88580c1e35ba30e048d
Cr-Commit-Position: refs/heads/master@{#357049}

[michaelbai, 2015-10-29 22:21:32]

Description was changed from

==========
GN WebView: add resources

BUG=532905
==========

to

==========
GN WebView: add resources

BUG=532905
==========

[michaelbai, 2015-10-29 22:21:32]

michaelbai@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2015-10-30 01:48:43]

lgtm.

https://codereview.chromium.org/1425143004/diff/20001/android_webview/webview...
File android_webview/webview_repack_locales.gni (right):

https://codereview.chromium.org/1425143004/diff/20001/android_webview/webview...
android_webview/webview_repack_locales.gni:6: # modifications to meet WebView's
requirement.
it is unfortunate that we have to duplicate all of this logic. Do you mind
filing a bug to see if we can figure out a way to de-dup things w/
Proj-GN-Migration and maybe assign it to me?

[michaelbai, 2015-10-30 04:21:52]

https://codereview.chromium.org/1425143004/diff/20001/android_webview/webview...
File android_webview/webview_repack_locales.gni (right):

https://codereview.chromium.org/1425143004/diff/20001/android_webview/webview...
android_webview/webview_repack_locales.gni:6: # modifications to meet WebView's
requirement.
On 2015/10/30 01:48:43, Dirk Pranke wrote:
> it is unfortunate that we have to duplicate all of this logic. Do you mind
> filing a bug to see if we can figure out a way to de-dup things w/
> Proj-GN-Migration and maybe assign it to me?

I were thinking about this, but didn't come up with ideas

file the bug to you.

[michaelbai, 2015-10-30 04:21:57]

The CQ bit was checked by michaelbai@chromium.org

[commit-bot: I haz the power, 2015-10-30 04:22:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1425143004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1425143004/20001

[commit-bot: I haz the power, 2015-10-30 05:04:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-30 05:05:14]

Patchset 2 (id:??) landed as
https://crrev.com/ef836d865dbbea4ebbb1c88580c1e35ba30e048d
Cr-Commit-Position: refs/heads/master@{#357049}