aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* SkQP: fix upload scriptGravatar Hal Canary2018-01-23
| | | | | | | | | Also report transparency Change-Id: I38c803d1abe1892dc4d7ee6d917f5a39eac6c1c3 Reviewed-on: https://skia-review.googlesource.com/98762 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Added bullet proofing to WrappedProxyConversionTestGravatar Robert Phillips2018-01-23
| | | | | | | Change-Id: I8a3f023b2e81d0f3224022a9a5e3a37bc0b37f64 Reviewed-on: https://skia-review.googlesource.com/99041 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Remove superfluous SkReadBuffer validity checksGravatar Robert Phillips2018-01-23
| | | | | | | | | The pattern is to create the spurious object & delete it later Change-Id: I435e2872127e02c241b5dd27cd965f3c7047a77a Reviewed-on: https://skia-review.googlesource.com/98783 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Revert "converted vertex shaders to device coords"Gravatar Ethan Nicholas2018-01-23
| | | | | | | | | | | | This reverts commit e7e81c15c144b8133f696d0744ed9f7e8d06e936. Reason for revert: Chrome perf regressions Bug: skia: Change-Id: I17fadc97c4b8e80bfdccbf123554614a00c58473 Reviewed-on: https://skia-review.googlesource.com/99040 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Make SkAnimatedImage::update return sentinel when stoppedGravatar Leon Scroggins III2018-01-23
| | | | | | | | Bug: b/63908092 Change-Id: I585f3d3efde1db4d5be36b11f19fe6e88f131608 Reviewed-on: https://skia-review.googlesource.com/98062 Reviewed-by: Leon Scroggins <scroggo@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
* Move glyph cache and descriptor functions off of SkPaintGravatar Herb Derby2018-01-23
| | | | | | | | | BUG=skia:7515 Change-Id: If17b157db1077a9a3c0f9efd03929f62a3486419 Reviewed-on: https://skia-review.googlesource.com/98841 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Herb Derby <herb@google.com>
* ccpr: Use primitive restart feature on ARMGravatar Chris Dalton2018-01-23
| | | | | | | | Bug: skia: Change-Id: Ia7ab55f8b6ecdd674762f5197d35e8db58f07c01 Reviewed-on: https://skia-review.googlesource.com/98180 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
* move the guts of SkMaskFilter.h into SkMaskFilterBase.hGravatar Mike Reed2018-01-23
| | | | | | | | | Bug: skia: Change-Id: I29ad0960156562867429542d3cfbf3d639529cab Reviewed-on: https://skia-review.googlesource.com/98802 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Reviewed-by: Brian Salomon <bsalomon@google.com>
* GrTessellator: add a #define for max aa verb count.Gravatar Stephen White2018-01-23
| | | | | | | | | | | | We'd like to increase the maximum verb count for the edge-AA tessellating path renderer. In order to protect clients from this change, we allow them to set their own value. Bug:805031 Change-Id: I5d41059a217f059998d418d51fcc8c88464ef4da Reviewed-on: https://skia-review.googlesource.com/98886 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Stephen White <senorblanco@chromium.org>
* Various MSAN fixesGravatar Kevin Lubick2018-01-23
| | | | | | | | | Initialize fBitmap.* in SkPictureData Change-Id: I72c91a3a85881ae0ca7f64702263e2f8ea02a5c8 Reviewed-on: https://skia-review.googlesource.com/98740 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Increase depth threshold for SkClipStack::isRRect.Gravatar Brian Salomon2018-01-23
| | | | | | | | | | Also first check if stack is an intersection of rects. Bug: skia: Change-Id: Iaa0c65941b07739998cd8c0a4edd424d995da387 Reviewed-on: https://skia-review.googlesource.com/98761 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* Revert "Revert "Revert "Use GrOvalOpFactory ops to draw non-AA rrects, ↵Gravatar Brian Salomon2018-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ovals, and arcs.""" This reverts commit b4e965c103c3140f0072949343befb34bceadc96. Reason for revert: Thin stroke problems, chrome pixel test, g3 clang-tidy Original change's description: > Revert "Revert "Use GrOvalOpFactory ops to draw non-AA rrects, ovals, and arcs."" > > This reverts commit befff6580e9dc6b15bc6db3cc3f5fff22185cba8. > > Bug: skia: > Change-Id: Ia108408cd986c03783af5058845790f6b17201f4 > Reviewed-on: https://skia-review.googlesource.com/98703 > Reviewed-by: Brian Salomon <bsalomon@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> TBR=bsalomon@google.com Change-Id: I785af7efdfc39680052f5721a55ecffbeaf1f0ad No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia: Reviewed-on: https://skia-review.googlesource.com/98785 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* init out-param on buffer failureGravatar Mike Reed2018-01-23
| | | | | | | | Bug:804651 Change-Id: Ia3c71e2fbe35b55bf27075e6a11ec8c5bd3fb40f Reviewed-on: https://skia-review.googlesource.com/98663 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Fix skia image filter fuzz SkLightingImageFilter complaintGravatar Robert Phillips2018-01-23
| | | | | | | Change-Id: I3bd9831b0160e797481469ae93b6d037d81b07da Reviewed-on: https://skia-review.googlesource.com/98780 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Fix some fuzzer bugs from Skia's image filter fuzzerGravatar Robert Phillips2018-01-23
| | | | | | | | Change-Id: I432b3a351eecca0d36635e37f91d32c0e281b7d9 Reviewed-on: https://skia-review.googlesource.com/98384 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Mike Reed <reed@google.com>
* SkQP: upload_directory_contents.sh is shardedGravatar Hal Canary2018-01-23
| | | | | | | | NOTRY=true Change-Id: I1976778cd2c2bdc82f551285b3d36212034f8f7a Reviewed-on: https://skia-review.googlesource.com/98643 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* SkQP: set fDisableDriverCorrectnessWorkaroundsGravatar Hal Canary2018-01-23
| | | | | | | Change-Id: I43ea362c5ff726278e2b74bea73cd54a9ef07761 Reviewed-on: https://skia-review.googlesource.com/98843 Commit-Queue: Hal Canary <halcanary@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
* SkQP: Cloud ScriptsGravatar Hal Canary2018-01-23
| | | | | | | | | | | | | | | | | | - tools/skqp/generate_gn_args.sh - single script to generate configuration used to build SkQP. - tools/skqp/make_model.sh - script to execute make_gmkb.go and put the results in the correct place. - tools/skqp/{up,down}load_directory_contents.sh - scripts to move large assets into the cloud and retrieve them. - tools/skqp/gm_runner.cpp - treat missing text files as empty. - tools/skqp/make_apk.sh - make use of generate_gn_args.sh. - tools/skqp/make_{model,known_tests}.sh - better error handling, and also stop calling `git add`. Change-Id: Ib1803b4d68e63945f2c47f8eb6cb96375d24e3be Reviewed-on: https://skia-review.googlesource.com/98842 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* [skottie] Initial mask supportGravatar Florin Malita2018-01-23
| | | | | | | | | TBR= Change-Id: Ibf4baeb17f98e1ed359f04deefe2f1c09414540d Reviewed-on: https://skia-review.googlesource.com/98840 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* require finite blob boundsGravatar Mike Reed2018-01-23
| | | | | | | | | | inspired by https://fuzzer.skia.org/category/image_filter_deserialize/file/aW5jbHVkZS9jb3JlL1NrUGFpbnQuaA==? Bug: skia: Change-Id: I490585b9fdc0b4b29565849478b97ed7f5057ca1 Reviewed-on: https://skia-review.googlesource.com/98800 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Start of adding GrContextOption that overrides driver workarounds.Gravatar Brian Salomon2018-01-23
| | | | | | | | | This handles most of the GL driver workarounds but not Vulkan (yet). GL wokarounds related to config support are still not handled. Change-Id: I3d0f62ac8a0f6e9ba13632ea82acdc493912c63d Reviewed-on: https://skia-review.googlesource.com/98382 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
* check for valid vertices modeGravatar Mike Reed2018-01-23
| | | | | | | | Bug: skia:7512 Change-Id: I2ede2232f3807e7bad8562eb3bf6327ffc52a996 Reviewed-on: https://skia-review.googlesource.com/98760 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* Revert "Revert "Use GrOvalOpFactory ops to draw non-AA rrects, ovals, and ↵Gravatar Brian Salomon2018-01-23
| | | | | | | | | | | | arcs."" This reverts commit befff6580e9dc6b15bc6db3cc3f5fff22185cba8. Bug: skia: Change-Id: Ia108408cd986c03783af5058845790f6b17201f4 Reviewed-on: https://skia-review.googlesource.com/98703 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
* report error on global enumGravatar Cary Clark2018-01-23
| | | | | | | | | | | | | | | | rather than segfault when running bookmaker cross check, report a parsable error. In a separate CL, I'll add support for global enums. R=herb@google.com Bug: skia:6898 Change-Id: I91c62df5470acbf5c0acf0c558d2d5d3bf48d40e Reviewed-on: https://skia-review.googlesource.com/98682 Commit-Queue: Cary Clark <caryclark@google.com> Reviewed-by: Herb Derby <herb@google.com>
* rm spurious std::move from getFontAtlasImage_ForTestingGravatar Robert Phillips2018-01-23
| | | | | | | Change-Id: I3a384126b01d6f066f70f6ef8e75bf3445f94daf Reviewed-on: https://skia-review.googlesource.com/98706 Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Fixed a few spots where SkSL optimizer was not respecting side effectsGravatar Ethan Nicholas2018-01-23
| | | | | | | | | | | | | | This was affecting expressions such as 0 * float4(<expr>), which would be collapsed down to float4(0) - in some cases even if <expr> had side effects. This is obviously incorrect no matter what, but to make matters worse it could lead to a use-after-free when we eliminated an assignment which we were tracking as the current definition of a variable. Bug: skia:7467 Change-Id: I91ba154c57dad9cadf36b6062bec3211557248e0 Reviewed-on: https://skia-review.googlesource.com/98704 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
* Clean up SkScalerContext descriptor creationGravatar Herb Derby2018-01-23
| | | | | | | | | | | | | | | | Allow a descriptor creation using just a rec and effects. This is the most complex part of the change. Cleanup SkScalerContextFlags to be a real thing. This has spread everywhere. BUG=skia:7515 Change-Id: I67189c9aeb51f1777a1c60145216524d440cb890 Reviewed-on: https://skia-review.googlesource.com/98320 Commit-Queue: Herb Derby <herb@google.com> Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Mike Reed <reed@google.com>
* remove unneeded SkFlattenableSerializationGravatar Mike Reed2018-01-23
| | | | | | | | | Bug: skia: Change-Id: I5cf40b88b05dcd8e83383f6271b046eac8d55e35 Reviewed-on: https://skia-review.googlesource.com/98642 Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Roll skia/third_party/externals/angle2/ dd21ecf80..d66e19393 (1 commit)Gravatar angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com2018-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | https://chromium.googlesource.com/angle/angle.git/+log/dd21ecf80a62..d66e193930dd $ git log dd21ecf80..d66e19393 --date=short --no-merges --format='%ad %ae %s' 2018-01-17 oetuaho Allow creating EGL pbuffers from typeless D3D textures Created with: roll-dep skia/third_party/externals/angle2 The AutoRoll server is located here: https://angle-skia-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE TBR=robertphillips@google.com Change-Id: Ib7d6adc581baa5e134134fc29c3e9029c156d0c3 Reviewed-on: https://skia-review.googlesource.com/98720 Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
* IWYU -- SkShader.hGravatar Mike Reed2018-01-23
| | | | | | | | Bug: skia: Change-Id: I57bffc14c4888627a22158e35f3ac3b762138f02 Reviewed-on: https://skia-review.googlesource.com/98520 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
* [skottie] Discard orphaned geometry animatorsGravatar Florin Malita2018-01-23
| | | | | | | | | TBR= NOTRY=true Change-Id: I8e4de99c8ebc3ec28509753b59a287d896e65aea Reviewed-on: https://skia-review.googlesource.com/98705 Commit-Queue: Florin Malita <fmalita@chromium.org> Reviewed-by: Florin Malita <fmalita@chromium.org>
* Roll skia/third_party/externals/angle2/ 66f73b731..dd21ecf80 (1 commit)Gravatar angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com2018-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | https://chromium.googlesource.com/angle/angle.git/+log/66f73b731e97..dd21ecf80a62 $ git log 66f73b731..dd21ecf80 --date=short --no-merges --format='%ad %ae %s' 2018-01-10 oetuaho Add const qualification to symbol accesses Created with: roll-dep skia/third_party/externals/angle2 The AutoRoll server is located here: https://angle-skia-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE TBR=robertphillips@google.com Change-Id: I5cb83f53c69af9ef4baa277db3e2b9067223f64d Reviewed-on: https://skia-review.googlesource.com/98620 Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
* [skottie] Improved animation params parsingGravatar Florin Malita2018-01-23
| | | | | | | | | | Some BM versions wrap the Bezier animation params into arrays. TBR= Change-Id: I376b1ed2079105066413b513c3df33a61440cf41 Reviewed-on: https://skia-review.googlesource.com/98580 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* [skottie] Improved trim path supportGravatar Florin Malita2018-01-23
| | | | | | | | | | |offset| is an angular value in [0..360]. TBR= Change-Id: I10a91bec7e8f43db1f54c8b5358dd0604c974998 Reviewed-on: https://skia-review.googlesource.com/98560 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* Roll skia/third_party/externals/angle2/ f661686c1..66f73b731 (12 commits)Gravatar angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com2018-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://chromium.googlesource.com/angle/angle.git/+log/f661686c1ace..66f73b731e97 $ git log f661686c1..66f73b731 --date=short --no-merges --format='%ad %ae %s' 2018-01-22 lucferron Improve the dev setup instructions on Linux 2018-01-12 oetuaho Prevent changing AST expression type from outside 2018-01-19 jmadill Vulkan: Add vk_cache_utils.h. 2018-01-19 ynovikov Skip FindMSBAndFindLSBCornerCases on Android OpenGLES 2018-01-10 oetuaho Don't copy unmangled built-in names 2018-01-11 jiajia.qin Fixed ssbo end2end tests failure 2018-01-16 oetuaho Make TVariable type immutable 2018-01-18 oetuaho Fix handling array constructor statements in HLSL output 2018-01-09 oetuaho Don't duplicate symbol type information in AST nodes 2018-01-17 oetuaho Fix dxgi support table generation script 2018-01-09 xinghua.cao Add more test cases for bindImageTexture 2018-01-12 geofflang Add GLES1 targets and stub entry points. Created with: roll-dep skia/third_party/externals/angle2BUG=767240 The AutoRoll server is located here: https://angle-skia-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE,Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE,Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE,Build-Debian9-Clang-x86_64-Release-ANGLE TBR=robertphillips@google.com Change-Id: I30812c05143c4d832ae4757314759a238df8ecc0 Reviewed-on: https://skia-review.googlesource.com/98307 Commit-Queue: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by: <angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
* Update ANGLE's BUILD.gn to include definitions for GL_API.Gravatar Geoff Lang2018-01-23
| | | | | | | | Bug: skia: Change-Id: I57a16af6c57166ead768ad73a8124690f1fa58e4 Reviewed-on: https://skia-review.googlesource.com/98302 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* Verify Status is showing new commits.Gravatar Ben Wagner2018-01-22
| | | | | | | | No-Try: true Change-Id: Ic79fd6c1f991037f1c6506fae40bf227130d9626 Reviewed-on: https://skia-review.googlesource.com/98480 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* check for huge pathsGravatar Mike Reed2018-01-22
| | | | | | | | | Bug:802976 Change-Id: Ibb5930442f75ca8483afc8dfa5869cac98573904 Reviewed-on: https://skia-review.googlesource.com/98440 Reviewed-by: Cary Clark <caryclark@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Convert SVGPong sample to sksgGravatar Florin Malita2018-01-22
| | | | | | | | | | Use the new scene graph impl instead of old SVG DOM. TBR= Change-Id: I590798b362e953aa3554e85669193f7501ba46de Reviewed-on: https://skia-review.googlesource.com/98441 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
* SkQP: add some scripts for adminGravatar Hal Canary2018-01-22
| | | | | | | Change-Id: Idfba4c3e91eb048792ab395d4ab2e4a98ed8888c Reviewed-on: https://skia-review.googlesource.com/98323 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Make use of VkSemaphores thread safe assuming use of the same VkQueue.Gravatar Greg Daniel2018-01-22
| | | | | | | | | | | | This allows us to re-enable support for multiple GrContexts in GrBackendTextureImageGenerator. Bug: skia: Change-Id: Ifd6ac1ad81cdfbd1fd986467d8beb359399d6588 Reviewed-on: https://skia-review.googlesource.com/98340 Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
* Fix crash in ~GrContextGravatar Robert Phillips2018-01-22
| | | | | | | | BUG=804352, 804111 Change-Id: Ia8475a4a8a6de77438cde7df53931dc70c405b16 Reviewed-on: https://skia-review.googlesource.com/98341 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
* move zero-init to sk_malloc for masksGravatar Mike Reed2018-01-22
| | | | | | | | Bug: skia: Change-Id: I75d557068bdcd9e9f7e380e4fa447f9d83dd1554 Reviewed-on: https://skia-review.googlesource.com/98200 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Reed <reed@google.com>
* fetch-clang-format: better error checkingGravatar Hal Canary2018-01-22
| | | | | | | Change-Id: Idc0e984275056128301d4f1ae66aa6ace67b05d5 Reviewed-on: https://skia-review.googlesource.com/98322 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* SkQP: replace blacklist with: DoNotExecuteInExperimentalMode and ↵Gravatar Hal Canary2018-01-22
| | | | | | | | | | | NoScoreInCompatibilityTestMode Also clean up some things, fix docs, whitelist. Change-Id: I2818d973978ffe1b8ce0cc9c69f8d91ab4a0ef22 Reviewed-on: https://skia-review.googlesource.com/91805 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
* Set empty device clip when an image filter has zero boundsGravatar Xianzhu Wang2018-01-22
| | | | | | | | | | | | | | When an image filter has zero input bounds, we should not paint any contents of it, so we should set an empty clip. This is like the case that the clip rect doesn't intersect with the image filter's input bounds. Bug: chromium:771643 Change-Id: I063c14128dacb83e3572bd2ef4dfeee93c871064 Reviewed-on: https://skia-review.googlesource.com/96943 Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Stephen White <senorblanco@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
* Add SkAnimatedImage::isFinishedGravatar Leon Scroggins III2018-01-22
| | | | | | | | | | | Bug: b/63908092 Allows Android to know when to call onAnimationEnd. Change-Id: I9cc102fb485e944ad5983eed9f0b941153128e88 Reviewed-on: https://skia-review.googlesource.com/97401 Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
* Fix startup for SampleSlideGravatar Jim Van Verth2018-01-22
| | | | | | | | | | | | When we start up Viewer with a SampleSlide as the first slide, nothing gets rendered because SampleSlide depends on the backend being initialized before setting its dimensions. This pushes the first slide's initialization after the backend creation. Change-Id: I8f83fe7b3973c40234104f1a83c0a60750694311 Reviewed-on: https://skia-review.googlesource.com/98161 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Jim Van Verth <jvanverth@google.com>
* Respect repetition count in SkAnimatedImageGravatar Leon Scroggins III2018-01-22
| | | | | | | | | | | | | | | | | | | | | | | | | | Bug: b/63908092 By default use the repetition count stored in the encoded data (if any). Allow setting the repetition count manually, so that the animation will stop after n+1 total cycles (unless -1 is used for infinite). If the animation is complete, make start reset it. When the animation is not running, make update return max double (i.e. no need to update any time soon). Fix a bug where the first call to update returned -1. Share write_bm with CodecAnimTest, for debugging. Update Sample to check isRunning rather than keeping its own record of whether the animation is running. Change-Id: I883e4d7325f7a7b23a422fa9d756f9ea3018f0f8 Reviewed-on: https://skia-review.googlesource.com/97082 Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
* https://skia.org/skqpGravatar Hal Canary2018-01-22
| | | | | | | | | No-Try: true Docs-Preview: https://skia.org/?cl=94141 Change-Id: I43954417047642b8671ba8735fb0895eb6810321 Reviewed-on: https://skia-review.googlesource.com/94141 Commit-Queue: Derek Sollenberger <djsollen@google.com> Reviewed-by: Stephan Altmueller <stephana@google.com>