Convert layout test move-up-down-should-skip-hidden-elements.html to performance test

Convert layout test move-up-down-should-skip-hidden-elements.html to performance test

This patch converts layout test "move-up-down-should-skip-hidden-elements.html"
to performance tests, move line up and move line down, since this layout test
measures verifies execution speed regression. Using performance test is easier
to track performance regression than detecting timeout in layout test.

BUG=691841
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2689183003
Cr-Commit-Position: refs/heads/master@{#450278}
Committed: https://chromium.googlesource.com/chromium/src/+/c8a73d769c8090eb22f25cea950303a50ba98788

[yosin_UTC9, 2017-02-14 02:24:34]

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

[commit-bot: I haz the power, 2017-02-14 02:25:51]

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

[yosin_UTC9, 2017-02-14 02:27:50]

Description was changed from

==========
2017-02-14T11:22:37

2017-02-14T11:22:33

BUG=
==========

to

==========
Convert layout test move-up-down-should-skip-hidden-elements.html to performance
test

This patch converts layout test "move-up-down-should-skip-hidden-elements.html"
to performance tests, move line up and move line down, since this layout test
measures verifies execution speed regression. Using performance test is easier
to track performance regression than detecting timeout in layout test.

BUG=691841
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2017-02-14 02:28:16]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org, xiaochengh@chromium.org, yoichio@chromium.org

[yosin_UTC9, 2017-02-14 02:28:17]

PTAL

[commit-bot: I haz the power, 2017-02-14 03:45:10]

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

[commit-bot: I haz the power, 2017-02-14 03:45:11]

Dry run: This issue passed the CQ dry run.

[yoichio, 2017-02-14 04:00:20]

https://codereview.chromium.org/2689183003/diff/1/third_party/WebKit/Performa...
File
third_party/WebKit/PerformanceTests/Editing/move-down-with-hidden-elements.html
(right):

https://codereview.chromium.org/2689183003/diff/1/third_party/WebKit/Performa...
third_party/WebKit/PerformanceTests/Editing/move-down-with-hidden-elements.html:9:
for (let count = 0; count < 100; ++count)
Amount of 100 <meta>s is O.K. to get better metrics than 1000 of original?

[yosin_UTC9, 2017-02-14 04:10:47]

On 2017/02/14 at 04:00:20, yoichio wrote:
>
https://codereview.chromium.org/2689183003/diff/1/third_party/WebKit/Performa...
> File
third_party/WebKit/PerformanceTests/Editing/move-down-with-hidden-elements.html
(right):
> 
>
https://codereview.chromium.org/2689183003/diff/1/third_party/WebKit/Performa...
>
third_party/WebKit/PerformanceTests/Editing/move-down-with-hidden-elements.html:9:
for (let count = 0; count < 100; ++count)
> Amount of 100 <meta>s is O.K. to get better metrics than 1000 of original?

100:
avg 5846.717981020527 runs/s
median 4699.520361810619 runs/s
stdev 2184.440248591381 runs/s
min 4045.9169428040273 runs/s
max 12313.686349725533 runs/s

1000:
avg 4936.510072120326 runs/s
median 4136.322332609161 runs/s
stdev 1402.5910205950515 runs/s
min 4074.2013928675997 runs/s
max 8884.457877699893 runs/s

10000:
avg 2039.293546264967 runs/s
median 1910.5301778091807 runs/s
stdev 211.22840629262518 runs/s
min 1899.3101810939452 runs/s
max 2564.956284759471 runs/s

It seems 100 and 1000 are unstable. I change to 10,000.

[yosin_UTC9, 2017-02-14 04:14:07]

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

[commit-bot: I haz the power, 2017-02-14 04:15:04]

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

[yosin_UTC9, 2017-02-14 04:15:04]

PTAL

Change to traversal 10,000 to get significant difference on number of unrendered
elements.

[yoichio, 2017-02-14 04:18:49]

lgtm

[yosin_UTC9, 2017-02-14 04:20:07]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-02-14 04:20:10]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-02-14 04:21:10]

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-02-14 06:18:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487046010227250,
"parent_rev": "977fe146bd6777af01d8de34cbaab350b53d62f3", "commit_rev":
"c8a73d769c8090eb22f25cea950303a50ba98788"}

[commit-bot: I haz the power, 2017-02-14 06:19:13]

Description was changed from

==========
Convert layout test move-up-down-should-skip-hidden-elements.html to performance
test

This patch converts layout test "move-up-down-should-skip-hidden-elements.html"
to performance tests, move line up and move line down, since this layout test
measures verifies execution speed regression. Using performance test is easier
to track performance regression than detecting timeout in layout test.

BUG=691841
TEST=n/a; no behavior changes
==========

to

==========
Convert layout test move-up-down-should-skip-hidden-elements.html to performance
test

This patch converts layout test "move-up-down-should-skip-hidden-elements.html"
to performance tests, move line up and move line down, since this layout test
measures verifies execution speed regression. Using performance test is easier
to track performance regression than detecting timeout in layout test.

BUG=691841
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2689183003
Cr-Commit-Position: refs/heads/master@{#450278}
Committed:
https://chromium.googlesource.com/chromium/src/+/c8a73d769c8090eb22f25cea9503...
==========

[commit-bot: I haz the power, 2017-02-14 06:19:14]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c8a73d769c8090eb22f25cea9503...