| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
| |
BUG=skia:2935
Review URL: https://codereview.chromium.org/684833003
|
|
|
|
|
|
|
|
| |
When an error occurs reading a flattenable object, it should be unrefed instead of deleted because, in the case of SkXferMode, for example, we'd actually be deleting a cached object kept in a static array.
BUG=428789
Review URL: https://codereview.chromium.org/695633003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/670303005
|
|
|
|
|
|
|
|
|
|
|
| |
Since GrPrintf is just defined to SkDebugf, we can go with the later
directly.
BUG=None
TEST=None
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/695663003
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/689833006
|
|
|
|
|
|
|
|
|
|
|
|
| |
It is defined to SK_ARRAY_COUNT, and there are no uses of it.
Lets just use the SK_ variant.
BUG=None
TEST=None
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/691133003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/692953002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/696463003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change is here since previously color bitmap text was rendered using a
geometry processor in the coverage stage. The problem with this is that we
cannot correctly do xfer modes with this method. So I now make color bitmap text
draw using a color stage in the same was as a draw bitmap call.
One issue that arrises from this fix is that we end up adding this final color
processor after any previous color processors. Thus if we have a custom blend
implemented as a color processor it will be before this text one and we won't
blend correctly. This issue will get fixed once an xfer processor is fully
implemented. I have hacked a test locally to show that if we can add the text
color processor to the begining of the color stages we do blend correctly in all
cases (so the xfer processor will be a fix).
BUG=skia:
Review URL: https://codereview.chromium.org/689923004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/694693002
|
|
|
|
|
|
|
|
|
|
|
| |
Revert "Disable SkTaskGroup in SkMultiPictureDraw temporarily."
Revert "Revert harder, removing SkTaskGroup.cpp from core temporarily."
NOTREECHECKS=true
BUG=skia:
Review URL: https://codereview.chromium.org/687263007
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will be a bit hairy to review.
The FillBounds and CollectLayers code has diverged significantly resulting in the rendering path seeing different bounds than the hoisting path. This CL merges the FillBounds changes into CollectLayers. A follow on CL will, hopefully, find a way to layer CollectLayers on top of FillBounds.
The only code in CollectLayers that is different from FillBounds is bracketed by "LAYER HOISTING" comments.
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/685263004
|
|
|
|
|
|
|
|
| |
Got a few crashes running the fuzzer locally, all related to handling NULL members/parameters in an inconsistent way.
BUG=skia:
Review URL: https://codereview.chromium.org/675013003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/689943002
|
|
|
|
|
|
| |
BUG=skia:3032
Review URL: https://codereview.chromium.org/694573002
|
|
|
|
|
|
|
| |
This allows us to upgrade to NDK r10c as the dftext gm no longer fails
when compiled with -O2.
Review URL: https://codereview.chromium.org/690053002
|
|
|
|
|
|
| |
This assists debugging layer hoisting errors (e.g., when a layer is hoisted but not actually used in subsequent rendering).
Review URL: https://codereview.chromium.org/694533004
|
|
|
|
|
|
|
|
| |
This renaming was missed in (MultiPictureDraw is taskgroup aware - https://codereview.chromium.org/684923002)
TBR=reed@google.com
Review URL: https://codereview.chromium.org/654293005
|
|
|
|
|
|
| |
These two functions do the same thing, combine them.
Review URL: https://codereview.chromium.org/688133003
|
|
|
|
|
|
| |
Fixes build.
Review URL: https://codereview.chromium.org/692913003
|
|
|
|
|
|
| |
BUG=skia:3074
Review URL: https://codereview.chromium.org/678303004
|
|
|
|
|
|
| |
A picture may possess many layers that get placed in one plot of the atlas. In this case we can only remove the plot from the plotUsage tracking structure when all the layers belonging to the picture in that plot have been removed.
Review URL: https://codereview.chromium.org/654463004
|
|
|
|
| |
Review URL: https://codereview.chromium.org/694533002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/685243002
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/688793003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/692803003
|
|
|
|
|
|
|
|
|
|
| |
Motivation: SkDocument_PDF can be refactored to stream all
PDF objects as SkCanvas calls are made, to save memory!
BUG=skia:3030
BUG=skia:2683
Review URL: https://codereview.chromium.org/691783002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/685153004/)
Reason for revert:
Experimental revert to see if this is blocking the DEPS roll
Original issue's description:
> scale xmin and xmax for FontMetrics
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/79c276e4fd93d9c9fd57a45bc6929c48e4415b2f
TBR=mtklein@google.com,bungeman@google.com,reed@google.com,reed@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/691033002
|
|
|
|
|
|
|
|
|
| |
Not all linkers are seeing the SkThread code is not needed.
BUG=skia:
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/673903007
|
|
|
|
|
|
|
|
| |
Think this should be enough to get rolling again.
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/687273006
|
|
|
|
|
|
|
| |
This removes ADB from the Skia checkout and also does some quick
checks to make sure the version of ADB used meets a minimum threshold
Review URL: https://codereview.chromium.org/692513003
|
|
|
|
|
|
|
|
|
| |
Adds miplevel as part of dfpath key, and scale factor so we know
how much to adjust to fit desired scale.
BUG=skia:2935
Review URL: https://codereview.chromium.org/687283002
|
|
|
|
|
|
|
|
| |
BUG=skia:3049
TBR=
NOTRY=true
Review URL: https://codereview.chromium.org/692843002
|
|
|
|
|
|
|
|
|
| |
NOTRY=true
TBR=rmistry
NOTREECHECKS=true
BUG=skia:
Review URL: https://codereview.chromium.org/693483003
|
|
|
|
| |
BUG=skia:3049
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/692543004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/684223002
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/665233005
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #5 id:80001 of https://codereview.chromium.org/680363003/)
Reason for revert:
Mac mini asserting
Original issue's description:
> Use SkTypeface::getBounds() in bounding-box calculations.
>
> This should produce tighter conservative bounding boxes for text than the
> approximation code it replaces.
>
> Recording performance is neutral on my desktop. Playback performance
> improves by up to 15% on text heavy pages, e.g.
>
> desk_pokemonwiki.skp_1 3.24ms -> 2.83ms 0.87x
> desk_baidu.skp_1 1.91ms -> 1.58ms 0.83x
>
> Committed: https://skia.googlesource.com/skia/+/bf8dc343df4fbdcb8af546eb68b640e011a33489
>
> CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot
>
> Committed: https://skia.googlesource.com/skia/+/c51add674dfb89b988a7fbc05f41838c203f9dcd
TBR=reed@google.com,mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/690833002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/688753002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should produce tighter conservative bounding boxes for text than the
approximation code it replaces.
Recording performance is neutral on my desktop. Playback performance
improves by up to 15% on text heavy pages, e.g.
desk_pokemonwiki.skp_1 3.24ms -> 2.83ms 0.87x
desk_baidu.skp_1 1.91ms -> 1.58ms 0.83x
Committed: https://skia.googlesource.com/skia/+/bf8dc343df4fbdcb8af546eb68b640e011a33489
CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot
Review URL: https://codereview.chromium.org/680363003
|
| |
|
|
|
|
|
|
|
|
| |
Porting QuiltTask isn't important in itself; this is mostly an API feeler.
BUG=skia:
Review URL: https://codereview.chromium.org/689673003
|
|
|
|
|
|
|
|
| |
Looks like a fairly large recording speed win with no playback cost.
BUG=skia:
Review URL: https://codereview.chromium.org/653023003
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This is intended to prevent ghosting on tiled architectures.
This CL also defers creation of the atlas (and its texture) until it is actually needed.
Committed: https://skia.googlesource.com/skia/+/6d5b5455743414ddb11d2b8c1fe9d7959f2b853d
Review URL: https://codereview.chromium.org/678403002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/685153004
|
|
|
|
| |
Review URL: https://codereview.chromium.org/692633002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/690533004/)
Reason for revert:
Breaking tests in Chromium, and not in a good way.
Original issue's description:
> scale xmin/xmax etc in fontmetrics
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/f8c2feffd68b3262f3c039ec1bd517c5f6cc9e4c
TBR=reed@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/686263002
|
|
|
|
|
|
|
|
| |
GrGL* subclasses of GrSurface.
BUG=skia:2889
Review URL: https://codereview.chromium.org/691523002
|