Removing unused resources from chrome/java/res

Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
Committed: https://crrev.com/9e4be9ba438612c55668fc95a55000e4a4cc0869
Cr-Commit-Position: refs/heads/master@{#410186}

[estevenson, 2016-08-03 20:54:01]

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

[commit-bot: I haz the power, 2016-08-03 20:54:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 21:48:05]

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

[commit-bot: I haz the power, 2016-08-03 21:48:06]

Dry run: This issue passed the CQ dry run.

[estevenson, 2016-08-04 14:50:00]

Description was changed from

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
==========

to

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
==========

[estevenson, 2016-08-04 19:37:34]

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

[agrieve, 2016-08-05 03:27:41]

lgtm

https://codereview.chromium.org/2207103003/diff/40001/build/android/lint/supp...
File build/android/lint/suppressions.xml (right):

https://codereview.chromium.org/2207103003/diff/40001/build/android/lint/supp...
build/android/lint/suppressions.xml:112: <ignore
path="chrome/android/java/res/xml/network_security_config.xml" />
nit: add a comment as to why we ignore it.

[agrieve, 2016-08-05 03:28:13]

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

[agrieve, 2016-08-05 03:28:13]

+tedchoc

[estevenson, 2016-08-05 13:59:44]

https://codereview.chromium.org/2207103003/diff/40001/build/android/lint/supp...
File build/android/lint/suppressions.xml (right):

https://codereview.chromium.org/2207103003/diff/40001/build/android/lint/supp...
build/android/lint/suppressions.xml:112: <ignore
path="chrome/android/java/res/xml/network_security_config.xml" />
On 2016/08/05 03:27:41, agrieve wrote:
> nit: add a comment as to why we ignore it.

Done.

[Ted C, 2016-08-05 20:48:33]

lgtm

https://codereview.chromium.org/2207103003/diff/60001/build/android/lint/supp...
File build/android/lint/suppressions.xml (right):

https://codereview.chromium.org/2207103003/diff/60001/build/android/lint/supp...
build/android/lint/suppressions.xml:112: <!-- This file isn't used if the target
SDK version is less than 24 -->
maybe add a TODO to remove this suppression once we roll to SDK 24?  We'll be
able to remove the conditional inclusion in AndroidManifest.xml at that point as
well.

[estevenson, 2016-08-05 21:21:45]

https://codereview.chromium.org/2207103003/diff/60001/build/android/lint/supp...
File build/android/lint/suppressions.xml (right):

https://codereview.chromium.org/2207103003/diff/60001/build/android/lint/supp...
build/android/lint/suppressions.xml:112: <!-- This file isn't used if the target
SDK version is less than 24 -->
On 2016/08/05 20:48:33, Ted C wrote:
> maybe add a TODO to remove this suppression once we roll to SDK 24?  We'll be
> able to remove the conditional inclusion in AndroidManifest.xml at that point
as
> well.

Done.

[estevenson, 2016-08-05 21:25:05]

The CQ bit was checked by estevenson@chromium.org

[estevenson, 2016-08-05 21:25:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
agrieve@chromium.org, tedchoc@chromium.org
Link to the patchset: https://codereview.chromium.org/2207103003/#ps80001
(title: "Added TODO")

[commit-bot: I haz the power, 2016-08-05 21:25:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-05 22:16:27]

Description was changed from

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
==========

to

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
==========

[commit-bot: I haz the power, 2016-08-05 22:16:27]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-05 22:18:25]

Description was changed from

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773
==========

to

==========
Removing unused resources from chrome/java/res

This will reduce the number of lint warnings generated during builds.

BUG=621773

Committed: https://crrev.com/9e4be9ba438612c55668fc95a55000e4a4cc0869
Cr-Commit-Position: refs/heads/master@{#410186}
==========

[commit-bot: I haz the power, 2016-08-05 22:18:27]

Patchset 5 (id:??) landed as
https://crrev.com/9e4be9ba438612c55668fc95a55000e4a4cc0869
Cr-Commit-Position: refs/heads/master@{#410186}