mtklein [Wed, 29 Jul 2015 14:49:40 +0000 (07:49 -0700)]
update C++11 docs
BUG=skia:
NOTRY=true
DOCS_PREVIEW= https://skia.org/dev/contrib/c++11?cl=
1263793002
Review URL: https://codereview.chromium.org/
1263793002
borenet [Wed, 29 Jul 2015 14:38:49 +0000 (07:38 -0700)]
Add builder_spec.py
Works like dm_flags.py and nanobench_flags.py; adds things like
GYP_DEFINES, additional environment variables, and build targets.
Required copying builder_name_schema from the tools/build repo.
BUG=skia:4132
Review URL: https://codereview.chromium.org/
1265623002
caryclark [Wed, 29 Jul 2015 13:58:40 +0000 (06:58 -0700)]
fix dstreadshuffle text portable gm
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1259093004
bsalomon [Wed, 29 Jul 2015 13:54:33 +0000 (06:54 -0700)]
Fix silly mistake in allocating geoms in TextBatch
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/
1259103004
mtklein [Wed, 29 Jul 2015 13:37:28 +0000 (06:37 -0700)]
Allow Srcs to veto Sinks based on their broad type.
This breaks Sinks down into three auto-detected types:
- GPU: anything that requests to be run in the GPU enclave
- Vector: anything that writes to the stream instead of the bitmap
- Raster: everything else
Some examples: gpu -> GPU, msaa16 -> GPU, 8888 -> raster, pdf -> vector,
svg -> vector, pipe-8888 -> raster, tiles_rt-gpu -> GPU
This lets image decoding sinks veto non-raster backends explicitly,
and can let particular GMs veto GPU or non-GPU sinks as they like.
BUG=skia:
Review URL: https://codereview.chromium.org/
1239953004
bsalomon [Wed, 29 Jul 2015 13:08:46 +0000 (06:08 -0700)]
Skip temp texture when config conversion test has already failed.
Review URL: https://codereview.chromium.org/
1261033005
caryclark [Wed, 29 Jul 2015 12:27:47 +0000 (05:27 -0700)]
make tests portable by using 565 compatible colors
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1262703002
bsalomon [Tue, 28 Jul 2015 22:39:45 +0000 (15:39 -0700)]
Don't read from unref'ed RT in GrContext::readSurfacePixels
BUG=chromium:514716
TBR=robertphillips@google.com
Review URL: https://codereview.chromium.org/
1258673009
bsalomon [Tue, 28 Jul 2015 20:26:15 +0000 (13:26 -0700)]
Move draw on upload decision in GrGpu
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
0341b4427e5f037e3b501ed6e57dfdb7b40f150e
Review URL: https://codereview.chromium.org/
1257073003
Mike Klein [Tue, 28 Jul 2015 20:01:24 +0000 (16:01 -0400)]
Revert "Move the rest of src headers used by include to include/private."
This reverts commit
d12e6ffa5cc1c1af47bf73c7c127d8d7f7443058.
Our Chrome roll canaries are failing with the dreaded
Ninja-says-there's-more-work-to-do message. I will break this up
smaller (if possible) and try again tomorrow.
BUG=skia:4126
Review URL: https://codereview.chromium.org/
1258293004 .
bsalomon [Tue, 28 Jul 2015 19:23:26 +0000 (12:23 -0700)]
Revert of Move draw on upload decision in GrGpu (patchset #4 id:50001 of https://codereview.chromium.org/
1257073003/)
Reason for revert:
breaking write pixels test on bots
Original issue's description:
> Move draw on upload decision in GrGpu
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
0341b4427e5f037e3b501ed6e57dfdb7b40f150e
TBR=robertphillips@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1260293004
caryclark [Tue, 28 Jul 2015 18:55:49 +0000 (11:55 -0700)]
gm missed portable text in textblobshader
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1263593004
mtklein [Tue, 28 Jul 2015 18:51:50 +0000 (11:51 -0700)]
Move the rest of src headers used by include to include/private.
$ git grep "../../src/" | grep include
now returns nothing.
BUG=skia:4126
No public API changes.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1261013003
wangyix [Tue, 28 Jul 2015 18:47:44 +0000 (11:47 -0700)]
Removed GrFragmentProcessor::fWillUseInputColor
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1259303003
caryclark [Tue, 28 Jul 2015 18:22:48 +0000 (11:22 -0700)]
gm fix missed portable gm text in samplerstress
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1261093002
caryclark [Tue, 28 Jul 2015 17:37:53 +0000 (10:37 -0700)]
make pixelsnap textblob* etc gm portable
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1263553002
joshualitt [Tue, 28 Jul 2015 17:20:08 +0000 (10:20 -0700)]
Change GrDefaultGeoProcFactory to a namespace
BUG=skia:
Review URL: https://codereview.chromium.org/
1257333002
joshualitt [Tue, 28 Jul 2015 17:01:18 +0000 (10:01 -0700)]
Cleanup GrDefaultGeoProcFactory localCoords
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1257193002
joshualitt [Tue, 28 Jul 2015 16:58:39 +0000 (09:58 -0700)]
Rename GrInOrderDrawBuffer to GrBufferedDrawTarget
BUG=skia:
Review URL: https://codereview.chromium.org/
1261033002
mtklein [Tue, 28 Jul 2015 16:54:52 +0000 (09:54 -0700)]
Move SkTHash.h to include/private.
include/views/SkOSWindow_Win.h includes it.
To move SkTHash.h to include/private, SkChecksum.h needs to go there too. To move SkChecksum.h to include/private, SkTLogic needs to go there too.
This adds a bunch of -Iinclude/private to tools.gyp I missed in the last CL.
No public API changes.
TBR=reed@google.com
BUG=skia:4126
Review URL: https://codereview.chromium.org/
1260613006
mtklein [Tue, 28 Jul 2015 16:35:19 +0000 (09:35 -0700)]
Fix a stray -Winconsistent-missing-override warning.
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/
1261043002
bsalomon [Tue, 28 Jul 2015 16:13:47 +0000 (09:13 -0700)]
Move draw on upload decision in GrGpu
BUG=skia:
Review URL: https://codereview.chromium.org/
1257073003
mtklein [Tue, 28 Jul 2015 15:55:14 +0000 (08:55 -0700)]
add -Iinclude/private anywhere we have -Isrc/core
I'll be moving headers from src/core to include/private, so this guarantees
that anyone who was finding them via -Isrc/core can now find them via
-Iinclude/private.
This is purely mechanical, mostly to preserve my sanity, so it's likely
(harmless) overkill.
Chromium's GYP and GN builds already set -Iinclude/private for Skia builds.
BUG=skia:4126
Review URL: https://codereview.chromium.org/
1265443002
mtklein [Tue, 28 Jul 2015 15:54:12 +0000 (08:54 -0700)]
update presubmit to know about include/private
BUG=skia:4126
Review URL: https://codereview.chromium.org/
1259953004
reed [Tue, 28 Jul 2015 14:35:14 +0000 (07:35 -0700)]
remove all guards for changes to drawBitmapRect / drawImageRect
BUG=skia:
Review URL: https://codereview.chromium.org/
1239193002
reed [Tue, 28 Jul 2015 13:00:50 +0000 (06:00 -0700)]
change getBounds to return 0000 iff there are zero points
This is a contract change for SkPath::getBounds(), which formally was defined to return 0,0,0,0 for a 1-point path, regardless of the coordinates of that point. This seems wacky/inconsistent, and was causing other bugs (incorrect bounds) when this was unioned with other rects.
Does anyone remember why we defined it this way?
BUG=513799
Review URL: https://codereview.chromium.org/
1261773002
borenet [Tue, 28 Jul 2015 13:00:36 +0000 (06:00 -0700)]
Fix android_launch_app after Gradle change
NOTRY=true
TBR=djsollen
BUG=skia:
Review URL: https://codereview.chromium.org/
1258253002
borenet [Tue, 28 Jul 2015 12:13:41 +0000 (05:13 -0700)]
android_install_app: Use lower-cased app name
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1261873002
caryclark [Tue, 28 Jul 2015 12:12:19 +0000 (05:12 -0700)]
compute split conic endpoints exactly
The divide by w can generate slightly erroneous results even
for t == 0 or t == 1. The error in turn defeats detecting
a point in common for a pair of curves that travel in
opposite directions.
Instead, special case endpoints when the t is 0 or 1.
TBR=reed@google.com
BUG=514118
Review URL: https://codereview.chromium.org/
1259513004
msarett [Mon, 27 Jul 2015 22:13:17 +0000 (15:13 -0700)]
Pass the destination pointer to next() in SkSwizzler
Per our discussion, we can make the swizzler simpler and more usable
for SkCodec and SkScanlineDecoder by only having a single version of
next() which takes a pointer to the srcRow and a pointer to the
dstRow.
BUG=skia:
Review URL: https://codereview.chromium.org/
1256373002
halcanary [Mon, 27 Jul 2015 20:13:03 +0000 (13:13 -0700)]
documentation: more API details and examples.
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/api/skpaint?cl=
1240893003
Review URL: https://codereview.chromium.org/
1240893003
junov [Mon, 27 Jul 2015 20:11:55 +0000 (13:11 -0700)]
Fixing src rect constraint support for drawImage with SkPicture
Follow-up to https://codereview.chromium.org/
1228083004
BUG=skia:
Review URL: https://codereview.chromium.org/
1256853004
reed [Mon, 27 Jul 2015 19:19:16 +0000 (12:19 -0700)]
remove pixel assert from ctable validator
BUG=514143
Review URL: https://codereview.chromium.org/
1252973003
joshualitt [Mon, 27 Jul 2015 19:13:14 +0000 (12:13 -0700)]
Cleanup Default Geo Proc API
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1253393002
mtklein [Mon, 27 Jul 2015 19:07:29 +0000 (12:07 -0700)]
No one calls SkXfermode::GetProc16
BUG=skia:
Review URL: https://codereview.chromium.org/
1253493002
mtklein [Mon, 27 Jul 2015 19:03:23 +0000 (12:03 -0700)]
Revert of Lay groundwork for SkOpts. (patchset #3 id:40001 of https://codereview.chromium.org/
1255193002/)
Reason for revert:
Chromium doesn't call SkGraphics::Init(). This setup won't work.
Original issue's description:
> Lay groundwork for SkOpts.
>
> This doesn't really do anything yet. It's just the CPU detection code, skeleton new .cpp files, and a few little .gyp tweaks.
>
> BUG=skia:4117
>
> Committed: https://skia.googlesource.com/skia/+/
ce2c5055cee5d5d3c9fc84c1b3eeed4b4d84a827
TBR=djsollen@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:4117
Review URL: https://codereview.chromium.org/
1261743002
joshualitt [Mon, 27 Jul 2015 18:41:15 +0000 (11:41 -0700)]
fix up GrImmediateDrawTarget.cpp
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1260023003
mtklein [Mon, 27 Jul 2015 18:08:28 +0000 (11:08 -0700)]
Remove sk_memcpy32
It's only implemented on x86, where the exisiting benchmark says memcpy() is
faster for all cases:
Timer overhead: 24ns
curr/maxrss loops min median mean max stddev samples config bench
10/10 MB 1 35.9µs 36.2µs 36.2µs 36.6µs 1% ▁▂▄▅▅▃█▄▄▅ nonrendering sk_memcpy32_100000
10/10 MB 13 2.27µs 2.28µs 2.28µs 2.29µs 0% █▄▃▅▃▁▃▅▁▄ nonrendering sk_memcpy32_10000
11/11 MB 677 91.6ns 95.9ns 94.5ns 99.4ns 3% ▅▅▅▅▅█▁▁▁▁ nonrendering sk_memcpy32_1000
11/11 MB 1171 20ns 20.9ns 21.3ns 23.4ns 6% ▁▁▇▃▃▃█▇▃▃ nonrendering sk_memcpy32_100
11/11 MB 1952 14ns 14ns 14.3ns 15.2ns 3% ▁▁██▁▁▁▁▁▁ nonrendering sk_memcpy32_10
11/11 MB 5 33.6µs 33.7µs 34.1µs 35.2µs 2% ▆▇█▁▁▁▁▁▁▁ nonrendering memcpy32_memcpy_100000
11/11 MB 18 2.12µs 2.22µs 2.24µs 2.39µs 5% ▂█▄▇█▄▇▁▁▁ nonrendering memcpy32_memcpy_10000
11/11 MB 1112 87.3ns 87.3ns 89.1ns 93.7ns 3% ▄██▄▁▁▁▁▁▁ nonrendering memcpy32_memcpy_1000
11/11 MB 2124 12.8ns 13.3ns 13.5ns 14.8ns 6% ▁▁▁█▃▃█▇▃▃ nonrendering memcpy32_memcpy_100
11/11 MB 3077 9ns 9.41ns 9.52ns 10.2ns 4% ▃█▁█▃▃▃▃▃▃ nonrendering memcpy32_memcpy_10
(Why? One fewer thing to port to SkOpts.)
BUG=skia:4117
Review URL: https://codereview.chromium.org/
1256763003
mtklein [Mon, 27 Jul 2015 17:52:33 +0000 (10:52 -0700)]
Lay groundwork for SkOpts.
This doesn't really do anything yet. It's just the CPU detection code, skeleton new .cpp files, and a few little .gyp tweaks.
BUG=skia:4117
Review URL: https://codereview.chromium.org/
1255193002
fmalita [Mon, 27 Jul 2015 17:27:28 +0000 (10:27 -0700)]
Make peekPixels() usable with raster surface snapshots
SkSurface_Raster snapshots do not lock their backing bitmaps when the
pixel ref is shared - they only lock on deep-copy.
But since for raster surfaces the pixels are always in memory, I think
it would be OK to also lock in the former case.
This allows for optimized (zero-copy) reads of raster surface snapshot
data.
R=reed@google.com
Review URL: https://codereview.chromium.org/
1256993002
bsalomon [Mon, 27 Jul 2015 14:00:00 +0000 (07:00 -0700)]
Steal refs from other TextBatch in onCombineIfPossible
Review URL: https://codereview.chromium.org/
1257683004
mtklein [Mon, 27 Jul 2015 13:12:05 +0000 (06:12 -0700)]
NEON has a ternary instruction.
Nothing seems to run any faster or slower, but it is terser.
BUG=skia:
Review URL: https://codereview.chromium.org/
1255913004
bsalomon [Mon, 27 Jul 2015 13:12:01 +0000 (06:12 -0700)]
Make allocation count in TextBatch implicit
Review URL: https://codereview.chromium.org/
1254903002
Brian Salomon [Mon, 27 Jul 2015 13:10:36 +0000 (09:10 -0400)]
Remove skia_arch_width, fold into skia_arch_type.
DOCS_PREVIEW= https://skia.org/?cl=
1228553010
BUG=skia:4042
R=borenet@google.com, mtklein@google.com
Review URL: https://codereview.chromium.org/
1228553010
wangyix [Fri, 24 Jul 2015 20:48:26 +0000 (13:48 -0700)]
Added GrGLBlend.h|cpp with helper function AppendPorterDuffBlend() in preparation for SkComposeShader gpu backend
BUG=skia:
Review URL: https://codereview.chromium.org/
1254833003
joshualitt [Fri, 24 Jul 2015 20:16:24 +0000 (13:16 -0700)]
mixed text blobs really draws LCD
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1261483002
djsollen [Fri, 24 Jul 2015 20:15:58 +0000 (13:15 -0700)]
Update Android Apps to use gradle
This CL replaces ant with gradle for the task of building APKs.
The primary driver of this change is that it now allow us to
develop and test our apps using Android Studio.
DOCS_PREVIEW= https://skia.org/?cl=
1215023017
Committed: https://skia.googlesource.com/skia/+/
425535f1626932e4e22f61a2571f9c3c2b1c5977
Review URL: https://codereview.chromium.org/
1215023017
caryclark [Fri, 24 Jul 2015 19:09:25 +0000 (12:09 -0700)]
rename portable_typeface_always to portable_typeface
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1257773002
joshualitt [Fri, 24 Jul 2015 18:24:31 +0000 (11:24 -0700)]
fix for GrAtlasTextContext occasionally crashes on mixed runs
TBR=bsalomon@google.com
BUG=510931
Review URL: https://codereview.chromium.org/
1252423002
caryclark [Fri, 24 Jul 2015 18:08:01 +0000 (11:08 -0700)]
make fontscalar gammatext lcdtext typeface verttext2 gm portable
Pass generic font names to tool util function to generate
platform specific fonts and gm test output by unique name.
R=bungeman@google.com
Review URL: https://codereview.chromium.org/
1256903002
joshualitt [Fri, 24 Jul 2015 17:37:00 +0000 (10:37 -0700)]
Minimize retrieving SkGlyph in GrTextContext
BUG=skia:
Review URL: https://codereview.chromium.org/
1257603005
robertphillips [Fri, 24 Jul 2015 17:20:45 +0000 (10:20 -0700)]
Fix Ganesh drawAtlas bug with quad colors
Ganesh was not expanding the quad colors to vertex colors before calling drawVertices.
The new GM would've caught this bug and reveals Ganesh's limitations re the various xfer modes used with drawAtlas (i.e., w/o AA Ganesh only supports kModulate, w/ AA Ganesh only supports the coefficient-based xfer modes).
Review URL: https://codereview.chromium.org/
1254943002
bsalomon [Fri, 24 Jul 2015 15:10:43 +0000 (08:10 -0700)]
Set preallocated TextBatch geometries to 4 rather than 32
Review URL: https://codereview.chromium.org/
1256843003
hendrikw [Fri, 24 Jul 2015 13:52:40 +0000 (06:52 -0700)]
skia: wrong calling convention on eglGetPlatformDisplayEXT
When attempting to run the release compile of nanobench on windows, I would
immediately crash due to c++'s buffer security check.
This was caused by calling the function with the wrong calling
convention. I'm not sure how this ever worked for anyone.
Anyway, fix is to use eglext.h's version of the function definition.
Review URL: https://codereview.chromium.org/
1250383002
caryclark [Thu, 23 Jul 2015 21:07:58 +0000 (14:07 -0700)]
make dftext gm not overlap itself
TBR=jvanverth@google.com
Review URL: https://codereview.chromium.org/
1253643004
caryclark [Thu, 23 Jul 2015 20:48:31 +0000 (13:48 -0700)]
make fontcache textblob gm portable
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1245123006
caryclark [Thu, 23 Jul 2015 19:40:22 +0000 (12:40 -0700)]
fix path ops fuzz buster
Mark collapsed segments as done and remove collapsed
segment references from the coincidence array.
Also add test names to global debugging.
R=fmalita@chromium.org
BUG=512592
Review URL: https://codereview.chromium.org/
1250293002
caryclark [Thu, 23 Jul 2015 19:38:59 +0000 (12:38 -0700)]
make dftext the same on Linux and Android
Use the same font sizes on both platforms so that the gm images
are easier to compare one to the other.
TBR=jvanverth@google.com
Review URL: https://codereview.chromium.org/
1253573006
bsalomon [Thu, 23 Jul 2015 19:22:19 +0000 (12:22 -0700)]
Remove test of illegal behavior that modifies a texture backing an SkImage.
Review URL: https://codereview.chromium.org/
1244143004
msarett [Thu, 23 Jul 2015 18:57:13 +0000 (11:57 -0700)]
Fix Android x86 compile on mac
It appears I failed to fully disable it on the first attempt.
BUG=skia:
Review URL: https://codereview.chromium.org/
1249083004
hendrikw [Thu, 23 Jul 2015 18:56:40 +0000 (11:56 -0700)]
Skia: ANGLE Roll
Includes fix for vertex index.
BUG=510585
Review URL: https://codereview.chromium.org/
1246143005
caryclark [Thu, 23 Jul 2015 18:52:01 +0000 (11:52 -0700)]
fix typeface leak
TBR=bungeman@google.com
Review URL: https://codereview.chromium.org/
1239303003
bsalomon [Thu, 23 Jul 2015 18:45:02 +0000 (11:45 -0700)]
Fix ImageNewSurface test on S4.
This still leaves the SkImage_NewFromTexture broken.
Review URL: https://codereview.chromium.org/
1253513004
robertphillips [Thu, 23 Jul 2015 18:35:08 +0000 (11:35 -0700)]
Fix variable shadowing in SkMorphologyImageFilter
Review URL: https://codereview.chromium.org/
1245883005
jvanverth [Thu, 23 Jul 2015 18:14:29 +0000 (11:14 -0700)]
Update fallback rsqrt implementation to use optimal constants.
Improves max relative error from 0.
00175126 to 0.
000650197.
Also add unit tests to check error bounds.
BUG=chromium:511458
Review URL: https://codereview.chromium.org/
1251423002
joshualitt [Thu, 23 Jul 2015 17:58:48 +0000 (10:58 -0700)]
fix comment on GrBatchTextStrike
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1252783002
caryclark [Thu, 23 Jul 2015 16:11:58 +0000 (09:11 -0700)]
fix typo
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1245993003
mtklein [Thu, 23 Jul 2015 15:37:02 +0000 (08:37 -0700)]
Have Windows use _mm_rsqrt_ss too.
Tidy up a little while I'm in here:
1) SIMD headers are now included by SkTypes.h as appropriate.
2) _mm_cvtss_f32() is pithier and generates the same code.
Looks like this is the only code checking for SSE wrong. After this CL:
~/skia (sse) $ git grep __SSE
include/core/SkPreConfig.h: #if defined(__SSE4_2__)
include/core/SkPreConfig.h: #elif defined(__SSE4_1__)
include/core/SkPreConfig.h: #elif defined(__SSE3__)
include/core/SkPreConfig.h: #elif defined(__SSE2__)
every other check is in SkPreConfig.h where it belongs.
This is going to affect some GMs subtly on Windows.
BUG=chromium:511458
No public API changes.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1248503004
joshualitt [Thu, 23 Jul 2015 15:30:25 +0000 (08:30 -0700)]
Small tweak to TextBlobCacheTest
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1251383003
joshualitt [Thu, 23 Jul 2015 15:09:35 +0000 (08:09 -0700)]
fix memory leak in TextBlobCacheTest
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1252593002
bsalomon [Thu, 23 Jul 2015 15:07:21 +0000 (08:07 -0700)]
Attempt to somewhat simplify GrContext::readSurfacePixels interaction with GrGpu.
Review URL: https://codereview.chromium.org/
1255483005
rmistry [Thu, 23 Jul 2015 15:02:13 +0000 (08:02 -0700)]
Fix formatting in the CQ keywords markdown
BUG=skia:
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=
1247323002
TBR=jcgregorio
Review URL: https://codereview.chromium.org/
1247323002
wangyix [Thu, 23 Jul 2015 15:02:09 +0000 (08:02 -0700)]
Name of primitive processor will now be printed in generated shader code
BUG=skia:
Review URL: https://codereview.chromium.org/
1253513003
joshualitt [Thu, 23 Jul 2015 15:00:33 +0000 (08:00 -0700)]
Revert "Update Android Apps to use gradle"
This reverts commit
425535f1626932e4e22f61a2571f9c3c2b1c5977.
NOTRY=true
NOTREECHECKS=true
TBR=djsollen@google.com
BUG=skia:
DOCS_PREVIEW= https://skia.org/?cl=
1250233002
Review URL: https://codereview.chromium.org/
1250233002
wangyix [Wed, 22 Jul 2015 22:08:53 +0000 (15:08 -0700)]
Added GrGLFragmentProcessor::EmitArgs struct for use with emitCode()
BUG=skia:
Review URL: https://codereview.chromium.org/
1251173002
borenet [Wed, 22 Jul 2015 20:29:25 +0000 (13:29 -0700)]
Add skia_use_android_framework_defines GYP_DEFINE
BUG=skia:4082
Review URL: https://codereview.chromium.org/
1243383002
robertphillips [Wed, 22 Jul 2015 18:54:44 +0000 (11:54 -0700)]
Misc cleanup
This is split off of https://codereview.chromium.org/
1225923010/ (Start tightening correspondence betweeen GrDrawContext and GrRenderTarget). It:
fixes some style nits
replaces some passing of GrContext with GrTextureProvider & GrDrawContext
does a bit of the finer grained creation of GrDrawContexts
Review URL: https://codereview.chromium.org/
1245183002
djsollen [Wed, 22 Jul 2015 18:33:24 +0000 (11:33 -0700)]
Update Android Apps to use gradle
This CL replaces ant with gradle for the task of building APKs.
The primary driver of this change is that it now allow us to
develop and test our apps using Android Studio.
DOCS_PREVIEW= https://skia.org/?cl=
1215023017
Review URL: https://codereview.chromium.org/
1215023017
joshualitt [Wed, 22 Jul 2015 18:00:32 +0000 (11:00 -0700)]
Fix up textblobcachetest
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1250003003
mtklein [Wed, 22 Jul 2015 17:52:53 +0000 (10:52 -0700)]
565 support for SIMD xfermodes
This uses the most basic approach possible:
- to load an Sk4px from 565, convert to SkPMColors on the stack serially then load those SkPMColors.
- to store an Sk4px to 565, store to SkPMColors on the stack then convert to 565 serially.
Clearly, we can optimize these loads and stores. That's a TODO.
The code using SkPMFloat is the same idea but a little more long-term viable, as we're only operating on one pixel at a time anyway. We could probably write 565 <-> SkPMFloat methods, but I'd rather not until it's really compelling.
The speedups are varied but similar across SSE and NEON: a few uninteresting, many 50% faster, some 2x faster, and SoftLight ~4x faster.
This will cause minor GM diffs, but I don't think any layout test changes.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
942930dcaa51f66d82cdaf46ae62efebd16c8cd0
Committed: https://skia.googlesource.com/skia/+/
860dcaa2ddfdadc050af4f943a84a9d499315066
Review URL: https://codereview.chromium.org/
1245673002
fmalita [Wed, 22 Jul 2015 17:23:01 +0000 (10:23 -0700)]
Add drawImage{Rect,} support to SkDebugCanvas
R=robertphillips@google.com,mtklein@google.com
Review URL: https://codereview.chromium.org/
1253473002
wangyix [Wed, 22 Jul 2015 17:21:17 +0000 (10:21 -0700)]
Moved GrGLFragmentProcessor definition to its own file
BUG=skia:
Review URL: https://codereview.chromium.org/
1246193002
bsalomon [Wed, 22 Jul 2015 17:17:26 +0000 (10:17 -0700)]
Remove some redundant fields from BitmapTextBatch (and rename to TextBatch).
Review URL: https://codereview.chromium.org/
1244093004
hendrikw [Wed, 22 Jul 2015 16:30:08 +0000 (09:30 -0700)]
skia: GrGLAssembleGLInterface update load chromium extension functions
Command buffer will expose GL_CHROMIUM_framebuffer_multisample and
GL_CHROMIUM_map_sub, added support for these to enable interface
validation to succeed.
Review URL: https://codereview.chromium.org/
1248853003
msarett [Wed, 22 Jul 2015 16:08:28 +0000 (09:08 -0700)]
Fix Android framework build
The Android framework was failing on conditions in
libjpeg-turbo.gyp, even though libjpeg-turbo is listed
in dependencies! for the framework (maybe because I
forgot to add export_dependent_settings!). This is fixed
by rearranging the gyp file.
BUG=skia:
Review URL: https://codereview.chromium.org/
1249003002
jvanverth [Wed, 22 Jul 2015 15:41:51 +0000 (08:41 -0700)]
Fix tile drop-out on S4 for texture decal mode.
Switch to use highp on interpolants.
Also removes some unnecessary formatting.
BUG=skia:3381
Review URL: https://codereview.chromium.org/
1245703004
borenet [Wed, 22 Jul 2015 15:19:25 +0000 (08:19 -0700)]
Coverage: add a trace for not_covered_lines per file
Example output:
{
"build_number": "2",
"gitHash": "
71f5db892ee7ed9fbb573da28ccb906e18ba24a4",
"key": {
"arch": "x86_64",
"compiler": "Clang",
"configuration": "Coverage",
"cpu_or_gpu": "CPU",
"cpu_or_gpu_value": "AVX2",
"model": "GCE",
"os": "Ubuntu"
},
"results": {
"dm_DMGpuSupport_h": {
"coverage": {
"lines_not_covered": 6,
"options": {
"dir": "dm",
"fullname": "dm/DMGpuSupport.h",
"source_type": "coverage"
},
"percent": 33.
33333333333333
}
},
"dm_DMJsonWriter_cpp": {
......
BUG=skia:2430
Review URL: https://codereview.chromium.org/
1246043002
scroggo [Wed, 22 Jul 2015 14:16:20 +0000 (07:16 -0700)]
Add the ability to decode a subset to SkCodec
This allows codecs that support subsets natively (i.e. WEBP) to do so.
Add a field on SkCodec::Options representing the subset.
Add a method on SkCodec to find a valid subset which approximately
matches a desired subset.
Implement subset decodes in SkWebpCodec.
Add a test in DM for decoding subsets.
Notice that we only start on even boundaries. This is due to the
way libwebp's API works. SkWEBPImageDecoder does not take this into
account, which results in visual artifacts.
FIXME: Subsets with scaling are not pixel identical, but close. (This
may be fine, though - they are not perceptually different. We'll just
need to mark another set of images in gold as valid, once
https://skbug.com/4038 is fixed, so we can tests scaled webp without
generating new images on each run.)
Review URL: https://codereview.chromium.org/
1240143002
fmalita [Wed, 22 Jul 2015 12:56:16 +0000 (05:56 -0700)]
Fix SkCanvas::wouldOverwriteEntireSurface() contains test
R=reed@google.com,robertphillips@google.com,bsalomon@google.com
Review URL: https://codereview.chromium.org/
1244093005
rmistry [Wed, 22 Jul 2015 11:29:38 +0000 (04:29 -0700)]
Fix CQ keyword and add more detail
BUG=skia:
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=
1246393002
TBR=jcgregorio
Review URL: https://codereview.chromium.org/
1246393002
mtklein [Wed, 22 Jul 2015 01:34:42 +0000 (18:34 -0700)]
Revert of 565 support for SIMD xfermodes (patchset #4 id:60001 of https://codereview.chromium.org/
1245673002/)
Reason for revert:
NEON 565 gold images have gone ugly. This is what I get for writing and testing SSE and just writing NEON.
E.g. colortype_xfermodes, dstreadshuffle, bigbitmaprect, pictures, textbloblooper, aaxfermodes (only Plus)
Original issue's description:
> 565 support for SIMD xfermodes
>
> This uses the most basic approach possible:
> - to load an Sk4px from 565, convert to SkPMColors on the stack serially then load those SkPMColors.
> - to store an Sk4px to 565, store to SkPMColors on the stack then convert to 565 serially.
>
> Clearly, we can optimize these loads and stores. That's a TODO.
>
> The code using SkPMFloat is the same idea but a little more long-term viable, as we're only operating on one pixel at a time anyway. We could probably write 565 <-> SkPMFloat methods, but I'd rather not until it's really compelling.
>
> The speedups are varied but similar across SSE and NEON: a few uninteresting, many 50% faster, some 2x faster, and SoftLight ~4x faster.
>
> This will cause minor GM diffs, but I don't think any layout test changes.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
942930dcaa51f66d82cdaf46ae62efebd16c8cd0
>
> Committed: https://skia.googlesource.com/skia/+/
860dcaa2ddfdadc050af4f943a84a9d499315066
TBR=msarett@google.com,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1248893004
mtklein [Wed, 22 Jul 2015 00:23:39 +0000 (17:23 -0700)]
565 support for SIMD xfermodes
This uses the most basic approach possible:
- to load an Sk4px from 565, convert to SkPMColors on the stack serially then load those SkPMColors.
- to store an Sk4px to 565, store to SkPMColors on the stack then convert to 565 serially.
Clearly, we can optimize these loads and stores. That's a TODO.
The code using SkPMFloat is the same idea but a little more long-term viable, as we're only operating on one pixel at a time anyway. We could probably write 565 <-> SkPMFloat methods, but I'd rather not until it's really compelling.
The speedups are varied but similar across SSE and NEON: a few uninteresting, many 50% faster, some 2x faster, and SoftLight ~4x faster.
This will cause minor GM diffs, but I don't think any layout test changes.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
942930dcaa51f66d82cdaf46ae62efebd16c8cd0
Review URL: https://codereview.chromium.org/
1245673002
herb [Tue, 21 Jul 2015 22:36:25 +0000 (15:36 -0700)]
Change the GlyphCache to use a hash table instead of doing its own ad-hoc
hashing. This change appears to be performance neutral.
BUG=skia:
Review URL: https://codereview.chromium.org/
1216983003
hendrikw [Tue, 21 Jul 2015 21:50:07 +0000 (14:50 -0700)]
ANGLE deps roll
If we ever want to allow the command buffer as a skia gles2 backend,
we need a more up to date version of ANGLE, specifically there are
4 defines that differ between newer and older versions of ANGLE which
we use in skia, I've updated these in this change.
I'm not quite sure if what I've done for the 'angle_path' is correct,
I tried setting it to a path relative to skia, and to '<(DEPTH)', both
of which do not compile correctly, only '../' worked.
Committed: https://skia.googlesource.com/skia/+/
db0b1e796ddbd08e6be8a666537318b1c0e2ce56
Review URL: https://codereview.chromium.org/
1244843003
bungeman [Tue, 21 Jul 2015 21:14:30 +0000 (14:14 -0700)]
Compile with VS2015.
Visual Studio 2015 has additional warnings around noexcept and
disabling exceptions, which can be worked around with the
(undocumented) _HAS_EXCEPTIONS macro.
Visual Studio 2013 and 2015 have roundf in math.h, so use it to
avoid extra work and casts.
We avoid using cmath, as it undefs isfinite on gcc, but Visual Studio
2015 no longer provides overloads of copysign from math.h (which is
actually correct). As a result, use copysignf (which is available in
math.h in 2013 and 2015) directly.
Review URL: https://codereview.chromium.org/
1244173005
ericrk [Tue, 21 Jul 2015 21:06:16 +0000 (14:06 -0700)]
Revert of Bilinear optimization for 1D convolution. (patchset #5 id:200001 of https://codereview.chromium.org/
1216623003/)
Reason for revert:
Ok, I am now seeing a couple issues. going to revert and investigate further.
Original issue's description:
> Bilinear optimization for 1D convolution.
>
> Splits GrGLConvolutionEffect into GrGLBilerpConvolutionEffect and
> GrGLBoundedConvolutionEffect. When doing a non-bounded convolution we now
> always use the GrGLBilerpConvolutionEffect which uses bilinear filtering to
> perform half as many samples in the texture.
>
> BUG=skia:3986
>
> Committed: https://skia.googlesource.com/skia/+/
91abe10af417148939548551e210c001022d3bda
>
> Committed: https://skia.googlesource.com/skia/+/
0f38612b0facf585854aba4556433b858cbf7da8
TBR=bsalomon@google.com,senorblanco@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:3986
Review URL: https://codereview.chromium.org/
1247063005
hendrikw [Tue, 21 Jul 2015 21:06:10 +0000 (14:06 -0700)]
Revert of skia: ANGLE deps roll (patchset #1 id:1 of https://codereview.chromium.org/
1244843003/)
Reason for revert:
Compile error that the try bots didn't catch :(
Original issue's description:
> ANGLE deps roll
>
> If we ever want to allow the command buffer as a skia gles2 backend,
> we need a more up to date version of ANGLE, specifically there are
> 4 defines that differ between newer and older versions of ANGLE which
> we use in skia, I've updated these in this change.
>
> I'm not quite sure if what I've done for the 'angle_path' is correct,
> I tried setting it to a path relative to skia, and to '<(DEPTH)', both
> of which do not compile correctly, only '../' worked.
>
> Committed: https://skia.googlesource.com/skia/+/
db0b1e796ddbd08e6be8a666537318b1c0e2ce56
TBR=bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/
1245223007
hendrikw [Tue, 21 Jul 2015 20:18:27 +0000 (13:18 -0700)]
ANGLE deps roll
If we ever want to allow the command buffer as a skia gles2 backend,
we need a more up to date version of ANGLE, specifically there are
4 defines that differ between newer and older versions of ANGLE which
we use in skia, I've updated these in this change.
I'm not quite sure if what I've done for the 'angle_path' is correct,
I tried setting it to a path relative to skia, and to '<(DEPTH)', both
of which do not compile correctly, only '../' worked.
Review URL: https://codereview.chromium.org/
1244843003
ericrk [Tue, 21 Jul 2015 20:15:47 +0000 (13:15 -0700)]
Bilinear optimization for 1D convolution.
Splits GrGLConvolutionEffect into GrGLBilerpConvolutionEffect and
GrGLBoundedConvolutionEffect. When doing a non-bounded convolution we now
always use the GrGLBilerpConvolutionEffect which uses bilinear filtering to
perform half as many samples in the texture.
BUG=skia:3986
Committed: https://skia.googlesource.com/skia/+/
91abe10af417148939548551e210c001022d3bda
Review URL: https://codereview.chromium.org/
1216623003
caryclark [Tue, 21 Jul 2015 20:11:26 +0000 (13:11 -0700)]
make mixedtextblobs gm portable
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1250753004