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

Issue 12207157: New class to simplify rasterization. (Closed)

Created:
7 years, 10 months ago by Tom Hudson
Modified:
7 years, 10 months ago
Reviewers:
reed2, reed1
CC:
chromium-reviews, nduca, Sami, danakj
Visibility:
Public.

Description

New class to simplify rasterization. skia::PaintSimplifier subclasses SkDrawFilter to reduce the quality of draws. The intent is to be able to place less of a burden on the rasterizer during Android flings or other situations where we need a lot of pixels but they won't spend long on screen. This CL does not actually invoke the PaintSimplifier. The code is extracted from https://codereview.chromium.org/12210081/; deciding when to simplify will be a separate CL. BUG=174945 TBR=reed Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182475

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -0 lines) Patch
A skia/ext/paint_simplifier.h View 1 chunk +33 lines, -0 lines 0 comments Download
A skia/ext/paint_simplifier.cc View 1 chunk +41 lines, -0 lines 0 comments Download
M skia/skia.gyp View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
commit-bot: I haz the power
No comments yet.
7 years, 10 months ago (2013-02-13 19:46:37 UTC) #1
bsalomon
On 2013/02/13 19:46:37, I haz the power (commit-bot) wrote: > No comments yet. lgtm
7 years, 10 months ago (2013-02-13 19:52:03 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
7 years, 10 months ago (2013-02-13 19:55:25 UTC) #3
reed1
lgtm
7 years, 10 months ago (2013-02-14 10:27:03 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tomhudson@chromium.org/12207157/1
7 years, 10 months ago (2013-02-14 10:31:53 UTC) #5
commit-bot: I haz the power
7 years, 10 months ago (2013-02-14 16:36:31 UTC) #6
Message was sent while issue was closed.
Change committed as 182475

Powered by Google App Engine
This is Rietveld 408576698