Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

It always returns true, even when the Window Server is no longer using it.
CVPixelBufferPool does the appropriate checking.

BUG=621899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/0f5be6cf7b919397dd97ee350b1a663227ac13ea
Cr-Commit-Position: refs/heads/master@{#401217}

[erikchen, 2016-06-22 01:39:11]

Description was changed from

==========
Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

It always returns true, even when the Window Server is no longer using it.
CVPixelBufferPool does the appropriate checking.

BUG=621899
==========

to

==========
Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

It always returns true, even when the Window Server is no longer using it.
CVPixelBufferPool does the appropriate checking.

BUG=621899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[erikchen, 2016-06-22 01:40:53]

erikchen@chromium.org changed reviewers:
+ ccameron@chromium.org

[erikchen, 2016-06-22 01:40:54]

ccameron: Please review.

[erikchen, 2016-06-22 01:41:02]

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

[commit-bot: I haz the power, 2016-06-22 01:41:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085173002/1

[ccameron, 2016-06-22 02:06:22]

lgtm

https://codereview.chromium.org/2085173002/diff/1/gpu/ipc/service/image_trans...
File gpu/ipc/service/image_transport_surface_overlay_mac.h (left):

https://codereview.chromium.org/2085173002/diff/1/gpu/ipc/service/image_trans...
gpu/ipc/service/image_transport_surface_overlay_mac.h:103: struct
IOSurfaceInUseQuery {
Seems like reasonable cleanp.

[commit-bot: I haz the power, 2016-06-22 02:52:42]

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

[commit-bot: I haz the power, 2016-06-22 02:52:43]

Dry run: This issue passed the CQ dry run.

[erikchen, 2016-06-22 06:41:11]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-06-22 06:41:30]

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

[commit-bot: I haz the power, 2016-06-22 06:45:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-22 06:47:06]

Description was changed from

==========
Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

It always returns true, even when the Window Server is no longer using it.
CVPixelBufferPool does the appropriate checking.

BUG=621899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Don't check IOSurfaceIsInUse for IOSurfaces wrapped in a CVPixelBuffer.

It always returns true, even when the Window Server is no longer using it.
CVPixelBufferPool does the appropriate checking.

BUG=621899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/0f5be6cf7b919397dd97ee350b1a663227ac13ea
Cr-Commit-Position: refs/heads/master@{#401217}
==========

[commit-bot: I haz the power, 2016-06-22 06:47:07]

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