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

Issue 886153002: Revert of Add device space "nudge" to gpu draws (Closed)

Created:
5 years, 10 months ago by robertphillips
Modified:
5 years, 10 months ago
Reviewers:
bsalomon
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Revert of Add device space "nudge" to gpu draws (patchset #6 id:90001 of https://codereview.chromium.org/877473005/) Reason for revert: Blink layout tests this time :( Original issue's description: > Add device space "nudge" to gpu draws > > This CL nudges all the GPU draws and clips slightly to match raster's round behavior for BW draws. We assume the effect will be negligible and do it for AA draws too. > > BUG=423834 > > Committed: https://skia.googlesource.com/skia/+/2d55d07501c56310f97d2092d789a2bc9fa01b78 > > Committed: https://skia.googlesource.com/skia/+/b9329991426d0b77ea194a380d72d73fb855308a TBR=bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=423834 Committed: https://skia.googlesource.com/skia/+/7f14c9bbf1954a88226a0b56f1abb1a17f31534c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+83 lines, -88 lines) Patch
M samplecode/SampleClipDrawMatch.cpp View 7 chunks +59 lines, -66 lines 0 comments Download
M src/core/SkClipStack.cpp View 1 chunk +13 lines, -0 lines 0 comments Download
M src/gpu/effects/GrConvexPolyEffect.cpp View 3 chunks +7 lines, -16 lines 0 comments Download
M src/gpu/gl/builders/GrGLVertexShaderBuilder.cpp View 1 chunk +4 lines, -6 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
robertphillips
Created Revert of Add device space "nudge" to gpu draws
5 years, 10 months ago (2015-01-30 22:43:31 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/886153002/1
5 years, 10 months ago (2015-01-30 22:44:17 UTC) #2
commit-bot: I haz the power
5 years, 10 months ago (2015-01-30 22:44:26 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/7f14c9bbf1954a88226a0b56f1abb1a17f31534c

Powered by Google App Engine
This is Rietveld 408576698