Move BSP tree perf test to cc/output/.

Move BSP tree perf test to cc/output/.

It's weird to have a perf test for BSP tree stuff in the same file
as perf tests for CalcDrawProps.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/4e214038360be95e05245fabd2b53a9126b7f668
Cr-Commit-Position: refs/heads/master@{#417357}

[danakj, 2016-09-08 02:01:29]

Description was changed from

==========
Move BSP tree perf test to cc/output/.

It's weird to have a perf test for BSP tree stuff in the same file
as perf tests for CalcDrawProps.

R=enne
BUG=606056
==========

to

==========
Move BSP tree perf test to cc/output/.

It's weird to have a perf test for BSP tree stuff in the same file
as perf tests for CalcDrawProps.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-09-08 02:02:13]

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

[commit-bot: I haz the power, 2016-09-08 02:02:40]

Dry run: 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, 2016-09-08 03:56:01]

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

[commit-bot: I haz the power, 2016-09-08 03:56:01]

Dry run: This issue passed the CQ dry run.

[enne (OOO), 2016-09-08 17:26:06]

lgtm

And sure, this is fine to move, although I guess it still depends on trees/
things from cc/output? It is really more of an output test though, I agree.

[danakj, 2016-09-08 19:02:13]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-09-08 19:02:31]

On Thu, Sep 8, 2016 at 10:26 AM, enne@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com> wrote:

> lgtm
>
> And sure, this is fine to move, although I guess it still depends on trees/
> things from cc/output? It is really more of an output test though, I agree.
>

Ya, I think ideally this test just makes some quads or something, but I'm
ok with it depending on trees. I think to make it not we could hoist the
trees code up to cc/test/ but this works for now.


>
> https://codereview.chromium.org/2320883002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2016-09-08 19:03: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, 2016-09-08 19:08:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-08 19:11:20]

Description was changed from

==========
Move BSP tree perf test to cc/output/.

It's weird to have a perf test for BSP tree stuff in the same file
as perf tests for CalcDrawProps.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Move BSP tree perf test to cc/output/.

It's weird to have a perf test for BSP tree stuff in the same file
as perf tests for CalcDrawProps.

R=enne
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/4e214038360be95e05245fabd2b53a9126b7f668
Cr-Commit-Position: refs/heads/master@{#417357}
==========

[commit-bot: I haz the power, 2016-09-08 19:11:21]

Patchset 1 (id:??) landed as
https://crrev.com/4e214038360be95e05245fabd2b53a9126b7f668
Cr-Commit-Position: refs/heads/master@{#417357}