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

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromium.org,vmpstr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259, 621422, 621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/12074d3d1ac750fb170355f503d160f48cfefed0
Cr-Commit-Position: refs/heads/master@{#400873}

[sunnyps, 2016-06-21 00:10:50]

The CQ bit was checked by sunnyps@chromium.org

[sunnyps, 2016-06-21 00:10:50]

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

[commit-bot: I haz the power, 2016-06-21 00:11:21]

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

[commit-bot: I haz the power, 2016-06-21 00:13:44]

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

[commit-bot: I haz the power, 2016-06-21 00:13:45]

Try jobs failed on following builders:
  ios-device-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[sunnyps, 2016-06-21 00:50:59]

Description was changed from

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259
==========

to

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[sunnyps, 2016-06-21 00:56:31]

The CQ bit was checked by sunnyps@chromium.org

[commit-bot: I haz the power, 2016-06-21 00:56:45]

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

[sunnyps, 2016-06-21 01:16:09]

Description was changed from

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259,621422,621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-06-21 01:58:03]

Description was changed from

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259,621422,621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259,621422,621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-06-21 01:58:05]

Committed patchset #2 (id:310001)

[commit-bot: I haz the power, 2016-06-21 02:01:41]

Description was changed from

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259,621422,621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

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

Reason for revert:
Bug 621422, 621130

Some tasks can be completed twice which leads to crashes.

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
> Committed: https://crrev.com/3b0f0b8d3db0a9f66864d5b7da87c82f49e74a29
> Cr-Original-Commit-Position: refs/heads/master@{#398204}
> Cr-Commit-Position: refs/heads/master@{#399983}

TBR=piman@chromium.org,reveman@chromium.org,vmiura@chromium.org,ericrk@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=525259,621422,621130
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/12074d3d1ac750fb170355f503d160f48cfefed0
Cr-Commit-Position: refs/heads/master@{#400873}
==========

[commit-bot: I haz the power, 2016-06-21 02:01:42]

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