cc: Add more seriliazation perf test cases

cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that. Added modifier in PrintResult to reflect
this option.

Also changed and renamed two previous test that are repeated and
incorrectly named.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

[weiliangc, 2017-01-25 20:53:50]

Description was changed from

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that.

BUG=
R=fsamuel
==========

to

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Fady Samuel, 2017-01-25 20:57:54]

fsamuel@chromium.org changed reviewers:
+ fsamuel@chromium.org

[Fady Samuel, 2017-01-25 20:57:54]

Could you share some numbers in here? Thanks!

[Fady Samuel, 2017-01-25 20:57:55]

Could you share some numbers in here? Thanks!

[weiliangc, 2017-01-25 21:03:42]

On 2017/01/25 at 20:57:55, fsamuel wrote:
> Could you share some numbers in here? Thanks!

I only have some runs on my linux box, would that be useful? Not sure what would
you like to use the numbers here for.

In general SQS per quad v.s. only one SQS on RP makes the run ~50% to ~100%
slower.

[weiliangc, 2017-01-25 21:13:23]

Description was changed from

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that. Added modifier in PrintResult to reflect
this option.

Also changed and renamed two previous test that are repeated and
incorrectly named.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[weiliangc, 2017-01-25 21:13:45]

Also added comment to indicate the two renamed tests

https://codereview.chromium.org/2657873003/diff/1/cc/ipc/cc_serialization_per...
File cc/ipc/cc_serialization_perftest.cc (right):

https://codereview.chromium.org/2657873003/diff/1/cc/ipc/cc_serialization_per...
cc/ipc/cc_serialization_perftest.cc:276: TEST_F(CCSerializationPerfTest,
DelegatedFrame_ManyQuads_100_4000) {
Previously incorrectly names, also was a repeat of ManyQuads_1_4000.

Changed to run 100_4000 and renamed to reflect that.

https://codereview.chromium.org/2657873003/diff/1/cc/ipc/cc_serialization_per...
cc/ipc/cc_serialization_perftest.cc:281: TEST_F(CCSerializationPerfTest,
DelegatedFrame_ManyQuads_10_100000) {
Previously incorrectly names, also was a repeat of ManyQuads_1_100000.

Changed to run 10_100000 and renamed to reflect that.

[Fady Samuel, 2017-01-25 21:15:53]

LGTM + request for comment. Then pass this along to danakj@ I guess. Thanks!

https://codereview.chromium.org/2657873003/diff/1/cc/ipc/cc_serialization_per...
File cc/ipc/cc_serialization_perftest.cc (left):

https://codereview.chromium.org/2657873003/diff/1/cc/ipc/cc_serialization_per...
cc/ipc/cc_serialization_perftest.cc:228: TEST_F(CCSerializationPerfTest,
DelegatedFrame_ManyQuads_1_4000) {
Maybe add a comment above to explain the test nomenclature? _1_4000. What's 1,
what's 4000?

[weiliangc, 2017-01-25 22:14:31]

weiliangc@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2017-01-31 15:40:05]

LGTM

[weiliangc, 2017-01-31 17:36:48]

The CQ bit was checked by weiliangc@chromium.org

[weiliangc, 2017-01-31 17:36:48]

The patchset sent to the CQ was uploaded after l-g-t-m from fsamuel@chromium.org
Link to the patchset: https://codereview.chromium.org/2657873003/#ps20001
(title: "add comments according to review comments")

[commit-bot: I haz the power, 2017-01-31 17:37:23]

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-31 17:51:41]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485884208332960,
"parent_rev": "b88b2c6f367bf0d9638eb005cde9888195c977cf", "commit_rev":
"a02090236d8c8ad6cafcb568bcfeb910fde09d9c"}

[commit-bot: I haz the power, 2017-01-31 17:53:22]

Description was changed from

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that. Added modifier in PrintResult to reflect
this option.

Also changed and renamed two previous test that are repeated and
incorrectly named.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Add more seriliazation perf test cases

Number of SharedQuadState would affect serialization test results too.
Add more test cases for that. Added modifier in PrintResult to reflect
this option.

Also changed and renamed two previous test that are repeated and
incorrectly named.

BUG=
R=fsamuel
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

[commit-bot: I haz the power, 2017-01-31 17:53:23]

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