🍮 GN: Allow android lint to fail builds

GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none

Committed: https://crrev.com/6c2b9752863847c6d1c4db39e5531fe335349afe
Cr-Commit-Position: refs/heads/master@{#384290}

[agrieve, 2016-03-31 03:04:03]

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

[commit-bot: I haz the power, 2016-03-31 03:04:40]

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

[commit-bot: I haz the power, 2016-03-31 03:45:27]

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

[commit-bot: I haz the power, 2016-03-31 03:45:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[agrieve, 2016-03-31 13:54:20]

Description was changed from

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none
==========

to

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none
==========

[agrieve, 2016-03-31 13:54:20]

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

[agrieve, 2016-03-31 13:54:40]

On 2016/03/31 03:45:28, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yaron, 2016-03-31 13:55:33]

lgtm

[agrieve, 2016-03-31 15:34:24]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-03-31 15:34:48]

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

[commit-bot: I haz the power, 2016-03-31 15:52:12]

Description was changed from

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none
==========

to

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none
==========

[commit-bot: I haz the power, 2016-03-31 15:52:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-31 15:54:42]

Description was changed from

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none
==========

to

==========
GN: Allow android lint to fail builds

This was enabled for GYP in June, but got missed for GN

https://codereview.chromium.org/1190103002

BUG=none

Committed: https://crrev.com/6c2b9752863847c6d1c4db39e5531fe335349afe
Cr-Commit-Position: refs/heads/master@{#384290}
==========

[commit-bot: I haz the power, 2016-03-31 15:54:42]

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

[agrieve, 2016-03-31 17:29:57]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1848793003/ by agrieve@chromium.org.

The reason for reverting is: Some warnings actually do exist, but bots didn't
catch it because lint wasn't being re-run..