| Commit message (Collapse) | Author | Age |
| |
|
| |
|
|
|
|
|
|
|
|
| |
This will help with the ability to subclass, add to, etc. GrInvariantOutput. Also it was simply
getting a little too big to be a "supporting" subclass
BUG=skia:
Review URL: https://codereview.chromium.org/699943003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/698873003
|
|
|
|
|
|
|
|
|
|
|
| |
This define was added in
"SK_USE_SCALED_FONTMETRICS for correct scaling"
c17c6582ec0b7bc211d7de9e8d634692d239c123.
Users all now define this flag, so it may now be removed.
BUG=chromium:420901
Review URL: https://codereview.chromium.org/720743003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/717143003
|
|
|
|
|
|
|
|
|
|
| |
Now that CollectLayers directly uses FillBounds we can:
skip the explicit intersection with the clipBounds after an adjustAndMap call
skip the storage and use of the clipBounds in SaveLayerInfo
Review URL: https://codereview.chromium.org/719793002
|
|
|
|
|
|
| |
Rather then exposing parts of FillBounds (as in Expose FillBounds to allow GrPictureUtils::CollectLayers to be layered on top of it - https://codereview.chromium.org/698643002/), this CL moves CollectLayers into SkRecordDraw.cpp to accomplish the layering.
Review URL: https://codereview.chromium.org/716913003
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/717033002
|
|
|
|
|
|
|
|
|
|
| |
It is probably broken, and by now nobody should be using it really
anyway.
BUG=None
R=borenet@google.com
Review URL: https://codereview.chromium.org/720603002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/716793004
|
|
|
|
|
|
| |
BUG=skia:3059
Review URL: https://codereview.chromium.org/715093003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/715873002
|
|
|
|
|
|
|
|
| |
Internal only helper class for manipulating and accessing cache keys.
BUG=skia:2889
Review URL: https://codereview.chromium.org/703303003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #1 id:1 of https://codereview.chromium.org/714113002/)
Reason for revert:
The code this depends on is not ready yet. We can submit once https://android-review.googlesource.com/#/c/113924/ is submitted.
Original issue's description:
> Move from libstlport to libc++ for Android framework builds
>
> Committed: https://skia.googlesource.com/skia/+/ce259510a738fab05a0d456440104f5aba1c9d72
TBR=tomhudson@google.com,djsollen@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/718793003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/683133006
|
|
|
|
|
|
| |
BUG=skia:3117
Review URL: https://codereview.chromium.org/691383003
|
|
|
|
|
|
|
|
| |
This reverts commit c71ffd4e76d1abcd28ac74463349970f60a3350f.
TBR=
Review URL: https://codereview.chromium.org/683003003
|
|
|
|
| |
Commit d09ade4b44edc11 was missing a needed edit.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/717753002/)
Reason for revert:
modecolorfilters, perhaps more changed
Original issue's description:
> modify nothingToDraw to notice filters
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d5688c509293a450c058e0cc1f4673be2931bb5c
TBR=djsollen@google.com,reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/717813002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/717753002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On my laptop:
maxrss loops min median mean max stddev samples config bench
37M 1 14ms 14.2ms 14.6ms 18.2ms 9% ▁█▁▁▁▁▂▂▂▁ gpu tiled_playback_tilegrid_tiled
40M 1 17ms 17.2ms 17.2ms 17.6ms 1% ▆▃▁█▄▇▂▁▁▁ gpu tiled_playback_tilegrid_random
40M 1 14.6ms 14.9ms 15.8ms 19.1ms 11% ▂▁▁▁▁▁▁█▅█ gpu tiled_playback_rtree_tiled
43M 1 16.5ms 16.7ms 16.8ms 17.4ms 1% ▂▃▅█▃▂▁▃▃▂ gpu tiled_playback_rtree_random
43M 1 15.9ms 16.1ms 16.5ms 18.7ms 6% ▁▁█▇▁▁▁▂▁▁ gpu tiled_playback_none_tiled
44M 1 17.9ms 17.9ms 18ms 18.1ms 1% ▂▁▅▁▇▃▁▂█▇ gpu tiled_playback_none_random
TileGrid and RTree perform pretty much the same, both beating no BBH.
BUG=skia:3085
Review URL: https://codereview.chromium.org/699313006
|
|
|
|
|
|
|
| |
Need to include (transitively) SkUserConfig before checking
SK_SUPPORT_GPU.
Review URL: https://codereview.chromium.org/693613005
|
|
|
|
|
|
| |
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/718763002
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/691143006
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/699023003
|
|
|
|
|
|
|
|
|
| |
content key can be set after pending io scheduled (e.g. when scaling up a npot texture to pot).
BUG=skia:2889
TBR=robertphillips@google.com
Review URL: https://codereview.chromium.org/693843004
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL rebaselines the buggy multipicturedraw GMs related to:
Always enable layer hoisting in Skia (https://codereview.chromium.org/698493002/)
TBR=jcgregorio@google.com
BUG=skia:3116
Review URL: https://codereview.chromium.org/715823002
|
|
|
|
|
|
|
| |
This adds support to rebaseline_server to fetch actual results from
tryjobs run on rietveld issues.
Review URL: https://codereview.chromium.org/688353003
|
|
|
|
|
|
|
|
|
|
| |
This CL rebaselines the multipicturedraw GMs as a follow up to:
Always enable layer hoisting in Skia (https://codereview.chromium.org/698493002/)
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/714153002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/697363004
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/707413003
|
|
|
|
|
|
|
|
|
| |
Since we now snap and create the ODS one time in the inorder draw buffer,
there is no need for us to keep a cached version of it around.
BUG=skia:
Review URL: https://codereview.chromium.org/701123003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/712363003
|
|
|
|
|
|
| |
BUG=skia:2889
Review URL: https://codereview.chromium.org/705413002
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Most recently introduced here: https://codereview.chromium.org/304333002
It appears these are not currently needed (I just disabled it and
successfully built).
Since we appear to be out of the transitional period, I propose we
remove them so we can know if we deprecate something Android uses
(and fix the call site).
Review URL: https://codereview.chromium.org/696063006
|
|
|
|
| |
Review URL: https://codereview.chromium.org/714113002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tests path_hairline_{small,big}_AA_conic were calling the test
function with NVPR. This caused a warning in nanobench.
The here removed hunk comes from commit referring to skia:2042 ("Enable
NVPR by default"). This is a workaround for a bug. The bug is fixed by
the commit referring to skia:2078 ("Logan bot fails NVPR assertion in
bench").
The proper fix is indeed make sure that path renderer chain ends up
trying software path renderer, if the path contains conics and is a
hairline.
The removed hunk refers also to skia:2033 ("Figure out what is happening
with conic path segments in NVPR"). The above solution is correct also in case
NVPR would support conics, as NVPR would not still support hairlines.
BUG=skia:2078
Review URL: https://codereview.chromium.org/685213005
|
|
|
|
|
|
|
|
|
| |
work on tests
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-ASAN-Trybot,Test-Ubuntu12-ShuttleA-GTX660-x86-Debug-Trybot,Test-Win7-ShuttleA-HD2000-x86_64-Debug-Trybot,Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot
BUG=skia:
Review URL: https://codereview.chromium.org/704923003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/696063005
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL:
1) removes the EXPERIMENTAL_optimize on SkCanvas & SkDevice
2) moves the saveLayer gathering step to endRecording
3) Replaces GPUOptimize with SkRecordComputeLayers
4) Update bench_pictures & render_pictures to provide the new flag
#2 also necessitated moving the BBH computation (and record optimization) out of SkPicture's ctor (and into endRecording)
Review URL: https://codereview.chromium.org/718443002
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/711253003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/709133003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/711143002
|
|
|
|
|
|
|
|
| |
As suggested by reed@ in http://crrev.com/711133002
TBR=reed@google.com
Review URL: https://codereview.chromium.org/707533003
|
|
|
|
|
|
|
| |
The comment was no longer relevant with
https://codereview.chromium.org/483593002
Review URL: https://codereview.chromium.org/715673002
|
|
|
|
|
|
|
|
|
| |
Fixes change introduced in http://crrev.com/645563002
This will allow SkDocument to be a drop-in repalcement to
SkPDFDocument/SkPDFDevice in Chromium.
Review URL: https://codereview.chromium.org/711133002
|
|
|
|
|
|
|
|
|
| |
This removes the old guarded code and enables the new api
introduced with "Update fontMgr to take list of bcp47 language tags."
c20386e3937d3d398ac9b35f9c7d997e972ade98 . Blink on Android is
already using the new code.
Review URL: https://codereview.chromium.org/705843004
|
|
|
|
|
|
|
|
|
|
|
|
| |
We previously saw crashes decoding bad ICO files. Add tests for
known bad files.
While testing, I learned that one of them still crashes. Check for
large offset and size separately to fix the crash.
BUG=skia:2878
Review URL: https://codereview.chromium.org/712123002
|
|
|
|
|
|
| |
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/684413004
|