Expose <meta-data> in apk_helper.py

Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: chromium:762024
Review-Url: https://chromiumcodereview.appspot.com/3014693002
Committed: https://chromium.googlesource.com/catapult/+/946d4188c628f9d0747c54309b4298ec2dad25f3

[agrieve, 2017-09-28 00:30:13]

Description was changed from

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: 762024
==========

to

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: 762024
==========

[agrieve, 2017-09-28 00:30:13]

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

[agrieve, 2017-09-28 00:30:25]

On 2017/09/28 00:30:13, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:jbudorick@chromium.org

🍞

[agrieve, 2017-09-28 01:10:17]

agrieve@chromium.org changed reviewers:
+ perezju@chromium.org
- jbudorick@chromium.org

[agrieve, 2017-09-28 01:10:42]

On 2017/09/28 00:30:25, agrieve wrote:
> On 2017/09/28 00:30:13, agrieve wrote:
> > mailto:agrieve@chromium.org changed reviewers:
> > + mailto:jbudorick@chromium.org
> 
> 🍞

perezju - looks like jbudorick's OOO, could you please review?

[perezju, 2017-09-28 10:24:38]

lgtm w/question

Also, probably this needs to be uploaded via gerrit now?
https://groups.google.com/a/chromium.org/forum/?utm_medium=email&utm_source=f...

https://codereview.chromium.org/3014693002/diff/1/devil/devil/android/apk_hel...
File devil/devil/android/apk_helper.py (right):

https://codereview.chromium.org/3014693002/diff/1/devil/devil/android/apk_hel...
devil/devil/android/apk_helper.py:192: services = application['service']
what's the reason for this change from all applications to just the first?

[agrieve, 2017-09-28 11:54:34]

Will re-upload with gerrif if this fails. Just did a "git cl upload" and this is
what it chose, but perhaps I wasn't synced to master...

https://codereview.chromium.org/3014693002/diff/1/devil/devil/android/apk_hel...
File devil/devil/android/apk_helper.py (right):

https://codereview.chromium.org/3014693002/diff/1/devil/devil/android/apk_hel...
devil/devil/android/apk_helper.py:192: services = application['service']
On 2017/09/28 10:24:38, perezju wrote:
> what's the reason for this change from all applications to just the first?

Just wanted to make it consistent with how GetAllMetadata() works & to simplify.
It's not valid to have multiple application tags, so there's no reason for the
complexity.

[agrieve, 2017-09-28 11:54:42]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-09-28 11:54:48]

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-09-28 11:57:57]

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

[commit-bot: I haz the power, 2017-09-28 11:57:58]

Try jobs failed on following builders:
  Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)

[agrieve, 2017-09-28 11:58:16]

Description was changed from

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: 762024
==========

to

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: chromium:762024
==========

[agrieve, 2017-09-28 11:58:36]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-09-28 11:58:38]

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-09-28 12:26:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1506599916152470, "parent_rev":
"591738408846bcf010452d5fbbc2c229efaa1dcf", "commit_rev":
"946d4188c628f9d0747c54309b4298ec2dad25f3"}

[commit-bot: I haz the power, 2017-09-28 12:26:53]

Description was changed from

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: chromium:762024
==========

to

==========
Expose <meta-data> in apk_helper.py

Need to use this in test runner to inspect incremental install <meta-data>.

Bug: chromium:762024
Review-Url: https://chromiumcodereview.appspot.com/3014693002
Committed:
https://chromium.googlesource.com/catapult/+/946d4188c628f9d0747c54309b4298ec...
==========

[commit-bot: I haz the power, 2017-09-28 12:26:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/catapult/+/946d4188c628f9d0747c54309b4298ec...