| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
|
| |
This reverts commit 4cf9e7e14797f87f08b2078fa1afaf731cbf8f52.
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/1284173002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1278643006
|
|
|
|
|
|
|
|
|
|
|
| |
Compile SkJpegCodec and SkImageDecoder_libjpeg with
chromium's libjpeg-turbo. SkImageDecoder_libjpeg still uses
libjpeg on Android and the Android framework. SkJpegCodec is
still not compiled on the Android framework.
BUG=skia:
Review URL: https://codereview.chromium.org/1275773004
|
|
|
|
|
|
| |
BUG=519581
Review URL: https://codereview.chromium.org/1284953002
|
|
|
|
|
|
|
|
|
|
|
| |
To disable PDF:
GYP_DEFINES='skia_pdf=0' bin/sync-and-gyp
ninja -C out/Debug dm SampleApp
When disabled, SkDocument::CreatePDF() always returns NULL.
Review URL: https://codereview.chromium.org/1279123007
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit eedf0fb20489bc6c3706539dd6ab499f41cbe048.
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/1270313005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1289483002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1282893002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Local SKP nanobenching ranges SSE between 1.05x and 0.87x, much more heavily weighted toward <1.0x ratios (speedups).
I profiled the top five regressions (1.05x-1.01x) and they look like noise. Will follow up after broad bot results.
NEON looks similar but less extreme than SSE changes, ranging between 1.02x and 0.95x, again mostly speedups in 0.99x-0.97x range.
The old code trifurcated into black, opaque-but-not-black, and general versions as a function of the constant src color. I did not see a significant difference between general and opaque-but-not-black, and I don't think a black version would be faster using SIMD. So we have here just one version of the code, the general version.
Somewhat fantastically, I see no pixel diffs on GMs or SKPs.
I will be following up with more CLs for the other procs called by SkBlitMask.
BUG=skia:
Review URL: https://codereview.chromium.org/1278253003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1275003004
|
|
|
|
|
|
|
| |
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1282883002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1279303002
|
|
|
|
|
|
|
|
| |
TBR=mtklein@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1282333002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
empty (patchset #1 id:1 of https://codereview.chromium.org/1284693002/ )
Reason for revert:
Breaking DEPS roll on Assert in SkTextBlobBuilder::TightRunBounds.
Original issue's description:
> [TextBlob] Fall back to TightRunBounds when the font bounds are empty
>
> Empty font bounds are likely an indication of a font bug. As a best
> effort, we can use TightRunBounds in this easily detectable case.
>
> BUG=507022
> R=reed@google.com,bungeman@google.com
>
> Committed: https://skia.googlesource.com/skia/+/d6b99cc6b84b3ec864221cbe9945d203bd9eb072
TBR=bungeman@google.com,reed@google.com,fmalita@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=507022
Review URL: https://codereview.chromium.org/1283853002
|
|
|
|
|
|
| |
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/1274513005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1282283002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1287433003
|
|
|
|
|
|
| |
TBR=mtklein
Review URL: https://codereview.chromium.org/1277433005
|
|
|
|
|
|
|
|
|
|
| |
Empty font bounds are likely an indication of a font bug. As a best
effort, we can use TightRunBounds in this easily detectable case.
BUG=507022
R=reed@google.com,bungeman@google.com
Review URL: https://codereview.chromium.org/1284693002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1282723004
|
|
|
|
|
|
|
|
| |
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/1281123002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1276383003
|
|
|
|
|
|
| |
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:
Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
Review URL: https://codereview.chromium.org/1271873002
|
|
|
|
|
|
| |
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=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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|