👾 GN(android): Filter R.class files before obfuscating via proguard

GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
Committed: https://crrev.com/2142bcde69837585c1571a46cf59715a81d573f9
Cr-Commit-Position: refs/heads/master@{#394638}

[agrieve, 2016-05-18 19:58:24]

The CQ bit was checked by agrieve@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-18 19:59:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1993893002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1993893002/1

[commit-bot: I haz the power, 2016-05-18 21:11:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-18 21:11:20]

Dry run: This issue passed the CQ dry run.

[agrieve, 2016-05-19 01:03:01]

Description was changed from

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
==========

to

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
==========

[agrieve, 2016-05-19 01:03:02]

agrieve@chromium.org changed reviewers:
+ yfriedman@chromium.org

[agrieve, 2016-05-19 01:03:09]

On 2016/05/18 21:11:20, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

[Yaron, 2016-05-19 02:04:36]

lgtm

[agrieve, 2016-05-19 02:12:51]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-05-19 02:13:26]

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

[commit-bot: I haz the power, 2016-05-19 02:18:07]

Description was changed from

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
==========

to

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
==========

[commit-bot: I haz the power, 2016-05-19 02:18:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-19 02:19:50]

Description was changed from

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576
==========

to

==========
GN(android): Filter R.class files before obfuscating via proguard

I haven't heard of this causing issues for us yet, but I realized that
filtering after proguarding could have the result that proguard renames
a class to "R.class", and then filtering strips it out.

BUG=585576

Committed: https://crrev.com/2142bcde69837585c1571a46cf59715a81d573f9
Cr-Commit-Position: refs/heads/master@{#394638}
==========

[commit-bot: I haz the power, 2016-05-19 02:19:51]

Patchset 1 (id:??) landed as
https://crrev.com/2142bcde69837585c1571a46cf59715a81d573f9
Cr-Commit-Position: refs/heads/master@{#394638}