Add a JNI boot test to catch ARM dynamic linker regressions.

Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229

Committed: https://crrev.com/613152af11d6e9a4d046af3c48a7be7642dfcc68
Cr-Commit-Position: refs/heads/master@{#15101}

[phoglund, 2016-11-15 12:27:50]

phoglund@webrtc.org changed reviewers:
+ perkj@webrtc.org

[perkj_webrtc, 2016-11-15 14:58:49]

perkj@webrtc.org changed reviewers:
+ magjed@webrtc.org

[perkj_webrtc, 2016-11-15 14:58:49]

I will leave this to magjed to decide and review.

But other pc tests are located here:
src/third_party/webrtc/api/androidtests/src/org/webrtc/

[phoglund, 2016-11-15 15:41:55]

On 2016/11/15 14:58:49, perkj_webrtc wrote:
> I will leave this to magjed to decide and review.
> 
> But other pc tests are located here:
> src/third_party/webrtc/api/androidtests/src/org/webrtc/

You are right, I could move this test to general peer connection tests. That's a
better fit
since this doesn't test AppRTC directly.

[phoglund, 2016-11-15 15:55:39]

PTAL

https://codereview.webrtc.org/2506603002/diff/20001/webrtc/api/androidtests/A...
File webrtc/api/androidtests/AndroidManifest.xml (right):

https://codereview.webrtc.org/2506603002/diff/20001/webrtc/api/androidtests/A...
webrtc/api/androidtests/AndroidManifest.xml:25:
android:label="AndroidPeerconnectionTests" >
Note: I got rid of the resources, because all ways of using resource files
directly in tests are deprecated downstream. They're unnecessary here anyway
(unless the linter complains...)

[phoglund, 2016-11-15 15:57:48]

Description was changed from

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

BUG=b/32820229
==========

to

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229
==========

[phoglund, 2016-11-15 15:57:57]

The CQ bit was checked by phoglund@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-15 15:58:06]

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

[commit-bot: I haz the power, 2016-11-15 16:16:54]

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

[commit-bot: I haz the power, 2016-11-15 16:16:54]

Dry run: This issue passed the CQ dry run.

[magjed_webrtc, 2016-11-16 09:27:47]

lgtm

[phoglund, 2016-11-16 09:29:41]

The CQ bit was checked by phoglund@webrtc.org

[commit-bot: I haz the power, 2016-11-16 09:29:44]

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

[commit-bot: I haz the power, 2016-11-16 09:31:22]

Description was changed from

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229
==========

to

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229
==========

[commit-bot: I haz the power, 2016-11-16 09:31:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-16 09:31:32]

Description was changed from

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229
==========

to

==========
Add a JNI boot test to catch ARM dynamic linker regressions.

The peer connection loopback test could catch regressions too, but it's
too slow to run on downstream ARM emulators. I'm adding a test here
that just makes sure we can load the JNI and init audio/video engines
in WebRTC.

This test overlaps in functionality with the existing tests,
but we need it anyway since all existing tests are too timing-sensitive.

Removes resources from the test; they're awkward downstream and we
don't really need them anyway.

BUG=b/32820229

Committed: https://crrev.com/613152af11d6e9a4d046af3c48a7be7642dfcc68
Cr-Commit-Position: refs/heads/master@{#15101}
==========

[commit-bot: I haz the power, 2016-11-16 09:31:33]

Patchset 2 (id:??) landed as
https://crrev.com/613152af11d6e9a4d046af3c48a7be7642dfcc68
Cr-Commit-Position: refs/heads/master@{#15101}