(Reland) [Android] Get package_info by using apk_under_test package.

(Reland) [Android] Get package_info by using apk_under_test package.

Currently, the package_info used by the test apk is looked up by
by looking at the test_apk package in the list of package_infos.
Instead, we can just look at the package entries in the list
of package_infos and compare it to the package of the apk_under_test.

This makes things much nicer, when for instance you have multiple
test apks targeting the same package.

BUG=

Committed: https://crrev.com/9454d5d5b562cce1186b398d6afe9cf3b42c08a1
Cr-Commit-Position: refs/heads/master@{#388042}

[mikecase (-- gone --), 2016-04-18 20:25:47]

mikecase@chromium.org changed reviewers:
+ agrieve@chromium.org, rnephew@chromium.org

[mikecase (-- gone --), 2016-04-18 20:25:48]

Got the error...
AttributeError: 'NoneType' object has no attribute 'GetPackageName'
...when running Mojo tests. I didn't realize instrumentation tests didn't need
an apk under test.

Relanding with a simple fix to check that self._apk_under_test is not none
before looking for its package. This will not change the current behavior for
Mojo tests. In the current Mojo tests logs, there is....

W    0.028s Main  Unable to find package info for org.chromium.mojo.tests

[agrieve, 2016-04-18 20:34:14]

On 2016/04/18 20:25:48, mikecase wrote:
> Got the error...
> AttributeError: 'NoneType' object has no attribute 'GetPackageName'
> ...when running Mojo tests. I didn't realize instrumentation tests didn't need
> an apk under test.
> 
> Relanding with a simple fix to check that self._apk_under_test is not none
> before looking for its package. This will not change the current behavior for
> Mojo tests. In the current Mojo tests logs, there is....
> 
> W    0.028s Main  Unable to find package info for org.chromium.mojo.tests

lgtm

[mikecase (-- gone --), 2016-04-18 20:34:48]

The CQ bit was checked by mikecase@chromium.org

[commit-bot: I haz the power, 2016-04-18 20:35:37]

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

[commit-bot: I haz the power, 2016-04-18 22:17:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-18 22:20:27]

Description was changed from

==========
(Reland) [Android] Get package_info by using apk_under_test package.

Currently, the package_info used by the test apk is looked up by
by looking at the test_apk package in the list of package_infos.
Instead, we can just look at the package entries in the list
of package_infos and compare it to the package of the apk_under_test.

This makes things much nicer, when for instance you have multiple
test apks targeting the same package.

BUG=
==========

to

==========
(Reland) [Android] Get package_info by using apk_under_test package.

Currently, the package_info used by the test apk is looked up by
by looking at the test_apk package in the list of package_infos.
Instead, we can just look at the package entries in the list
of package_infos and compare it to the package of the apk_under_test.

This makes things much nicer, when for instance you have multiple
test apks targeting the same package.

BUG=

Committed: https://crrev.com/9454d5d5b562cce1186b398d6afe9cf3b42c08a1
Cr-Commit-Position: refs/heads/master@{#388042}
==========

[commit-bot: I haz the power, 2016-04-18 22:20:28]

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