aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Move SkTDArray to private.Gravatar bungeman2016-02-18
| | | | | | | TBR=reed Moving to private is good. Review URL: https://codereview.chromium.org/1707213002
* Update SkSpecialImage APIGravatar robertphillips2016-02-18
| | | | | | | | | | | | | | | | This adds some temporary compatibility features: internal_getBM & internal_fromBM - to convert back & forth from Bitmaps in the ImageFilters internal_getProxy - to allow the special images to interface with the current ImagerFilter system It also adds a unique ID, opaque flag and size method. The unique ID and size method are needed for the image filter's caching system. The opaque flag is need for swapping back & forth from Bitmaps These are all calved off of https://codereview.chromium.org/1695823002/ (Get OffsetImageFilter really working with SkSpecialImages) which actually makes use of them. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705113002 Review URL: https://codereview.chromium.org/1705113002
* lcd blits for sRGBGravatar reed2016-02-18
| | | | | | | | | Unimplemented for F16 for the moment, but not needed yet. Surprisingly not much slower than current impl (not srgb correct) GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707883002 Review URL: https://codereview.chromium.org/1707883002
* Port Skia recipe to normal Python scripts, move to Skia repoGravatar borenet2016-02-18
| | | | | | | BUG=skia:4763 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703663002 Review URL: https://codereview.chromium.org/1703663002
* Remove FuzzPaeth now that we have some real Fuzzes.Gravatar mtklein2016-02-18
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1711753002 Review URL: https://codereview.chromium.org/1711753002
* Make build_command_buffer.py work on Mac and Windows.Gravatar bsalomon2016-02-18
| | | | | | | | | Also specify GYP_GENERATORS and add flags to gclient sync to make it more likely to succeed. BUG=skia:4957 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1704193002 Review URL: https://codereview.chromium.org/1704193002
* Create ParsePath API fuzzGravatar kjlubick2016-02-18
| | | | | | | | | This is based on https://codereview.chromium.org/1675053002 BUG=skia:4438 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1702383003 Review URL: https://codereview.chromium.org/1702383003
* remove legacy flagsGravatar caryclark2016-02-18
| | | | | | | | R=fmalita@chromium.org BUG=573166 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707383002 Review URL: https://codereview.chromium.org/1707383002
* fix asan error deref-ing null ptrGravatar caryclark2016-02-18
| | | | | | | | TBR=tomhudson@google.com BUG=skia:4549 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1709023002 Review URL: https://codereview.chromium.org/1709023002
* Remove position from FrontBufferedStreamGravatar scroggo2016-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When FrontBufferedStream was written, it implemented getPosition() and returned true for hasPosition(). The CL that introduced it (crrev.com/23717055) does not have any indication why, but I'm guessing it was because it was easy to implement. None of our decoders rely on this (only some tests do). Now that we have a decoder (SkRawCodec) that expects to be able to seek a stream if it has a position (which makes sense - SkStream.h associates that with SkStreamSeekable, and there is no other way to check to see if a stream is seekable), it is failing because FrontBufferedStream reports it has a position and the decoder tries to seek. Remove FrontBufferedStream::hasPosition() (reverting to the default, false) and ::getPosition() (so it will return 0). Fix tests - do not call FrontBufferedStream::getPosition() Update CodexTest to test using an FrontBufferedStream, like Android does. BUG=b/27218441 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703293002 Review URL: https://codereview.chromium.org/1703293002
* remove legacy bitmap filterGravatar caryclark2016-02-18
| | | | | | | | R=fmalita@chromium.org BUG=skia:2261 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1711683002 Review URL: https://codereview.chromium.org/1711683002
* fix comma in GrAuditTrail jsonGravatar joshualitt2016-02-18
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1704133002 Review URL: https://codereview.chromium.org/1704133002
* add interp pathGravatar caryclark2016-02-18
| | | | | | | | | | | Add path methods to determine if a pair of paths can be interpolated, and to interpolate them. R=reed@google.com, robertphillips@google.com BUG=skia:4549 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703943003 Review URL: https://codereview.chromium.org/1703943003
* Add bilerp filtering.Gravatar herb2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705203002 Review URL: https://codereview.chromium.org/1705203002
* some modes need clamping due to numerical imprecisionGravatar reed2016-02-17
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1710453003 TBR= Review URL: https://codereview.chromium.org/1710453003
* Adding direct access to SkMipMap levels.Gravatar cblume2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1639693002 Review URL: https://codereview.chromium.org/1639693002
* try plain-old code for sk_memset16/32 now that NEON is compile-timeGravatar mtklein2016-02-17
| | | | | | | | | | | | | Most of these implementations now just say "always inline". Let's see if we can get away with the simplicity of doing that all the time. These inlined implementations can autovectorize easily. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1639863002 CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot Review URL: https://codereview.chromium.org/1639863002
* Speculative fix for Google3 iOS buildGravatar msarett2016-02-17
| | | | | | | | | | | | | | | Now that SkImageGenerator is hooked up to use SkCodec by default, we need to compile SkCodec in order to compile Skia. The good news is that we can now turn on/off individual codecs. This CL enables SkCodec on iOS, but does not turn on any of the codecs that require third_party libraries. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1710493002 Review URL: https://codereview.chromium.org/1710493002
* Return the proper error code in SkBmpRLECodecGravatar msarett2016-02-17
| | | | | | | BUG=skia:4963 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707933002 Review URL: https://codereview.chromium.org/1707933002
* Add GrGpu support for Vulkan command buffer flushGravatar jvanverth2016-02-17
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705133002 Review URL: https://codereview.chromium.org/1705133002
* Mark existing image filter entry points that will be going away with DeprecatedGravatar robertphillips2016-02-17
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1709753002 Review URL: https://codereview.chromium.org/1709753002
* Don't you just feel bad every time you type 'delete'?Gravatar mtklein2016-02-17
| | | | | | | | | | This should fix https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-ASAN/builds/4787/steps/nanobench/logs/stdio BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703203003 Review URL: https://codereview.chromium.org/1703203003
* Further reduction of GPU workload in shapes benchmarks. Hopefully this ↵Gravatar brianosman2016-02-17
| | | | | | | | | squelches the random crashes on Mali 400. BUG=skia:4937 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703233002 Review URL: https://codereview.chromium.org/1703233002
* Move SkTArray to include/private.Gravatar bungeman2016-02-17
| | | | | | | TBR=reed Agreed moving to private is good. Review URL: https://codereview.chromium.org/1702073002
* Delete KTX tests to fix Android buildGravatar msarett2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707053002 Review URL: https://codereview.chromium.org/1707053002
* bin/sync-and-gyp: better error checkingGravatar halcanary2016-02-17
| | | | | | NOTRY=true Review URL: https://codereview.chromium.org/1704143002
* Enable GL interfaces for iOS buildGravatar iroth2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703923002 Review URL: https://codereview.chromium.org/1703923002
* add last move to serialized pathGravatar caryclark2016-02-17
| | | | | | | | | | | | A deserialized path keeps the last move location so that a subsequent append to the path does not add an unwanted close verb. R=reed@oogle.com BUG=skia:4847 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1702393002 Review URL: https://codereview.chromium.org/1702393002
* Fix name creation scope.Gravatar herb2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1701413003 Review URL: https://codereview.chromium.org/1701413003
* converted skiaserve PNGs from RGBA to RGBGravatar ethannicholas2016-02-17
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703893003 Review URL: https://codereview.chromium.org/1703893003
* Do not create BmpMaskCodecs with invalid bits per pixelGravatar msarett2016-02-17
| | | | | | | BUG=skia:4962 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1704063002 Review URL: https://codereview.chromium.org/1704063002
* Add batch information to jsonGravatar joshualitt2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705093002 Review URL: https://codereview.chromium.org/1705093002
* Fix "^" in --match flag for nanobench SKPsGravatar cdalton2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1706923002 Review URL: https://codereview.chromium.org/1706923002
* Remove 6-param applyCropRect() from lighting filters (GPU path).Gravatar senorblanco2016-02-17
| | | | | | | | | This required adding a texture domain effect if the destination bounds are not contained within the source bounds. BUG=skia:4502 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1698913004 Review URL: https://codereview.chromium.org/1698913004
* Add a script to build and copy the command buffer shared library. This ↵Gravatar bsalomon2016-02-17
| | | | | | | | | | | | script will be used by the bots to test skia on the command buffer. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1699273002 NOTREECHECKS=true NOTRY=true BUG=skia:4957 Review URL: https://codereview.chromium.org/1699273002
* Revert of further skiaserve image performance improvements (patchset #1 id:1 ↵Gravatar ethannicholas2016-02-17
| | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/1707453003/ ) Reason for revert: Broke skiaserve GPU rendering Original issue's description: > further skiaserve image performance improvements > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707453003 > > Committed: https://skia.googlesource.com/skia/+/68c063bf11c91b2b3b15cb93168cf2af9b2c642d TBR=joshualitt@google.com,jcgregorio@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1708683002
* Add request cache to SkFontHost_fontconfig.Gravatar bungeman2016-02-17
| | | | | | | | | | | The current code deduplicates SkTypeface instances as all font lookups should for better use of the glyph cache. This adds a request cache as well, so that repeated recent requests will return the cached result instead of doing a full lookup. BUG=chromium:424082, chromium:444894 Review URL: https://codereview.chromium.org/1683883002
* Make SkPicture/SkImageGenerator default to SkCodecGravatar msarett2016-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | Remove reference to SkImageDecoder from SkPicture. Make the default InstallPixelRefProc passed to CreateFromStream use SkImageGenerator::NewFromEncoded instead. Make SkImageGenerator::NewFromEncoded create an SkCodecImageGenerator. Remove the old version that used SkImageDecoder. Remove all versions of lazy_decode_bitmap/LazyDecodeBitmap. The default now behaves lazily. Update all clients to use the default. Move SkImageDecoderGenerator into KtxTest.cpp, and use it directly. This is a rebased version of: https://codereview.chromium.org/1671193002/ TBR=reed@google.com BUG=skia:4691 BUG=skia:4290 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1699183004 Review URL: https://codereview.chromium.org/1699183004
* Simplified linear pipeline.Gravatar herb2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1704583003 Review URL: https://codereview.chromium.org/1704583003
* further skiaserve image performance improvementsGravatar ethannicholas2016-02-17
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707453003 Review URL: https://codereview.chromium.org/1707453003
* bin/sync-and-gyp: pass --deps='...' to `gclient sync`.Gravatar halcanary2016-02-17
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705063002 Review URL: https://codereview.chromium.org/1705063002
* Individually enable and disable SkCodecsGravatar msarett2016-02-17
| | | | | | | | BUG=skia:4956 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1702533004 CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot Review URL: https://codereview.chromium.org/1702533004
* Updates Piex and uses it to obtain the DNG dimensions.Gravatar ebrauer2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1659873002 Review URL: https://codereview.chromium.org/1659873002
* fast sk4f <-> sk4i SSE methodsGravatar mtklein2016-02-17
| | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1707673002 CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot Review URL: https://codereview.chromium.org/1707673002
* misc fixes to make float buffers work a little betterGravatar reed2016-02-17
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703013002 TBR=fmalita Review URL: https://codereview.chromium.org/1703013002
* dead code: really kill these off againGravatar mtklein2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1702343002 Review URL: https://codereview.chromium.org/1702343002
* come on, this is how you drive-byGravatar mtklein2016-02-17
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1708563002 Review URL: https://codereview.chromium.org/1708563002
* Use geometric mean when selecting a mipmap scaleGravatar fmalita2016-02-17
| | | | | | | | | | Workaround for anisotropic mipmap quality issues. R=reed@google.com,robertphillips@google.com BUG=skia:4863,chromium:586894 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1697423002 Review URL: https://codereview.chromium.org/1697423002
* Update PIEX version. This should solve the stack buffer overflow issueGravatar yujieqin2016-02-17
| | | | | | | BUG=skia:4902 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1690853003 Review URL: https://codereview.chromium.org/1690853003
* Fix memory leakGravatar joshualitt2016-02-17
| | | | | | | | TBR=bsalomon@google.com BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1706773002 Review URL: https://codereview.chromium.org/1706773002