Revert of Pepper takes ownership of a mailbox before passing it to the texture layer.

Revert of Pepper takes ownership of a mailbox before passing it to the texture layer. (patchset #19 id:350001 of https://codereview.chromium.org/1912833002/ )

Reason for revert:
Top crasher on renderer and gpu processes:

https://bugs.chromium.org/p/chromium/issues/detail?id=608163

Original issue's description:
> Pepper takes ownership of a mailbox before passing it to the texture layer.
>
> I replaced the IPC message GpuCommandBufferMsg_ProduceFrontBuffer with
> GpuCommandBufferMsg_TakeFrontBuffer and GpuCommandBufferMsg_ReturnFrontBuffer.
> TakeFrontBuffer gives ownership of the front buffer to the client. When the
> client returns it with ReturnFrontBuffer, the command buffer may choose to reuse
> it.
>
> This means that pepper no longer needs to use
> SetTextureMailboxWithoutReleaseCallback.
>
> BUG=350204, 602484
> 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/ed5a6bb9f4321670482acc32a7cb4246a1e22f25
> Cr-Commit-Position: refs/heads/master@{#390570}

TBR=ccameron@chromium.org,piman@chromium.org,bbudge@chromium.org,tsepez@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=350204, 602484
Committed: https://crrev.com/2e2db2fd684658f0c75b8ab730ac3d2cc36f9958
Cr-Commit-Position: refs/heads/master@{#390986}

[erikchen, 2016-05-02 16:45:19]

Created Revert of Pepper takes ownership of a mailbox before passing it to the
texture layer.

[commit-bot: I haz the power, 2016-05-02 16:45:45]

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

[commit-bot: I haz the power, 2016-05-02 17:01:37]

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

[commit-bot: I haz the power, 2016-05-02 17:32:32]

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

[commit-bot: I haz the power, 2016-05-02 17:45:57]

Description was changed from

==========
Revert of Pepper takes ownership of a mailbox before passing it to the texture
layer. (patchset #19 id:350001 of https://codereview.chromium.org/1912833002/ )

Reason for revert:
Top crasher on renderer and gpu processes:

https://bugs.chromium.org/p/chromium/issues/detail?id=608163

Original issue's description:
> Pepper takes ownership of a mailbox before passing it to the texture layer.
> 
> I replaced the IPC message GpuCommandBufferMsg_ProduceFrontBuffer with
> GpuCommandBufferMsg_TakeFrontBuffer and GpuCommandBufferMsg_ReturnFrontBuffer.
> TakeFrontBuffer gives ownership of the front buffer to the client. When the
> client returns it with ReturnFrontBuffer, the command buffer may choose to
reuse
> it.
> 
> This means that pepper no longer needs to use
> SetTextureMailboxWithoutReleaseCallback.
> 
> BUG=350204, 602484
>
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/ed5a6bb9f4321670482acc32a7cb4246a1e22f25
> Cr-Commit-Position: refs/heads/master@{#390570}

TBR=ccameron@chromium.org,piman@chromium.org,bbudge@chromium.org,tsepez@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=350204, 602484
==========

to

==========
Revert of Pepper takes ownership of a mailbox before passing it to the texture
layer. (patchset #19 id:350001 of https://codereview.chromium.org/1912833002/ )

Reason for revert:
Top crasher on renderer and gpu processes:

https://bugs.chromium.org/p/chromium/issues/detail?id=608163

Original issue's description:
> Pepper takes ownership of a mailbox before passing it to the texture layer.
> 
> I replaced the IPC message GpuCommandBufferMsg_ProduceFrontBuffer with
> GpuCommandBufferMsg_TakeFrontBuffer and GpuCommandBufferMsg_ReturnFrontBuffer.
> TakeFrontBuffer gives ownership of the front buffer to the client. When the
> client returns it with ReturnFrontBuffer, the command buffer may choose to
reuse
> it.
> 
> This means that pepper no longer needs to use
> SetTextureMailboxWithoutReleaseCallback.
> 
> BUG=350204, 602484
>
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/ed5a6bb9f4321670482acc32a7cb4246a1e22f25
> Cr-Commit-Position: refs/heads/master@{#390570}

TBR=ccameron@chromium.org,piman@chromium.org,bbudge@chromium.org,tsepez@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=350204, 602484
==========

[commit-bot: I haz the power, 2016-05-02 17:45:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-02 17:47:48]

Description was changed from

==========
Revert of Pepper takes ownership of a mailbox before passing it to the texture
layer. (patchset #19 id:350001 of https://codereview.chromium.org/1912833002/ )

Reason for revert:
Top crasher on renderer and gpu processes:

https://bugs.chromium.org/p/chromium/issues/detail?id=608163

Original issue's description:
> Pepper takes ownership of a mailbox before passing it to the texture layer.
> 
> I replaced the IPC message GpuCommandBufferMsg_ProduceFrontBuffer with
> GpuCommandBufferMsg_TakeFrontBuffer and GpuCommandBufferMsg_ReturnFrontBuffer.
> TakeFrontBuffer gives ownership of the front buffer to the client. When the
> client returns it with ReturnFrontBuffer, the command buffer may choose to
reuse
> it.
> 
> This means that pepper no longer needs to use
> SetTextureMailboxWithoutReleaseCallback.
> 
> BUG=350204, 602484
>
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/ed5a6bb9f4321670482acc32a7cb4246a1e22f25
> Cr-Commit-Position: refs/heads/master@{#390570}

TBR=ccameron@chromium.org,piman@chromium.org,bbudge@chromium.org,tsepez@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=350204, 602484
==========

to

==========
Revert of Pepper takes ownership of a mailbox before passing it to the texture
layer. (patchset #19 id:350001 of https://codereview.chromium.org/1912833002/ )

Reason for revert:
Top crasher on renderer and gpu processes:

https://bugs.chromium.org/p/chromium/issues/detail?id=608163

Original issue's description:
> Pepper takes ownership of a mailbox before passing it to the texture layer.
>
> I replaced the IPC message GpuCommandBufferMsg_ProduceFrontBuffer with
> GpuCommandBufferMsg_TakeFrontBuffer and GpuCommandBufferMsg_ReturnFrontBuffer.
> TakeFrontBuffer gives ownership of the front buffer to the client. When the
> client returns it with ReturnFrontBuffer, the command buffer may choose to
reuse
> it.
>
> This means that pepper no longer needs to use
> SetTextureMailboxWithoutReleaseCallback.
>
> BUG=350204, 602484
>
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/ed5a6bb9f4321670482acc32a7cb4246a1e22f25
> Cr-Commit-Position: refs/heads/master@{#390570}

TBR=ccameron@chromium.org,piman@chromium.org,bbudge@chromium.org,tsepez@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=350204, 602484

Committed: https://crrev.com/2e2db2fd684658f0c75b8ab730ac3d2cc36f9958
Cr-Commit-Position: refs/heads/master@{#390986}
==========

[commit-bot: I haz the power, 2016-05-02 17:47:49]

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