[Android] Suppress findbugs stderr output.

[Android] Suppress findbugs stderr output.

BUG=514342
Committed: https://crrev.com/5194d46039a269294d3e5bf7a1c827303521d79d
Cr-Commit-Position: refs/heads/master@{#344992}

[jbudorick, 2015-07-28 21:40:33]

jbudorick@chromium.org changed reviewers:
+ cjhopman@chromium.org

[jbudorick, 2015-08-04 15:24:08]

ping

[cjhopman, 2015-08-04 23:09:28]

It's always the case that findbugs stderr output isn't helpful?

[jbudorick, 2015-08-04 23:16:02]

On 2015/08/04 at 23:09:28, cjhopman wrote:
> It's always the case that findbugs stderr output isn't helpful?

It can be if we're missing something we care about. I haven't run into that
situation since I was redoing findbugs, though.

I included the findbugs_verbose escape hatch in case someone does care about it,
which I have set locally.

[jbudorick, 2015-08-17 14:24:47]

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

[jbudorick, 2015-08-17 14:24:48]

+dpranke since cjhopman is gone

[Dirk Pranke, 2015-08-17 19:25:02]

lgtm, though I'm no findbugs expert.

https://codereview.chromium.org/1258303003/diff/20001/build/config/android/ru...
File build/config/android/rules.gni (right):

https://codereview.chromium.org/1258303003/diff/20001/build/config/android/ru...
build/config/android/rules.gni:1597: }
this can be replaced with forward_variables_from(invoker, ["run_findbugs",
"deps"])
now, but I probably would do that change in a separate patch that cleaned up
other
uses of this idiom at the same time.

[jbudorick, 2015-08-22 22:45:18]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2015-08-22 22:45:19]

https://codereview.chromium.org/1258303003/diff/20001/build/config/android/ru...
File build/config/android/rules.gni (right):

https://codereview.chromium.org/1258303003/diff/20001/build/config/android/ru...
build/config/android/rules.gni:1597: }
On 2015/08/17 at 19:25:02, Dirk Pranke wrote:
> this can be replaced with forward_variables_from(invoker, ["run_findbugs",
"deps"])
> now, but I probably would do that change in a separate patch that cleaned up
other
> uses of this idiom at the same time.

ack, I'll send you a follow-up CL. agrieve partially did this in
https://codereview.chromium.org/1310613002

[jbudorick, 2015-08-22 22:45:19]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1258303003/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2015-08-22 22:45:25]

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

[commit-bot: I haz the power, 2015-08-22 23:33:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-08-22 23:34:34]

Patchset 3 (id:??) landed as
https://crrev.com/5194d46039a269294d3e5bf7a1c827303521d79d
Cr-Commit-Position: refs/heads/master@{#344992}