Deflake Android fullscreen tests.

Deflake Android fullscreen tests.

The flaky tests all dealt with manually hiding the top controls
based on user gesture.  The flakiness comes from the fact that
determining whether the top controls can be scrolled is controlled
in the renderer compositor and even if we were to monitor Tab.java's
latest value that it has sent, it does not take affect until some
time later.

As a result, this adds a new function that attempts to scroll the
top controls in a loop until it takes affect and then proceeds.

BUG=488393, 339668, 512299
Committed: https://crrev.com/326db8c916155a1c3f5655700731c6b4991857f0
Cr-Commit-Position: refs/heads/master@{#369880}

[Ted C, 2016-01-15 21:48:26]

tedchoc@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Ted C, 2016-01-15 21:48:26]

PTAL

[gone, 2016-01-15 22:53:57]

lgtm % offline comments about making a function for dealing with that
disableBrowserOverrides call on the UI thread

[Ted C, 2016-01-15 22:56:16]

Offline comment addressed

[Ted C, 2016-01-15 22:56:19]

The CQ bit was checked by tedchoc@chromium.org

[Ted C, 2016-01-15 22:56:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/1598433004/#ps20001
(title: "Make function to disable browser overrides")

[commit-bot: I haz the power, 2016-01-15 22:56:33]

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

[commit-bot: I haz the power, 2016-01-15 23:38:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-15 23:39:59]

Description was changed from

==========
Deflake Android fullscreen tests.

The flaky tests all dealt with manually hiding the top controls
based on user gesture.  The flakiness comes from the fact that
determining whether the top controls can be scrolled is controlled
in the renderer compositor and even if we were to monitor Tab.java's
latest value that it has sent, it does not take affect until some
time later.

As a result, this adds a new function that attempts to scroll the
top controls in a loop until it takes affect and then proceeds.

BUG=488393,339668,512299
==========

to

==========
Deflake Android fullscreen tests.

The flaky tests all dealt with manually hiding the top controls
based on user gesture.  The flakiness comes from the fact that
determining whether the top controls can be scrolled is controlled
in the renderer compositor and even if we were to monitor Tab.java's
latest value that it has sent, it does not take affect until some
time later.

As a result, this adds a new function that attempts to scroll the
top controls in a loop until it takes affect and then proceeds.

BUG=488393,339668,512299

Committed: https://crrev.com/326db8c916155a1c3f5655700731c6b4991857f0
Cr-Commit-Position: refs/heads/master@{#369880}
==========

[commit-bot: I haz the power, 2016-01-15 23:40:00]

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