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

Issue 1945653002: Revert of Enable GPU Raster on OSX (Closed)

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

Description

Revert of Enable GPU Raster on OSX (patchset #3 id:40001 of https://codereview.chromium.org/1881053003/ ) Reason for revert: Failures on all webkit mac bots: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.9/builds/31353 Original issue's description: > This change enables GPU Raster on OSX. > > I've compiled a bit of benchmark data to support turning this on - > https://docs.google.com/a/google.com/document/d/1RPEnillJOaOKW47t9DYuttQkvWDo7ilx9KDB_nJnwpc/edit?usp=sharing > Still filling in more data, but I'm pretty happy with the results so far. In > a number of cases we have a performance win, and in most cases we have a power win. > > Checking this in now will give us a long canary period before the next branch > point to catch any issues. > > BUG=542959 > CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel > > Committed: https://crrev.com/dc7207cc95e97078afea95e5084ea81a284f5d29 > Cr-Commit-Position: refs/heads/master@{#391204} TBR=vmiura@chromium.org,piman@chromium.org,ericrk@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=542959 Committed: https://crrev.com/5ffff49d496c8a651a3429bb9c6df0bf1d472397 Cr-Commit-Position: refs/heads/master@{#391218}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -27 lines) Patch
M content/browser/gpu/compositor_util.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M gpu/config/software_rendering_list_json.cc View 7 chunks +6 lines, -24 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Mathieu
Created Revert of Enable GPU Raster on OSX
4 years, 7 months ago (2016-05-03 13:24:40 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1945653002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1945653002/1
4 years, 7 months ago (2016-05-03 13:24:47 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-03 13:25:15 UTC) #3
commit-bot: I haz the power
4 years, 7 months ago (2016-05-03 13:26:36 UTC) #5
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/5ffff49d496c8a651a3429bb9c6df0bf1d472397
Cr-Commit-Position: refs/heads/master@{#391218}

Powered by Google App Engine
This is Rietveld 408576698