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

Issue 2253413002: Revert of Fix ineffective --disable-gpu-driver-bug-workarounds (Closed)

Created:
4 years, 4 months ago by Julien Isorce Samsung
Modified:
4 years, 4 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam, piman+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Fix ineffective --disable-gpu-driver-bug-workarounds (patchset #11 id:200001 of https://codereview.chromium.org/2195743005/ ) Reason for revert: Cause failures on Nexus 9 GPU bot https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%209%29/builds/1385 Original issue's description: > Fix ineffective --disable-gpu-driver-bug-workarounds > > It was hidden to the user since chrome://gpu was not showing > any workarounds, but they were still applied in the gpu process. > > Also add a gpu test that sets > --disable-gpu-driver-bug-workarounds --use_gpu_driver_workaround_for_testing > and verifies that the only active workaround is USE_TESTING_GPU_DRIVER_WORKAROUND > in both browser and gpu processes. > > BUG=359367 > > R=kbr@chromium.org, zmo@chromium.org > > TEST=./content/test/gpu/run_gpu_test.py gpu_process --show-stdout --browser=exact --extra-browser-args="--no-sandbox" --browser-executable=./out/build/chrome > CQ_INCLUDE_TRYBOTS=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 > > Committed: https://crrev.com/d18006d595e7b6bf070f9ab9777d353ac8457755 > Committed: https://crrev.com/7eee382917c50ad10a487ef1c1402adc8c84e9fd > Cr-Original-Commit-Position: refs/heads/master@{#409197} > Cr-Commit-Position: refs/heads/master@{#412685} TBR=kbr@chromium.org,zmo@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=359367 Committed: https://crrev.com/ccb4502860e77bcf5a7aa38da0a5bc6d7ebac432 Cr-Commit-Position: refs/heads/master@{#412775}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -118 lines) Patch
M content/browser/gpu/gpu_process_host.cc View 1 chunk +0 lines, -1 line 0 comments Download
M content/test/gpu/page_sets/gpu_process_tests.py View 4 chunks +25 lines, -117 lines 0 comments Download

Messages

Total messages: 16 (7 generated)
Julien Isorce Samsung
Created Revert of Fix ineffective --disable-gpu-driver-bug-workarounds
4 years, 4 months ago (2016-08-18 06:59:32 UTC) #2
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/2253413002/1
4 years, 4 months ago (2016-08-18 06:59:43 UTC) #3
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 4 months ago (2016-08-18 06:59:47 UTC) #5
Julien Isorce Samsung
On 2016/08/18 06:59:47, commit-bot: I haz the power wrote: > No L-G-T-M from a valid ...
4 years, 4 months ago (2016-08-18 07:35:57 UTC) #8
mlamouri (slow - plz ping)
lgtm
4 years, 4 months ago (2016-08-18 09:47:28 UTC) #9
mlamouri (slow - plz ping)
lgtm
4 years, 4 months ago (2016-08-18 09:47:28 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/2253413002/1
4 years, 4 months ago (2016-08-18 09:47:50 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 4 months ago (2016-08-18 09:48:37 UTC) #14
commit-bot: I haz the power
4 years, 4 months ago (2016-08-18 09:51:00 UTC) #16
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ccb4502860e77bcf5a7aa38da0a5bc6d7ebac432
Cr-Commit-Position: refs/heads/master@{#412775}

Powered by Google App Engine
This is Rietveld 408576698