Set media_perftests up for android and to be built by the perfbots.

Set media_perftests up for android and to be built by the perfbots.

BUG=310837
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235552

[rileya (GONE FROM CHROMIUM), 2013-11-06 19:45:15]

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp
File build/all_android.gyp (right):

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp#newcode77
build/all_android.gyp:77: '../media/media.gyp:media_perftests_apk',
Not quite sure if this is the right way of going about this. The goal is to
build and run the media_perftests on the android perfbots, I've sort of been
following along with how cc_perftests is set up, but I'm not terribly sure of
how this all fits together.

[bulach, 2013-11-07 16:21:34]

thanks! drive-by fine by me, but ilevy needs to approve since it changes
buildbot loads (and yeah, we probably should update the comment below!)

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp
File build/all_android.gyp (right):

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp#newcode51
build/all_android.gyp:51: # on tryserver triggers and load.  Discuss with jrg
please.
ilevy?

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp#newcode77
build/all_android.gyp:77: '../media/media.gyp:media_perftests_apk',
On 2013/11/06 19:45:15, rileya wrote:
> Not quite sure if this is the right way of going about this. The goal is to
> build and run the media_perftests on the android perfbots, I've sort of been
> following along with how cc_perftests is set up, but I'm not terribly sure of
> how this all fits together.

what's media_perftests_apk? :)
if it's a gtest-based perf test, then yeah, cc_perftests is the closest
sibling..

[rileya (GONE FROM CHROMIUM), 2013-11-07 18:13:33]

+ilevy

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp
File build/all_android.gyp (right):

https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp#newcode77
build/all_android.gyp:77: '../media/media.gyp:media_perftests_apk',
On 2013/11/07 16:21:34, bulach wrote:
> On 2013/11/06 19:45:15, rileya wrote:
> > Not quite sure if this is the right way of going about this. The goal is to
> > build and run the media_perftests on the android perfbots, I've sort of been
> > following along with how cc_perftests is set up, but I'm not terribly sure
of
> > how this all fits together.
> 
> what's media_perftests_apk? :)
> if it's a gtest-based perf test, then yeah, cc_perftests is the closest
> sibling..

Yep, it's a gtest-based perf test.

[rileya (GONE FROM CHROMIUM), 2013-11-07 18:29:27]

On 2013/11/07 18:13:33, rileya wrote:
> +ilevy
> 
> https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp
> File build/all_android.gyp (right):
> 
>
https://codereview.chromium.org/62863002/diff/1/build/all_android.gyp#newcode77
> build/all_android.gyp:77: '../media/media.gyp:media_perftests_apk',
> On 2013/11/07 16:21:34, bulach wrote:
> > On 2013/11/06 19:45:15, rileya wrote:
> > > Not quite sure if this is the right way of going about this. The goal is
to
> > > build and run the media_perftests on the android perfbots, I've sort of
been
> > > following along with how cc_perftests is set up, but I'm not terribly sure
> of
> > > how this all fits together.
> > 
> > what's media_perftests_apk? :)
> > if it's a gtest-based perf test, then yeah, cc_perftests is the closest
> > sibling..
> 
> Yep, it's a gtest-based perf test.

BTW: I'm setting up an Android build right now to get some numbers on run-time.

[rileya (GONE FROM CHROMIUM), 2013-11-12 23:23:05]

On a 2013 Nexus 7, the tests take ~65s to run (80-85s counting test runner
setup/teardown). I tested on both a 4.3 and a 4.4 N7 and the numbers were
comparable.

A Nexus 4 did the tests in ~80s, (90-95s counting setup/teardown).

Note that these numbers are with some tweaks made in: 
https://codereview.chromium.org/70833002/ (currently in the CQ)

[rileya (GONE FROM CHROMIUM), 2013-11-12 23:27:58]

On 2013/11/12 23:23:05, rileya wrote:
> On a 2013 Nexus 7, the tests take ~65s to run (80-85s counting test runner
> setup/teardown). I tested on both a 4.3 and a 4.4 N7 and the numbers were
> comparable.
> 
> A Nexus 4 did the tests in ~80s, (90-95s counting setup/teardown).
> 
> Note that these numbers are with some tweaks made in: 
> https://codereview.chromium.org/70833002/ (currently in the CQ)

ilevy: Please take a took, and should jrg in the comment mentioned above be
replaced with you, or is there someone more appropriate?

[Isaac (away), 2013-11-13 21:57:05]

+ android committers with with cool usernames

[cjhopman, 2013-11-14 21:17:22]

lgtm

https://codereview.chromium.org/62863002/diff/80001/media/media.gyp
File media/media.gyp (right):

https://codereview.chromium.org/62863002/diff/80001/media/media.gyp#newcode1426
media/media.gyp:1426: 'input_shlib_path':
'<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)media_perftests<(SHARED_LIB_SUFFIX)',
input_shlib_path isn't actually needed (AFAICT).

[commit-bot: I haz the power, 2013-11-14 23:43:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/80001

[commit-bot: I haz the power, 2013-11-15 01:38:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/80001

[scherkus (not reviewing), 2013-11-15 02:17:53]

lgtm

[commit-bot: I haz the power, 2013-11-15 03:58:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/80001

[commit-bot: I haz the power, 2013-11-15 05:33:38]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-11-15 19:24:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/460001

[commit-bot: I haz the power, 2013-11-15 21:59:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/460001

[commit-bot: I haz the power, 2013-11-16 00:27:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rileya@chromium.org/62863002/460001

[commit-bot: I haz the power, 2013-11-17 21:20:27]

Change committed as 235552