cc: Fix recording of GPU rasterization histogram for renderer compositor.

cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/b54107d0f0360a1f9bd5e3664b90b3662da9e153
Cr-Commit-Position: refs/heads/master@{#364456}

[Khushal, 2015-12-10 00:25:47]

Description was changed from

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.
==========

to

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Khushal, 2015-12-10 00:26:42]

Description was changed from

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Khushal, 2015-12-10 00:27:21]

khushalsagar@chromium.org changed reviewers:
+ vmpstr@chromium.org

[vmpstr, 2015-12-10 00:35:00]

https://codereview.chromium.org/1511743004/diff/1/cc/trees/layer_tree_host.cc
File cc/trees/layer_tree_host.cc (right):

https://codereview.chromium.org/1511743004/diff/1/cc/trees/layer_tree_host.cc...
cc/trees/layer_tree_host.cc:741: if (gpu_rasterization_histogram_recorded_ ||
IsSingleThreaded())
It would be nice to add a test so this doesn't regress. Is it possible to write
a test that checks that gpu_rasterization_histogram_recorded_ doesn't become
true if this is run in single threaded mode?

[Khushal, 2015-12-10 01:59:41]

On 2015/12/10 00:35:00, vmpstr wrote:
> https://codereview.chromium.org/1511743004/diff/1/cc/trees/layer_tree_host.cc
> File cc/trees/layer_tree_host.cc (right):
> 
>
https://codereview.chromium.org/1511743004/diff/1/cc/trees/layer_tree_host.cc...
> cc/trees/layer_tree_host.cc:741: if (gpu_rasterization_histogram_recorded_ ||
> IsSingleThreaded())
> It would be nice to add a test so this doesn't regress. Is it possible to
write
> a test that checks that gpu_rasterization_histogram_recorded_ doesn't become
> true if this is run in single threaded mode?

Done.

[vmpstr, 2015-12-10 18:58:12]

lgtm, thanks for the test!

[Khushal, 2015-12-10 19:01:37]

On 2015/12/10 18:58:12, vmpstr wrote:
> lgtm, thanks for the test!

No problem. Sorry about the initial error. :(

[Khushal, 2015-12-10 19:01:42]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2015-12-10 19:04:32]

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

[commit-bot: I haz the power, 2015-12-10 20:21:02]

Description was changed from

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2015-12-10 20:21:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-10 20:22:56]

Description was changed from

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Fix recording of GPU rasterization histogram for renderer compositor.

The recording was disabled in https://codereview.chromium.org/1464313007/

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/b54107d0f0360a1f9bd5e3664b90b3662da9e153
Cr-Commit-Position: refs/heads/master@{#364456}
==========

[commit-bot: I haz the power, 2015-12-10 20:22:57]

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