Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(374)

Issue 2866353002: Add workaround for Mac stencil buffer leak (Closed)

Created:
3 years, 7 months ago by ericrk
Modified:
3 years, 7 months ago
Reviewers:
Tom Sepez, piman
CC:
chromium-reviews, rjkroege, Ian Vollick, jbauman+watch_chromium.org, asvitkine+watch_chromium.org, kalyank, piman+watch_chromium.org, cc-bugs_chromium.org, danakj+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add workaround for Mac stencil buffer leak This change adds a new workaround, "avoid_stencil_buffers", which is enabled on certain Macs with leaky drivers. This workaround currently only forwards to Skia, enabling a mode where stencil buffers are avoided. BUG=713854 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2866353002 Cr-Commit-Position: refs/heads/master@{#470712} Committed: https://chromium.googlesource.com/chromium/src/+/4f3b335015f66c266bc1ed11f7b22761907df6f4

Patch Set 1 #

Patch Set 2 : cleanup #

Patch Set 3 : Fix analyze issue #

Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -7 lines) Patch
M cc/output/in_process_context_provider.cc View 1 chunk +2 lines, -1 line 0 comments Download
M cc/test/test_in_process_context_provider.cc View 1 chunk +2 lines, -1 line 0 comments Download
M gpu/command_buffer/common/capabilities.h View 1 chunk +1 line, -0 lines 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 1 chunk +1 line, -0 lines 0 comments Download
M gpu/config/gpu_driver_bug_list.json View 2 chunks +15 lines, -1 line 0 comments Download
M gpu/config/gpu_driver_bug_workaround_type.h View 1 chunk +2 lines, -0 lines 0 comments Download
M gpu/ipc/common/gpu_command_buffer_traits_multi.h View 1 chunk +1 line, -0 lines 0 comments Download
M gpu/skia_bindings/BUILD.gn View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M gpu/skia_bindings/grcontext_for_gles2_interface.h View 2 chunks +4 lines, -1 line 0 comments Download
M gpu/skia_bindings/grcontext_for_gles2_interface.cc View 3 chunks +4 lines, -1 line 0 comments Download
M services/ui/public/cpp/gpu/context_provider_command_buffer.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ui/compositor/test/in_process_context_provider.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 18 (12 generated)
ericrk
3 years, 7 months ago (2017-05-09 21:57:01 UTC) #4
piman
lgtm
3 years, 7 months ago (2017-05-09 22:17:45 UTC) #5
ericrk
+tsepez for gpu\ipc\common\gpu_command_buffer_traits_multi.h Thanks!
3 years, 7 months ago (2017-05-10 16:37:24 UTC) #7
Tom Sepez
RS LGTM on adding a bool.
3 years, 7 months ago (2017-05-10 16:39:01 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2866353002/40001
3 years, 7 months ago (2017-05-10 17:47:30 UTC) #15
commit-bot: I haz the power
3 years, 7 months ago (2017-05-10 21:17:58 UTC) #18
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4f3b335015f66c266bc1ed11f7b2...

Powered by Google App Engine
This is Rietveld 408576698