CSS Transitions: Fix flaky test extra-transition.html

CSS Transitions: Fix flaky test extra-transition.html

We reduce the duration of the test to avoid timeouts.

The test has also been converted to use testharness.

BUG=248938
Committed: https://crrev.com/9fd376e988691e1b08a9a0302237d22ef7098809
Cr-Commit-Position: refs/heads/master@{#441330}

[Eric Willigers, 2017-01-04 03:37:56]

The CQ bit was checked by ericwilligers@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 03:38:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Eric Willigers, 2017-01-04 04:34:12]

ericwilligers@chromium.org changed reviewers:
+ alancutter@chromium.org

[commit-bot: I haz the power, 2017-01-04 04:55:03]

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

[commit-bot: I haz the power, 2017-01-04 04:55:04]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2017-01-04 05:33:04]

lgtm with nits.

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/transitions/extra-transition.html (right):

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:33: <ul
id="features">
features element unnecessary.

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:34: <li
id="item">
No need for this to be a <li>.

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:47:
document.body.offsetHeight;
Add comment explaining this line.

[Eric Willigers, 2017-01-04 06:13:12]

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/transitions/extra-transition.html (right):

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:33: <ul
id="features">
On 2017/01/04 05:33:04, alancutter wrote:
> features element unnecessary.

Done.

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:34: <li
id="item">
On 2017/01/04 05:33:04, alancutter wrote:
> No need for this to be a <li>.

Acknowledged.

https://codereview.chromium.org/2613663002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/transitions/extra-transition.html:47:
document.body.offsetHeight;
On 2017/01/04 05:33:04, alancutter wrote:
> Add comment explaining this line.

Done.

[Eric Willigers, 2017-01-04 06:14:25]

The CQ bit was checked by ericwilligers@chromium.org

[Eric Willigers, 2017-01-04 06:14:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2613663002/#ps20001
(title: "comment")

[commit-bot: I haz the power, 2017-01-04 06:14:34]

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-01-04 07:35:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483510465036180,
"parent_rev": "959e26c5ca3300147c1ad1c78de079e8b7e37994", "commit_rev":
"a6488cfa5d193fcd59ee8d2c7848025a76f648b9"}

[commit-bot: I haz the power, 2017-01-04 07:35:48]

Description was changed from

==========
CSS Transitions: Fix flaky test extra-transition.html

We reduce the duration of the test to avoid timeouts.

The test has also been converted to use testharness.

BUG=248938
==========

to

==========
CSS Transitions: Fix flaky test extra-transition.html

We reduce the duration of the test to avoid timeouts.

The test has also been converted to use testharness.

BUG=248938

Review-Url: https://codereview.chromium.org/2613663002
==========

[commit-bot: I haz the power, 2017-01-04 07:35:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 07:38:30]

Description was changed from

==========
CSS Transitions: Fix flaky test extra-transition.html

We reduce the duration of the test to avoid timeouts.

The test has also been converted to use testharness.

BUG=248938

Review-Url: https://codereview.chromium.org/2613663002
==========

to

==========
CSS Transitions: Fix flaky test extra-transition.html

We reduce the duration of the test to avoid timeouts.

The test has also been converted to use testharness.

BUG=248938

Committed: https://crrev.com/9fd376e988691e1b08a9a0302237d22ef7098809
Cr-Commit-Position: refs/heads/master@{#441330}
==========

[commit-bot: I haz the power, 2017-01-04 07:38:31]

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