Adding Android to the test OSes for scrolling while zoomed in.

Adding Android to the test OSes for scrolling while zoomed in.

Depend on https://codereview.chromium.org/999243003/.

BUG=466867
Committed: https://crrev.com/a4f783321e7f49a6e39d47470f002c65925569b6
Cr-Commit-Position: refs/heads/master@{#326913}

[cblume, 2015-04-06 22:21:47]

cblume@chromium.org changed reviewers:
+ nednguyen@google.com

[cblume, 2015-04-06 22:21:47]

This CL was created because of a comment on
https://codereview.chromium.org/999243003/

We were going to start by adding the tests to ChromeOS and later, in this CL,
add Android to the list of platforms.

[nednguyen, 2015-04-07 19:24:47]

On 2015/04/06 22:21:47, cblume wrote:
> This CL was created because of a comment on
> https://codereview.chromium.org/999243003/
> 
> We were going to start by adding the tests to ChromeOS and later, in this CL,
> add Android to the list of platforms.

Please ping me again when the other patch is landed & there is no redness on
cros's waterfall.

[cblume, 2015-04-23 18:23:21]

ChromeOS's water fall has been green for a while while this test has been in
place. http://build.chromium.org/p/chromium/waterfall

I believe it is okay to add Android to the list of test targets and close out
this bug.

[nednguyen, 2015-04-23 18:29:09]

On 2015/04/23 18:23:21, cblume wrote:
> ChromeOS's water fall has been green for a while while this test has been in
> place. http://build.chromium.org/p/chromium/waterfall
> 
> I believe it is okay to add Android to the list of test targets and close out
> this bug.

Please run a perf-tryjob on all android bots and make sure the benchmark run
properly on those. Note that there may be redness that's unrelated to your patch
and it's ok.

[cblume, 2015-04-24 21:15:17]

On 2015/04/23 18:29:09, nednguyen wrote:
> On 2015/04/23 18:23:21, cblume wrote:
> > ChromeOS's water fall has been green for a while while this test has been in
> > place. http://build.chromium.org/p/chromium/waterfall
> > 
> > I believe it is okay to add Android to the list of test targets and close
out
> > this bug.
> 
> Please run a perf-tryjob on all android bots and make sure the benchmark run
> properly on those. Note that there may be redness that's unrelated to your
patch
> and it's ok.

I ran on all Android: https://codereview.chromium.org/1107653002
This resulted in some greens and a few unrelated reds.
I reran the non-greens:
Nexus 7: https://codereview.chromium.org/1080003007
Android One: https://codereview.chromium.org/1095043004

Nexus 9 stayed purple on a second run:
https://codereview.chromium.org/1101313002

[nednguyen, 2015-04-24 21:22:24]

On 2015/04/24 21:15:17, cblume wrote:
> On 2015/04/23 18:29:09, nednguyen wrote:
> > On 2015/04/23 18:23:21, cblume wrote:
> > > ChromeOS's water fall has been green for a while while this test has been
in
> > > place. http://build.chromium.org/p/chromium/waterfall
> > > 
> > > I believe it is okay to add Android to the list of test targets and close
> out
> > > this bug.
> > 
> > Please run a perf-tryjob on all android bots and make sure the benchmark run
> > properly on those. Note that there may be redness that's unrelated to your
> patch
> > and it's ok.
> 
> I ran on all Android: https://codereview.chromium.org/1107653002
> This resulted in some greens and a few unrelated reds.
> I reran the non-greens:
> Nexus 7: https://codereview.chromium.org/1080003007
> Android One: https://codereview.chromium.org/1095043004
> 
> Nexus 9 stayed purple on a second run:
> https://codereview.chromium.org/1101313002

lgtm

Thanks for putting extra efforts to rerun the non-green, Chris. I usually just
verify that there is at least 1 green and check the log to make sure that the
failures on others is not related to my patch :P

[cblume, 2015-04-24 21:26:12]

The CQ bit was checked by cblume@chromium.org

[commit-bot: I haz the power, 2015-04-24 21:27:09]

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

[commit-bot: I haz the power, 2015-04-24 22:36:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-24 22:37:19]

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