Revert of cc: Add mailbox support to ResourceProvider write locks.

Revert of cc: Add mailbox support to ResourceProvider write locks. (patchset #12 id:220001 of https://codereview.chromium.org/1951193002/ )

Reason for revert:
Failing context_lost_tests on mac: https://build.chromium.org/p/chromium.gpu/builders/Mac%2010.10%20Retina%20Release%20%28AMD%29/builds/9301
Failing webgl_conformance_tests on Nexus 5:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28Nexus%205%29/builds/486
and Nexus 6: https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28Nexus%206%29/builds/585

Original issue's description:
> cc: Add mailbox support to ResourceProvider write locks.
>
> This adds support for mailboxes to ScopedWriteLockGL. Using the mailbox
> requires using ScopedTextureProvider/ScopedSkSurfaceProvider which
> ensures that the texture id for the mailbox is destroyed after use on
> the worker context.
>
> This CL also includes the following cleanup:
> 1. ResourceProvider locks don't keep resource pointers around.
> 2. ScopedSamplerGL does not inherit from ScopedReadLockGL.
> 3. GpuRasterizer is folded back into GpuRasterBufferProvider.
> 4. TileTaskManager does not own RasterBufferProvider.
>
> BUG=525259
> R=piman@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/5fa5dbdf25bbec21b84f752d3f0642cd184467e2
> Cr-Commit-Position: refs/heads/master@{#398204}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,sunnyps@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525259
Committed: https://crrev.com/889f7aed3cb75a8642d57b0147bd4ec2d2c3c999
Cr-Commit-Position: refs/heads/master@{#398327}

[Geoff Lang, 2016-06-07 17:38:51]

The CQ bit was checked by geofflang@chromium.org

[Geoff Lang, 2016-06-07 17:38:52]

Created Revert of cc: Add mailbox support to ResourceProvider write locks.

[commit-bot: I haz the power, 2016-06-07 17:39:27]

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

[commit-bot: I haz the power, 2016-06-07 17:41:44]

Description was changed from

==========
Revert of cc: Add mailbox support to ResourceProvider write locks. (patchset #12
id:220001 of https://codereview.chromium.org/1951193002/ )

Reason for revert:
Failing context_lost_tests on mac:
https://build.chromium.org/p/chromium.gpu/builders/Mac%2010.10%20Retina%20Rel...
Failing webgl_conformance_tests on Nexus 5:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...
and Nexus 6:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...

Original issue's description:
> cc: Add mailbox support to ResourceProvider write locks.
> 
> This adds support for mailboxes to ScopedWriteLockGL. Using the mailbox
> requires using ScopedTextureProvider/ScopedSkSurfaceProvider which
> ensures that the texture id for the mailbox is destroyed after use on
> the worker context.
> 
> This CL also includes the following cleanup:
> 1. ResourceProvider locks don't keep resource pointers around.
> 2. ScopedSamplerGL does not inherit from ScopedReadLockGL.
> 3. GpuRasterizer is folded back into GpuRasterBufferProvider.
> 4. TileTaskManager does not own RasterBufferProvider.
> 
> BUG=525259
> R=piman@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5fa5dbdf25bbec21b84f752d3f0642cd184467e2
> Cr-Commit-Position: refs/heads/master@{#398204}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,sunnyps@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525259
==========

to

==========
Revert of cc: Add mailbox support to ResourceProvider write locks. (patchset #12
id:220001 of https://codereview.chromium.org/1951193002/ )

Reason for revert:
Failing context_lost_tests on mac:
https://build.chromium.org/p/chromium.gpu/builders/Mac%2010.10%20Retina%20Rel...
Failing webgl_conformance_tests on Nexus 5:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...
and Nexus 6:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...

Original issue's description:
> cc: Add mailbox support to ResourceProvider write locks.
> 
> This adds support for mailboxes to ScopedWriteLockGL. Using the mailbox
> requires using ScopedTextureProvider/ScopedSkSurfaceProvider which
> ensures that the texture id for the mailbox is destroyed after use on
> the worker context.
> 
> This CL also includes the following cleanup:
> 1. ResourceProvider locks don't keep resource pointers around.
> 2. ScopedSamplerGL does not inherit from ScopedReadLockGL.
> 3. GpuRasterizer is folded back into GpuRasterBufferProvider.
> 4. TileTaskManager does not own RasterBufferProvider.
> 
> BUG=525259
> R=piman@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5fa5dbdf25bbec21b84f752d3f0642cd184467e2
> Cr-Commit-Position: refs/heads/master@{#398204}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,sunnyps@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525259
==========

[commit-bot: I haz the power, 2016-06-07 17:41:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-07 17:44:28]

Description was changed from

==========
Revert of cc: Add mailbox support to ResourceProvider write locks. (patchset #12
id:220001 of https://codereview.chromium.org/1951193002/ )

Reason for revert:
Failing context_lost_tests on mac:
https://build.chromium.org/p/chromium.gpu/builders/Mac%2010.10%20Retina%20Rel...
Failing webgl_conformance_tests on Nexus 5:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...
and Nexus 6:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...

Original issue's description:
> cc: Add mailbox support to ResourceProvider write locks.
> 
> This adds support for mailboxes to ScopedWriteLockGL. Using the mailbox
> requires using ScopedTextureProvider/ScopedSkSurfaceProvider which
> ensures that the texture id for the mailbox is destroyed after use on
> the worker context.
> 
> This CL also includes the following cleanup:
> 1. ResourceProvider locks don't keep resource pointers around.
> 2. ScopedSamplerGL does not inherit from ScopedReadLockGL.
> 3. GpuRasterizer is folded back into GpuRasterBufferProvider.
> 4. TileTaskManager does not own RasterBufferProvider.
> 
> BUG=525259
> R=piman@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/5fa5dbdf25bbec21b84f752d3f0642cd184467e2
> Cr-Commit-Position: refs/heads/master@{#398204}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,sunnyps@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525259
==========

to

==========
Revert of cc: Add mailbox support to ResourceProvider write locks. (patchset #12
id:220001 of https://codereview.chromium.org/1951193002/ )

Reason for revert:
Failing context_lost_tests on mac:
https://build.chromium.org/p/chromium.gpu/builders/Mac%2010.10%20Retina%20Rel...
Failing webgl_conformance_tests on Nexus 5:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...
and Nexus 6:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Debug%20%28N...

Original issue's description:
> cc: Add mailbox support to ResourceProvider write locks.
>
> This adds support for mailboxes to ScopedWriteLockGL. Using the mailbox
> requires using ScopedTextureProvider/ScopedSkSurfaceProvider which
> ensures that the texture id for the mailbox is destroyed after use on
> the worker context.
>
> This CL also includes the following cleanup:
> 1. ResourceProvider locks don't keep resource pointers around.
> 2. ScopedSamplerGL does not inherit from ScopedReadLockGL.
> 3. GpuRasterizer is folded back into GpuRasterBufferProvider.
> 4. TileTaskManager does not own RasterBufferProvider.
>
> BUG=525259
> R=piman@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/5fa5dbdf25bbec21b84f752d3f0642cd184467e2
> Cr-Commit-Position: refs/heads/master@{#398204}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,sunnyps@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525259

Committed: https://crrev.com/889f7aed3cb75a8642d57b0147bd4ec2d2c3c999
Cr-Commit-Position: refs/heads/master@{#398327}
==========

[commit-bot: I haz the power, 2016-06-07 17:44:29]

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