media: Fix build config for audio_unittests on Android

media: Fix build config for audio_unittests on Android

We were missing some deps which meant this test didn't run on Android.
We didn't notice because we usually run these tests through
media_unittests.

BUG=691054
Review-Url: https://codereview.chromium.org/2694323002
Cr-Commit-Position: refs/heads/master@{#450517}
Committed: https://chromium.googlesource.com/chromium/src/+/f7ac891577ddd5fddc4846eba0496f6d8dddab6c

[watk, 2017-02-14 20:38:47]

watk@chromium.org changed reviewers:
+ sandersd@chromium.org

[watk, 2017-02-14 20:41:56]

A reasonable question might be: why do we need these?

media's run_all_unittests initializes the jni bindings for the Java classes we
depend on, so it needs these. I'm sure we could somehow refactor things to only
initialize the bindings when we need them.

[sandersd (OOO until July 31), 2017-02-14 21:37:47]

lgtm, but I'm assuming you plan to upload another PS with updated comments.

[watk, 2017-02-14 21:43:08]

On 2017/02/14 21:37:47, sandersd wrote:
> lgtm, but I'm assuming you plan to upload another PS with updated comments.

I didn't realize that I'd already copied the comment from media_unittests. Did
you want another comment?

[sandersd (OOO until July 31), 2017-02-14 21:45:38]

On 2017/02/14 21:43:08, watk wrote:
> On 2017/02/14 21:37:47, sandersd wrote:
> > lgtm, but I'm assuming you plan to upload another PS with updated comments.
> 
> I didn't realize that I'd already copied the comment from media_unittests. Did
> you want another comment?

No, I don't need one, but I thought you were adding one to the
//media/base/android target.

[watk, 2017-02-14 21:50:10]

On 2017/02/14 21:45:38, sandersd wrote:
> On 2017/02/14 21:43:08, watk wrote:
> > On 2017/02/14 21:37:47, sandersd wrote:
> > > lgtm, but I'm assuming you plan to upload another PS with updated
comments.
> > 
> > I didn't realize that I'd already copied the comment from media_unittests.
Did
> > you want another comment?
> 
> No, I don't need one, but I thought you were adding one to the
> //media/base/android target.

Sure, may as well.

[watk, 2017-02-14 21:53:03]

The CQ bit was checked by watk@chromium.org

[watk, 2017-02-14 21:53:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
sandersd@chromium.org
Link to the patchset: https://codereview.chromium.org/2694323002/#ps20001
(title: "more commenting")

[sandersd (OOO until July 31), 2017-02-14 21:54:00]

lgtm

[commit-bot: I haz the power, 2017-02-14 21:54:02]

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, 2017-02-14 23:54:50]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487109183073790,
"parent_rev": "ee699e9ecfbaf7441d5ad5d71d2e7f5db15afde7", "commit_rev":
"f7ac891577ddd5fddc4846eba0496f6d8dddab6c"}

[commit-bot: I haz the power, 2017-02-14 23:55:28]

Description was changed from

==========
media: Fix build config for audio_unittests on Android

We were missing some deps which meant this test didn't run on Android.
We didn't notice because we usually run these tests through
media_unittests.

BUG=691054
==========

to

==========
media: Fix build config for audio_unittests on Android

We were missing some deps which meant this test didn't run on Android.
We didn't notice because we usually run these tests through
media_unittests.

BUG=691054

Review-Url: https://codereview.chromium.org/2694323002
Cr-Commit-Position: refs/heads/master@{#450517}
Committed:
https://chromium.googlesource.com/chromium/src/+/f7ac891577ddd5fddc4846eba049...
==========

[commit-bot: I haz the power, 2017-02-14 23:55:29]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/f7ac891577ddd5fddc4846eba049...