| Commit message (Collapse) | Author | Age |
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1282723004
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1275223003
|
|
|
|
|
|
|
|
| |
Motivation: this might catch future issues like this.
BUG=skia:4181
Review URL: https://codereview.chromium.org/1287443002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1273803004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1285593002
|
|
|
|
|
|
| |
BUG=chromium:505316
Review URL: https://codereview.chromium.org/1282143002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1278323002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1281123002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1276383003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:20001 of https://codereview.chromium.org/1280073002/ )
Reason for revert:
Compile failures, e.g.
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm7-Debug-Android_NoNeon/builds/1427/steps/build%20most/logs/stdio
Original issue's description:
> Update libwebp and resume testing scaled webp
>
> libwebp has a fix for [1]. Update to the commit that contains the fix.
>
> Update libwebp.gypi, corresponding to libwebp's latest makefile.
>
> Turn back on DM testing for scaled webp, now that it should no longer
> use uninitialized memory.
>
> Fix a warning in config.h
>
> [1] https://code.google.com/p/webp/issues/detail?id=254
>
> BUG=skia:4038
>
> Committed: https://skia.googlesource.com/skia/+/0575d3e6c272744a66ab3281f9871366717df339
TBR=djsollen@google.com,msarett@google.com,scroggo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:4038
Review URL: https://codereview.chromium.org/1278173004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
libwebp has a fix for [1]. Update to the commit that contains the fix.
Update libwebp.gypi, corresponding to libwebp's latest makefile.
Turn back on DM testing for scaled webp, now that it should no longer
use uninitialized memory.
Fix a warning in config.h
[1] https://code.google.com/p/webp/issues/detail?id=254
BUG=skia:4038
Review URL: https://codereview.chromium.org/1280073002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1276333004
|
|
|
|
|
|
| |
Committed: https://skia.googlesource.com/skia/+/b091c44c33edbad3466601b29d60bc2eaa4453fa
Review URL: https://codereview.chromium.org/1274013003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1274013003/ )
Reason for revert:
Build failures
Original issue's description:
> Move GrDrawAtlasBatch to separate file
>
> Committed: https://skia.googlesource.com/skia/+/b091c44c33edbad3466601b29d60bc2eaa4453fa
TBR=joshualitt@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1276173003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1274013003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1277233002
|
|
|
|
|
|
|
|
|
|
| |
Moves drawAtlas setup into its own method in GrDrawContext, and adds
DrawAtlasBatch.
Uses pre-built index buffer for quads.
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/1277933003
|
|
|
|
|
|
|
|
| |
BUG=skia:
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1276663003
Review URL: https://codereview.chromium.org/1276663003
|
|
|
|
|
|
|
|
| |
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
Review URL: https://codereview.chromium.org/1271873002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1280653003
|
|
|
|
|
|
| |
BUG=skia:4117
Review URL: https://codereview.chromium.org/1273203002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1276913002
|
|
|
|
|
|
|
|
|
|
|
|
| |
We called new float[...]. The pointer returned should be aligned for float writes.
See https://uberchromegw.corp.google.com/i/client.skia/builders/Test-iOS-Clang-iPad4-GPU-SGX554-Arm7-Release
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/30e4d75c3aebf49b179c265a91adbb6012970d2b
Review URL: https://codereview.chromium.org/1270403006
|
|
|
|
|
|
|
|
|
|
| |
We called new float[...]. The pointer returned should be aligned for float writes.
See https://uberchromegw.corp.google.com/i/client.skia/builders/Test-iOS-Clang-iPad4-GPU-SGX554-Arm7-Release
BUG=skia:
Review URL: https://codereview.chromium.org/1270403006
|
|
|
|
|
|
| |
TBR=reed@google.com
Review URL: https://codereview.chromium.org/1280853002
|
|
|
|
|
|
| |
TBR=bungeman@google.com
Review URL: https://codereview.chromium.org/1276133002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Will regress behavior on gold on test32bfv4.bmp, where we
will no longer fix transparent decodes.
TODO: Start fixing transparent decodes again, or decide
that we don't want to fix them and remove isTransparent
from SkSwizzler. I think this may become more clear when I
start implementing the scanline decoder.
BUG=skia:
Review URL: https://codereview.chromium.org/1258863008
|
|
|
|
|
|
| |
BUG=skia:3257
Review URL: https://codereview.chromium.org/1277893002
|
|
|
|
|
|
| |
TBR=mtklein,bsalomon
Review URL: https://codereview.chromium.org/1274963003
|
|
|
|
|
|
|
|
| |
When webp decoding to 565, swap the colors so they look right.
BUG=skia:4174
Review URL: https://codereview.chromium.org/1280693003
|
|
|
|
|
|
|
|
| |
BUG=skia:
NOTRY=True
TBR=
Review URL: https://codereview.chromium.org/1277963002
|
|
|
|
|
|
|
|
| |
As I begin to wade in here, it's nice to remove as much code as possible.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1277953002
|
|
|
|
|
|
|
|
| |
Can't hurt to simplify a little.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1274513003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1275543005
|
|
|
|
|
|
|
|
|
|
|
| |
This requires adding the stroke info to the cache key, and doing the
stroking and dashing before rendering as triangles.
BUG=skia:3755
Committed: https://skia.googlesource.com/skia/+/29e0d3f267a03546f236023347cdb4595ece2fd1
Review URL: https://codereview.chromium.org/1275553002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1272713005
|
|
|
|
|
|
| |
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/1275083002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1274763008
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Pretty vanilla translation. I cleaned up who calls whom a little.
Used to be utils -> opts -> utils, now it's just utils -> opts.
I may follow up with a pass over the NEON code for readability
and to clean up dead code.
This turns on NEON A8->R11EAC conversion for ARMv8.
Unit tests which now hit the NEON code still pass.
I can't find any related bench.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1273103002
|
|
|
|
|
|
| |
Cannot land before Blink https://codereview.chromium.org/1274673002/ (Remove reliance on legacy Skia SkPoint3 ctors in FELighting.cpp)
Review URL: https://codereview.chromium.org/1269383003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Prepare_rt_for_external_access() grabs the render target and then fires
access notifications.
But the notification handlers may trigger copy-on-write, causing the
returned render target to be stale (pointing at the detached snapshot).
We should grab the render target after firing notifications.
R=reed@google.com,bsalomon@google.com
Review URL: https://codereview.chromium.org/1276713002
|
|
|
|
|
|
| |
Also, new -> SkNEW.
Review URL: https://codereview.chromium.org/1280433002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1272743003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Although we initially discussed not supporting 565, SkCodec needs to
support Android's BitmapFactory and BitmapRegionDecoder, which need to
support 565.
We could instead implement 565 on top of SkCodec, but this would require
more memory and run more slowly.
An open question is whether to support dithering, and how. In order to
support dithering, we need to pass y to RowProc, which I believe means
we will need to pass y to SkSwizzler::swizzle().
I dislike having an option which is typically ignored - SkImageDecoder
allows you to turn on dithering when decoding to any color type, but it
is only meaningful only if the output color type is 565.
We could also make the client implement dithering, which I believe would
mean forcing them to do the conversion to 565 (since we have already
thrown away data when we convered to 565 without dithering).
BUG=skia:3257
Review URL: https://codereview.chromium.org/1277593002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1279733002
|
|
|
|
|
|
|
|
|
|
| |
The existing instructions seem misleading.
BUG=skia:
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1275753002
Review URL: https://codereview.chromium.org/1275753002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1276633003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Procedure:
$ platform_tools/android/toolchains/ndk-r10c-mips-darwin_v14/bin/mipsel-linux-android-gcc -dM -E - < /dev/null | sort > vanilla.mips
$ platform_tools/android/toolchains/ndk-r10c-mips-darwin_v14/bin/mipsel-linux-android-gcc -mdsp -dM -E - < /dev/null | sort > dsp.mips
$ platform_tools/android/toolchains/ndk-r10c-mips-darwin_v14/bin/mipsel-linux-android-gcc -mdspr2 -dM -E - < /dev/null | sort > dspr2.mips
$ diff vanilla.mips dsp.mips
239a240,241
> #define __mips_dsp 1
> #define __mips_dsp_rev 1
$ diff vanilla.mips dspr2.mips
239a240,242
> #define __mips_dsp 1
> #define __mips_dsp_rev 2
> #define __mips_dspr2 1
So, defined(__mips_dsp) -> SK_MIPS_HAS_DSP, defined(__mips_dspr2) -> SK_MIPS_HAS_DSPR2.
BUG=skia:
Review URL: https://codereview.chromium.org/1274873002
|
|
|
|
|
|
|
|
|
|
|
| |
Change https://codereview.chromium.org/1261643004 renamed a function
parameter so that some code became unreachable. This renames that
parameter back.
R=bsalomon@google.com
BUG=skia:4169
Review URL: https://codereview.chromium.org/1271353003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1269413006
|