Remove unused variables, cc/ edition.

Remove unused variables, cc/ edition.

These were exposed while trying to make various geometry classes constexpr.

BUG=none
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[Peter Kasting, 2016-06-10 21:20:18]

Description was changed from

==========
Remove unused variables, cc/ edition.

These were exposed while trying to make various geometry classes constexpr.

BUG=none
TEST=none
==========

to

==========
Remove unused variables, cc/ edition.

These were exposed while trying to make various geometry classes constexpr.

BUG=none
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Peter Kasting, 2016-06-10 21:24:35]

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

[Peter Kasting, 2016-06-10 21:24:36]

Please tell me what should be done with the code in
occlusiion_tracker_perftest.cc.

[danakj, 2016-06-10 21:30:58]

LGTM

https://codereview.chromium.org/2054253002/diff/20001/cc/trees/occlusion_trac...
File cc/trees/occlusion_tracker_perftest.cc (right):

https://codereview.chromium.org/2054253002/diff/20001/cc/trees/occlusion_trac...
cc/trees/occlusion_tracker_perftest.cc:191: /* This block doesn't seem to do
anything, missing part of the loop body?
TBH I think you can delete this file. That's easier than figuring this out.

[Peter Kasting, 2016-06-10 21:56:48]

https://codereview.chromium.org/2054253002/diff/20001/cc/trees/occlusion_trac...
File cc/trees/occlusion_tracker_perftest.cc (right):

https://codereview.chromium.org/2054253002/diff/20001/cc/trees/occlusion_trac...
cc/trees/occlusion_tracker_perftest.cc:191: /* This block doesn't seem to do
anything, missing part of the loop body?
On 2016/06/10 21:30:58, danakj wrote:
> TBH I think you can delete this file. That's easier than figuring this out.

That makes it easy.

[Peter Kasting, 2016-06-10 21:56:53]

The CQ bit was checked by pkasting@chromium.org

[Peter Kasting, 2016-06-10 21:56:54]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/2054253002/#ps40001
(title: "Delete perftest")

[commit-bot: I haz the power, 2016-06-10 21:57:33]

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

[commit-bot: I haz the power, 2016-06-11 01:40:12]

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

[commit-bot: I haz the power, 2016-06-11 01:40:13]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Peter Kasting, 2016-06-11 01:41:28]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2016-06-11 01:41:45]

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

[commit-bot: I haz the power, 2016-06-11 02:22:29]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-11 02:23:52]

Description was changed from

==========
Remove unused variables, cc/ edition.

These were exposed while trying to make various geometry classes constexpr.

BUG=none
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Remove unused variables, cc/ edition.

These were exposed while trying to make various geometry classes constexpr.

BUG=none
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-06-11 02:23:53]

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