Fix "unused variable" warnings.

Fix "unused variable" warnings.

These were exposed by changing DISALLOW_COPY_AND_ASSIGN to use "= delete", and
must be fixed before that change can land.

See https://codereview.chromium.org/1981053002/ for the full CL.

BUG=447156
TEST=none

Committed: https://crrev.com/1d3480cd3bfb90eeb201bfcc5a47db39537adf58
Cr-Commit-Position: refs/heads/master@{#394813}

[Peter Kasting, 2016-05-19 07:32:05]

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

[sadrul, 2016-05-19 13:10:07]

nice! lgtm

https://codereview.chromium.org/1995963002/diff/20001/ui/gl/sync_control_vsyn...
File ui/gl/sync_control_vsync_provider.cc (right):

https://codereview.chromium.org/1995963002/diff/20001/ui/gl/sync_control_vsyn...
ui/gl/sync_control_vsync_provider.cc:29: #if defined(OS_LINUX)
It looks like this whole file is for linux only. Perhaps should be excluded on
non-linux platforms.

[Peter Kasting, 2016-05-19 18:03:46]

https://codereview.chromium.org/1995963002/diff/20001/ui/gl/sync_control_vsyn...
File ui/gl/sync_control_vsync_provider.cc (right):

https://codereview.chromium.org/1995963002/diff/20001/ui/gl/sync_control_vsyn...
ui/gl/sync_control_vsync_provider.cc:29: #if defined(OS_LINUX)
On 2016/05/19 13:10:06, sadrul wrote:
> It looks like this whole file is for linux only. Perhaps should be excluded on
> non-linux platforms.

Yeah, I saw that -- it looks like we want to define these symbols for all
platforms but they're basically no-ops everywhere but Linux.  I think you're
right that it would make sense to put the Linux definitions in a Linux-specific
file.  I'd rather do that outside this change, though.

[Peter Kasting, 2016-05-19 18:03:53]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2016-05-19 18:04:16]

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

[commit-bot: I haz the power, 2016-05-19 18:10:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-19 18:11:29]

Description was changed from

==========
Fix "unused variable" warnings.

These were exposed by changing DISALLOW_COPY_AND_ASSIGN to use "= delete", and
must be fixed before that change can land.

See https://codereview.chromium.org/1981053002/ for the full CL.

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

to

==========
Fix "unused variable" warnings.

These were exposed by changing DISALLOW_COPY_AND_ASSIGN to use "= delete", and
must be fixed before that change can land.

See https://codereview.chromium.org/1981053002/ for the full CL.

BUG=447156
TEST=none

Committed: https://crrev.com/1d3480cd3bfb90eeb201bfcc5a47db39537adf58
Cr-Commit-Position: refs/heads/master@{#394813}
==========

[commit-bot: I haz the power, 2016-05-19 18:11:30]

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