commit | 889f7aed3cb75a8642d57b0147bd4ec2d2c3c999 | [log] [tgz] |
---|---|---|
author | geofflang <geofflang@chromium.org> | Tue Jun 07 17:41:33 2016 |
committer | Commit bot <commit-bot@chromium.org> | Tue Jun 07 17:44:18 2016 |
tree | bbd70cba7979a79863b4749a76c20316a62c32bd | |
parent | 7a43c3ede6d7db6e9ec25c80db8fbcd5ed80ad0f [diff] |
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 Review-Url: https://codereview.chromium.org/2046033002 Cr-Commit-Position: refs/heads/master@{#398327}