aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* move Stroke Rect and AAFill Rect to their own fileGravatar joshualitt2015-08-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1282723004
* Ensure that android "release" apps are signed with debug keyGravatar djsollen2015-08-10
| | | | Review URL: https://codereview.chromium.org/1275223003
* SkPDF: add assert for that SkWStream behaves.Gravatar halcanary2015-08-10
| | | | | | | | Motivation: this might catch future issues like this. BUG=skia:4181 Review URL: https://codereview.chromium.org/1287443002
* Use a factory function to create GrPipeline and remove unused memberGravatar bsalomon2015-08-10
| | | | Review URL: https://codereview.chromium.org/1273803004
* Wire up testcreate function for GrDrawAtlasBatchGravatar joshualitt2015-08-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1285593002
* Remove an unused function.Gravatar thakis2015-08-10
| | | | | | BUG=chromium:505316 Review URL: https://codereview.chromium.org/1282143002
* api helpers inspired by blink use-casesGravatar reed2015-08-09
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1278323002
* remove unneeded _DRAWIMAGERECT flagsGravatar reed2015-08-07
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1281123002
* fixup precision with configurable atlasGravatar joshualitt2015-08-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1276383003
* Revert of Update libwebp and resume testing scaled webp (patchset #2 ↵Gravatar scroggo2015-08-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* Update libwebp and resume testing scaled webpGravatar scroggo2015-08-07
| | | | | | | | | | | | | | | | | 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
* GrDrawVertices to batchesGravatar joshualitt2015-08-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1276333004
* Move GrDrawAtlasBatch to separate fileGravatar jvanverth2015-08-07
| | | | | | Committed: https://skia.googlesource.com/skia/+/b091c44c33edbad3466601b29d60bc2eaa4453fa Review URL: https://codereview.chromium.org/1274013003
* Revert of Move GrDrawAtlasBatch to separate file (patchset #1 id:1 of ↵Gravatar jvanverth2015-08-07
| | | | | | | | | | | | | | | | | | | 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
* Move GrDrawAtlasBatch to separate fileGravatar jvanverth2015-08-07
| | | | Review URL: https://codereview.chromium.org/1274013003
* Make folder for batchesGravatar joshualitt2015-08-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1277233002
* First pass at drawAtlas batching.Gravatar jvanverth2015-08-07
| | | | | | | | | | 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
* Gold Workflows DocumentationGravatar hcm2015-08-07
| | | | | | | | BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1276663003 Review URL: https://codereview.chromium.org/1276663003
* Break LCD and Bitmap text dependency on hardcoded atlas valuesGravatar joshualitt2015-08-07
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac Review URL: https://codereview.chromium.org/1271873002
* Add approxBytesUsed to hashes.Gravatar mtklein2015-08-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1280653003
* The compiler can generate smulbb perfectly well nowadays.Gravatar mtklein2015-08-07
| | | | | | BUG=skia:4117 Review URL: https://codereview.chromium.org/1273203002
* Add Batch loggingGravatar joshualitt2015-08-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1276913002
* This iOS crash makes little sense to me. Add some debugging.Gravatar mtklein2015-08-07
| | | | | | | | | | | | 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
* This iOS crash makes little sense to me. Add some debugging.Gravatar mtklein2015-08-07
| | | | | | | | | | 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
* Fix StringTest for VS2015Gravatar bsalomon2015-08-06
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1280853002
* Fix warnings on VS2015 x86_64 buildGravatar bsalomon2015-08-06
| | | | | | TBR=bungeman@google.com Review URL: https://codereview.chromium.org/1276133002
* Split SkBmpCodec into three separate classesGravatar msarett2015-08-06
| | | | | | | | | | | | | | 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
* Let SkWebpCodec decode to 565Gravatar scroggo2015-08-06
| | | | | | BUG=skia:3257 Review URL: https://codereview.chromium.org/1277893002
* Plug a leak in GrTestUtils.Gravatar senorblanco2015-08-06
| | | | | | TBR=mtklein,bsalomon Review URL: https://codereview.chromium.org/1274963003
* Fix color swap for webp 565Gravatar scroggo2015-08-06
| | | | | | | | When webp decoding to 565, swap the colors so they look right. BUG=skia:4174 Review URL: https://codereview.chromium.org/1280693003
* remove guard for SrcPtr in drawBitmapRect for androidGravatar reed2015-08-06
| | | | | | | | BUG=skia: NOTRY=True TBR= Review URL: https://codereview.chromium.org/1277963002
* Purge non-NEON ARM code.Gravatar mtklein2015-08-06
| | | | | | | | 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
* __ARM_NEON is defined both by ARMv8 and ARMv7 + -mfpu=neon.Gravatar mtklein2015-08-06
| | | | | | | | Can't hurt to simplify a little. BUG=skia:4117 Review URL: https://codereview.chromium.org/1274513003
* Move some work from backend onClear to base class clearGravatar egdaniel2015-08-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1275543005
* Implement caching of stroked paths in the tessellating path renderer.Gravatar senorblanco2015-08-06
| | | | | | | | | | | 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
* flag to use const& instead of const* for src-rectGravatar reed2015-08-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1272713005
* Don't pass pipeline to GrBatch::generateGeometry()Gravatar bsalomon2015-08-06
| | | | | | R=joshualitt@google.com Review URL: https://codereview.chromium.org/1275083002
* Another small fix for reorderingGravatar joshualitt2015-08-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1274763008
* Port SkTextureCompression opts to SkOptsGravatar mtklein2015-08-06
| | | | | | | | | | | | | | | | 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
* Remove SK_LEGACY_SKPOINT3_CTORS flagGravatar robertphillips2015-08-06
| | | | | | 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
* SkSurface copy-on-write can yield stale GPU render targets.Gravatar fmalita2015-08-06
| | | | | | | | | | | | | | 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
* Minor cleanup: remove GR_TESSELLATING_PATH_RENDERING #ifdef.Gravatar senorblanco2015-08-06
| | | | | | Also, new -> SkNEW. Review URL: https://codereview.chromium.org/1280433002
* use kFast_SrcRectConstraint to match drawAtlas semanticsGravatar reed2015-08-06
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1272743003
* Support decoding PNG to 565.Gravatar scroggo2015-08-06
| | | | | | | | | | | | | | | | | | | | | | | | | 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
* simulate drawatlasGravatar reed2015-08-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1279733002
* Rephrase Windows gyp instructionsGravatar mtklein2015-08-05
| | | | | | | | | | The existing instructions seem misleading. BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1275753002 Review URL: https://codereview.chromium.org/1275753002
* Fix reording across rendertarget switchingGravatar joshualitt2015-08-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1276633003
* Detect MIPS DSP and DSPR2 programattically.Gravatar mtklein2015-08-05
| | | | | | | | | | | | | | | | | | | | | | | | | 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
* Fix variable shadowing and unreachable code that it causedGravatar brucedawson2015-08-05
| | | | | | | | | | | 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
* Fix ASAN error for wbmp swizzlesGravatar msarett2015-08-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1269413006