| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
| |
BUG=skia:4223
Review URL: https://codereview.chromium.org/1301583006
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1294713009
|
|
|
|
|
|
|
|
|
|
|
| |
By adding a cast to SK_ARRAY_COUNT, the compiler treats the
compile-time value as if it were a const int, and like regular
numbers, permits it to be compared to signed and unsigned
numbers freely.
R=reed@google.com
Review URL: https://codereview.chromium.org/1299943002
|
|
|
|
|
|
|
|
|
| |
Chrome
BUG=chromium:454267
BUG=chromium:522315
Review URL: https://codereview.chromium.org/1300123002
|
|
|
|
|
|
|
|
| |
are generated for frag procs.
BUG=skia:4182
Review URL: https://codereview.chromium.org/1298233002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1287763004
|
|
|
|
|
|
|
|
| |
Waiting a day or so to see if the blink-removal of SkDeferredCanvas sticks
BUG=skia:
Review URL: https://codereview.chromium.org/1269093002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1255943006
|
|
|
|
|
|
|
|
|
|
| |
I created these images using mandrill_16.png (already in
resources) and an online tool.
https://codereview.chromium.org/1287863004/
BUG=skia:
Review URL: https://codereview.chromium.org/1297723004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1304443002/ )
Reason for revert:
derek + Test-Win8-MSVC-ShuttleB-CPU-AVX2-x86_64-Debug
Original issue's description:
> Have DM manually encode its .png outputs.
>
> This eliminates some variability on various axes: different PNG encoders, different libpng versions, different formats (RGB, indexed), different unpremultiplication, different sRGB tags.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3cc0dfffb70c0bd08ed8899efcd2e98da86a6ec7
TBR=stephana@google.com,msarett@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1297383002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1264133003/ )
Reason for revert:
Seems to be breaking some blink tests.
https://storage.googleapis.com/chromium-layout-test-archives/linux_blink_rel/75812/layout-test-results/results.html
Original issue's description:
> SkCanvas::onDrawPicture() quick-reject
>
> R=reed@google.com,mtklein@google.com
>
> Committed: https://skia.googlesource.com/skia/+/48ed62b29d45e42b971aac8858da06781c93e6d7
>
> Committed: https://skia.googlesource.com/skia/+/d3d07245e29504dbffa0083e84ace5bab85853d4
TBR=mtklein@google.com,reed@google.com,mtklein@chromium.org,fmalita@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1301973002
|
|
|
|
|
|
|
|
| |
This eliminates some variability on various axes: different PNG encoders, different libpng versions, different formats (RGB, indexed), different unpremultiplication, different sRGB tags.
BUG=skia:
Review URL: https://codereview.chromium.org/1304443002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1297143004
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1292073003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1295773003
|
|
|
|
|
|
| |
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/1300093007
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1283193008
|
|
|
|
|
|
| |
TBR=robertphillips@google.com
Review URL: https://codereview.chromium.org/1297283003
|
|
|
|
|
|
|
|
| |
R=reed@google.com,mtklein@google.com
Committed: https://skia.googlesource.com/skia/+/48ed62b29d45e42b971aac8858da06781c93e6d7
Review URL: https://codereview.chromium.org/1264133003
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1294113003
|
|
|
|
|
|
|
|
| |
Factor out some of https://crrev.com/1227913008
BUG=skia:3030
Review URL: https://codereview.chromium.org/1298243002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1293973003
|
|
|
|
|
|
|
|
|
|
| |
We don't want to test small images on Gold because they are
not interested to look at. Instead, I wrote a unit test to
verify that scaling small images does not cause crashes.
BUG=skia:
Review URL: https://codereview.chromium.org/1287863004
|
|
|
|
|
|
| |
BUG=skia:2774
Review URL: https://codereview.chromium.org/1290223005
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1298233003
|
|
|
|
|
|
| |
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/1292623004
|
|
|
|
|
|
|
|
| |
Remember failed attempt https://codereview.chromium.org/1286093004/ ? I think this one is simpler and safer and even technically legal C++.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1296183004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1298213002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1289673004
|
|
|
|
|
|
|
|
| |
AutoFragmentChildProcAdvance class
BUG=skia:4182
Review URL: https://codereview.chromium.org/1301523003
|
|
|
|
|
|
|
|
| |
Use malloc-ed memory and the updateData() call instead.
BUG=skia:4215
Review URL: https://codereview.chromium.org/1288683004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1301523005/ )
Reason for revert:
DM is failing on gm tests on Android bots. Cause is not yet clear.
Original issue's description:
> Update skia to test with libpng to 1.6.10
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/f272bb03df9b86e7ea2cf23fb4d5cc56624e0118
TBR=djsollen@google.com,scroggo@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1298223002
|
|
|
|
|
|
|
|
| |
GrGLFragmentProcessor will be the root of a tree of GrGLFragmentProcessors that mirrors the GrFragmentProcessor's tree. This allows setData() to be called recursively (removing the responsibility from compose shader) and allows gl instances direct access to their children gl instances so they can emit their code.
BUG=skia:4182
Review URL: https://codereview.chromium.org/1287023009
|
|
|
|
|
|
|
|
| |
... to support subsets in SkImage!
BUG=skia:
Review URL: https://codereview.chromium.org/1301633002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1301523005
|
|
|
|
|
|
|
|
| |
BUG=skia:4142
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1284043004
Review URL: https://codereview.chromium.org/1284043004
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1288963004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1298983002
|
|
|
|
|
|
|
|
| |
portable -> default, and everyone gets an sk_ prefix.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1299013003
|
|
|
|
|
|
| |
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/1303533003
|
|
|
|
|
|
|
|
| |
patch from issue 1273033005 at patchset 120001 (http://crrev.com/1273033005#ps120001)
BUG=skia:
Review URL: https://codereview.chromium.org/1288323004
|
|
|
|
|
|
|
|
|
|
|
|
| |
IDWriteFamily::GetFamilyNames appears to be 'helpful' and removes
parts of family names that look like style names. Since the iterator
is supposed to return the actual names and not just the name the
platform thinks the name is (as getFamilyName does), try returning
the raw names when possible.
BUG=skia:4217
Review URL: https://codereview.chromium.org/1302573002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1293563003
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- SSE2 files are unfortunately now mixed-case, _SSE2 or _sse2.
- Adds lists for SSSE3 and SSE4 files.
- Remove SkDocument_PDF_None.cpp
- Remove a few more references to animator.
- Exclude private headers from HDRS.
- Formatting and notes.
BUG=skia:
Review URL: https://codereview.chromium.org/1298003007
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1298813002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Old flow to serialize a picture:
1) serialize picture ops
2) serialize all sub pictures recursively
3) flatten the rest of this picture into a buffer, deduping flattenable factories and typefaces as we go
4) serialize the factories and typefaces
5) serialize the bytes from 3)
This allows the data in step 5) to refer to the deduplicated factories and typefaces from step 4). But, each sub picture in step 2) is completely siloed, so they can't dedup with the parent picture or each other.
New flow:
1) serialize picture ops
2) flatten the rest of this picture into a buffer, deduping flattenable factories and typefaces as we go
3) dummy-serialize sub pictures into /dev/null, with the effect of adding any new typefaces to our dedup set
4) serialize the factories and typefaces
5) serialize the bytes from 2)
6) serialize all sub pictures recursively, with perfect deduplication because of step 3).
Now all typefaces in the top-level picture and all sub pictures recursively should end up deduplicated in the top-level typeface set.
Decoding changes are similar: we just thread through the top-level typefaces to the sub pictures. What's convenient / surprising is that this new code correctly reads old pictures if we just have each picture prefer its local typeface set over the top-level one: old pictures always just use their own typefaces, and new pictures always use the top-level ones.
BUG=skia:4092
Review URL: https://codereview.chromium.org/1233953004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the cases we have come across so far, native scaling has better performance
and correctness than sampling. If native scaling is supported we want to use
it.
Jpegs native scale rounds up.
Ex: An 11x11 image with sampleSize=8 scales to 2x2.
SkScaledCodec rounds down.
Ex: An 11x11 image with sampleSize=8 scales to 1x1.
Before the CL, we would choose to use SkScaledCodec because it scales closer to
the "ideal" scale.
I think we want to go with the native option as long as its within 1 of the
ideal value.
BUG=skia:
Review URL: https://codereview.chromium.org/1284243004
|
|
|
|
|
|
|
|
|
| |
The GrPaint's color takes into account shaders and color filters, so is a
more accurate picture of the color state.
BUG=chromium:511787
Review URL: https://codereview.chromium.org/1297053004
|
|
|
|
|
|
|
|
|
|
| |
Added comment about how computeInvariantOutput() is non-recursive in GrFragmentProcessor
Made isEqual() recursive in GrFragmentProcessor
BUG=skia:4182
Review URL: https://codereview.chromium.org/1287343005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1301603004
|