Commit Graph

41690 Commits

Author SHA1 Message Date
Raul Tambre
0362dc268a Fix use of 'is' operator for comparison
The 'is' operator is not meant to be used for comparisons. It currently working is an implementation detail of CPython.
CPython 3.8 has added a SyntaxWarning for this.

Bug: chromium:958874
Change-Id: I7657bd3fddf37e849cc30aca34825fa3c0f71b58
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211663
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Auto-Submit: Raul Tambre <raul@tambre.ee>
2019-05-07 14:22:15 +00:00
Brian Salomon
b0d8b76c4f Make finish flush procs work with GL.
This will be required for async readback support.

This is implemented using sync objects when available and otherwise
calls glFinish.

Relaxes the unit test requirement that providing a callback with no
work to flush always calls the proc immediately.

Bug: skia:8962
Change-Id: Ieefcab6dccc3924e50260343f01904e7303bb12b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212198
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
2019-05-07 10:35:38 +00:00
skia-autoroll
d05c5eec42 Roll third_party/externals/angle2 6f0c5b8dd5f2..0f4d72e63b92 (8 commits)
6f0c5b8dd5..0f4d72e63b


git log 6f0c5b8dd5f2..0f4d72e63b92 --date=short --no-merges --format='%ad %ae %s'
2019-05-06 geofflang@chromium.org Add makeCurrent and unMakeCurrent to SurfaceImpl.
2019-05-06 geofflang@chromium.org Vulkan: Add a new garbage type gated by fences.
2019-05-06 geofflang@chromium.org Format all of ANGLE's python code.
2019-05-06 lsalzman@mozilla.com add support for EXT_blend_func_extended to D3D11
2019-05-06 syoussefi@chromium.org Suppress test hitting Nvidia shield driver bug
2019-05-06 jonahr@google.com Use EnumAdapters to properly detect primary GPU on Win
2019-05-06 geofflang@chromium.org Vulkan: Store fences instead of serials in SurfaceVK::SwapHistory
2019-05-06 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/glslang/src 86c72c9486a9..c0640dabfddd (2 commits)


Created with:
  gclient setdep -r third_party/externals/angle2@0f4d72e63b92

The AutoRoll server is located here: https://autoroll.skia.org/r/angle-skia-autoroll

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:Build-Debian9-Clang-x86_64-Release-ANGLE;skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE
TBR=robertphillips@google.com

Change-Id: Ibc2802e22275784c206aabc930704e908143674e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212342
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-07 05:21:41 +00:00
skia-autoroll
743913344b Roll ../src e300e8fa69a2..648a59893e4e (393 commits)
e300e8fa69..648a59893e


Created with:
  gclient setdep -r ../src@648a59893e

The AutoRoll server is located here: https://autoroll.skia.org/r/chromium-skia-autoroll

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-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Release-All-CommandBuffer;skia.primary:Test-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Debug-All-CommandBuffer
TBR=robertphillips@google.com

Change-Id: I0b5d073dffdf7f1c37de204834ceb282993038bd
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212343
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-07 04:52:41 +00:00
skia-autoroll
d699a66fa2 Roll third_party/externals/swiftshader bb12b0e41a09..b01f2559e017 (10 commits)
https://swiftshader.googlesource.com/SwiftShader.git/+log/bb12b0e41a09..b01f2559e017


git log bb12b0e41a09..b01f2559e017 --date=short --no-merges --format='%ad %ae %s'
2019-05-06 chrisforbes@google.com Add padding between slices
2019-05-06 chrisforbes@google.com Add support for spv::BuiltInSampleMask output
2019-05-06 capn@google.com Implement 1D array image sampling
2019-05-06 chrisforbes@google.com Wait on query being available, not !active.
2019-05-06 chrisforbes@google.com Allow VK_IMAGE_LAYOUT_GENERAL in vkCmdCopyImageToBuffer
2019-05-06 chrisforbes@google.com Plumb through Dref parameter to sampler
2019-05-06 chrisforbes@google.com Make shadow compare available in high precision path always
2019-05-06 sugoi@google.com Threaded Queue submit with events and fences
2019-05-06 sugoi@google.com Cube border computation
2019-05-06 sugoi@google.com ETC2 Image decompression and sampling


Created with:
  gclient setdep -r third_party/externals/swiftshader@b01f2559e017

The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-skia-autoroll

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:Test-Debian9-Clang-GCE-GPU-SwiftShader-x86_64-Debug-All-SwiftShader
TBR=robertphillips@google.com

Change-Id: I8b8e74b7829f9413e7ec32945833b46e0aa36525
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212341
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-07 04:33:51 +00:00
Hal Canary
5f2807b71e Clean up CanvasTest.cpp some more
Change-Id: I44929cbb13e784f1e1ab7410598c02d285b71def
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212268
Commit-Queue: Hal Canary <halcanary@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Auto-Submit: Hal Canary <halcanary@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
2019-05-06 21:23:36 +00:00
Ben Wagner
cbf6d30661 Clean up CanvasTest.cpp macros.
The CanvasTest test had a rather impenetrable set of macros and static
registration for its sub tests. This moves all the code only used by
this test into the test body and removes all of the static registration
and macros.

Change-Id: Ie6031591f86e17ec949db995023ac10360c9092f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212260
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 20:18:36 +00:00
Brian Osman
a9aaef0c99 Revert "Support GL_EXT_draw_buffers"
This reverts commit 77af4ecd35.

Reason for revert: iOS errors and bad images on Gold

Original change's description:
> Support GL_EXT_draw_buffers
> 
> Change-Id: I078aa7c42de4368602b0ef43bd7e18efbfd1e049
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212182
> Reviewed-by: Robert Phillips <robertphillips@google.com>
> Commit-Queue: Brian Osman <brianosman@google.com>

TBR=bsalomon@google.com,robertphillips@google.com,brianosman@google.com

Change-Id: Ib8759d4a1009f9a8774365a699e211834d634cff
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212264
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
2019-05-06 20:07:27 +00:00
Brian Osman
77af4ecd35 Support GL_EXT_draw_buffers
Change-Id: I078aa7c42de4368602b0ef43bd7e18efbfd1e049
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212182
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
2019-05-06 18:59:06 +00:00
Robert Phillips
d3c048234b Re-add flag to force opList splitting reduction
We need this again because I had to turn off opList splitting reduction by default but would like to keep testing it on some bots

Change-Id: I9e38d5a3d823e86799055892d191a5643a4b98ac
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212193
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2019-05-06 18:52:15 +00:00
recipe-roller
cb35914098 Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).


More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
depot_tools:
  https://crrev.com/b3aca437d08fb4a2c6f656ad0a0afae3d0396cec Revert "bot_update: Pass target-ref to gclient." (martiniss@chromium.org)
  https://crrev.com/47ec0697f06bbe67a9156dd45e0c5b478d0f7f65 git_cache: remove the mirror_path if it exists in an incomplete state. (jbudorick@chromium.org)
  https://crrev.com/a0ffbe45058667e3ba0810557ba45436416144e2 gclient: Convert target ref to patch to a remote ref. (ehmaldonado@chromium.org)
  https://crrev.com/867e3c9511394f5bb2f51f09278e5ccc773adc36 Reland "bot_update: Pass target-ref to gclient." (ehmaldonado@chromium.org)
  https://crrev.com/fe1dbe120c9905324026fa503ac1690dc96e16e2 Fix always false 'is not' comparison with a literal (raul@tambre.ee)
  https://crrev.com/9c0620120980e4c247ff8325ee8bbdcd4d9576e0 Fix multiple confusingly escaped regex strings (brucedawson@chromium.org)
recipe_engine:
  https://crrev.com/b8cb36a2fedc4739cb317ffb60fc6fa3cc231048 [docs] Add some skeleton documentation for writing recipes. (iannucci@chromium.org)
  https://crrev.com/729f9023cff72064acd9f3f2548d90f5ea03461e [buildbucket] Fix buildbucket.search (nodir@google.com)
  https://crrev.com/0589a429cf3c164004dae4ced4c75784a50afd81 Only delete the specified element, not all of them (sergiyb@chromium.org)


TBR=borenet@google.com

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I9314b04443deeec81ca082b28174d0f1a08e7238
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212195
Reviewed-by: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
2019-05-06 18:44:46 +00:00
Eric Boren
ebc21f64cd [infra] Pin infra code using Go modules
Change-Id: I005740aef6ab8ba78f3da4d3b9d744ce3fb326e6
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211982
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Ben Wagner aka dogben <benjaminwagner@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
2019-05-06 17:53:19 +00:00
Robert Phillips
25aac5c196 Make SkComposeImageFilter comply with SkImageFilter::filterBounds calling convention
In kForward mode the 'inputRect' parameter is supposed to be null

Bug: 959103
Change-Id: I2fceace384cdda3988e07dcb5d975750ba28bd39
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212187
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
2019-05-06 17:49:04 +00:00
Tom Anderson
e772c59d94 Remove static initializer in SkExecutor.cpp
Bug: chromium:956591
Change-Id: I79732cb0266020dc00e622b126cd565e953600dd
R=thakis@chromium.org
CC=liaoyuke@chromium.org
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211835
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
Auto-Submit: Thomas Anderson <thomasanderson@chromium.org>
2019-05-06 17:48:14 +00:00
Kevin Lubick
0921c10404 [canvaskit] 0.6.0
Change-Id: I4c408a8b711421856a14d67cee121512f7e06900
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212190
Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-05-06 17:47:09 +00:00
Kevin Lubick
d6b32ed39d [img-decode] Start on proposed new spec
This adds readPixel and a partial stub of window.createImageData

Change-Id: Iee992312b9331b71852fe2198f844a7e4ae9e963
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211344
Reviewed-by: Mike Reed <reed@google.com>
2019-05-06 17:39:19 +00:00
Brian Osman
5e7fbfd9b0 Add ShaderErrorHandler to GrContextOptions
Allows clients to customize behavior when shaders fail to compile.
Added nicer shader error handling to viewer.

Change-Id: If82b48e40d64fd786f37e88c564fd623b53c7f9d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211361
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
2019-05-06 16:29:13 +00:00
Michael Ludwig
7d2ad0fdef Fix local matrix for crop rects in backdrop image filters
Bug: chromium:955467
Change-Id: I1c4cae2499db926aa6b629e8d730fcc8cb45be6d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212030
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Michael Ludwig <michaelludwig@google.com>
2019-05-06 15:11:40 +00:00
skia-autoroll
2f3637bf2e Roll third_party/externals/angle2 ddc4d33a259a..6f0c5b8dd5f2 (15 commits)
ddc4d33a25..6f0c5b8dd5


git log ddc4d33a259a..6f0c5b8dd5f2 --date=short --no-merges --format='%ad %ae %s'
2019-05-04 jmadill@chromium.org D3D11: Fix potential invalid VAO access in dispatch.
2019-05-04 syoussefi@chromium.org Vulkan: Optimize shader source macro replacement
2019-05-03 geofflang@chromium.org Vulkan: Reset swap chain image dependency information when aquiring.
2019-05-03 syoussefi@chromium.org Vulkan: Fix qualifiers for varyings
2019-05-03 jmadill@chromium.org Add option to run each test config in a separate process.
2019-05-03 yiyix@google.com Add support for dynamically allocate initial stack for Angle
2019-05-03 jmadill@chromium.org Remove EGLTest.
2019-05-03 jmadill@chromium.org Add command line option to filter by test config.
2019-05-03 syoussefi@chromium.org Vulkan: Fix layout substitution for struct varyings
2019-05-03 jmadill@chromium.org Move ANGLE Platform Methods init to GetDisplay.
2019-05-03 jonahr@google.com Consolidate 32/64 bit dEQP results on doc.
2019-05-03 jmadill@chromium.org Removed SizedFloat helper class from upload tests.
2019-05-03 yiyix@chromium.org Add support for GCC Compilers
2019-05-03 jmadill@chromium.org Simplify ANGLE test config names.
2019-05-03 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src c8b09744c6a1..5fc5303eeceb (2 commits)


Created with:
  gclient setdep -r third_party/externals/angle2@6f0c5b8dd5f2

The AutoRoll server is located here: https://autoroll.skia.org/r/angle-skia-autoroll

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:Build-Debian9-Clang-x86_64-Release-ANGLE;skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE
TBR=robertphillips@google.com

Change-Id: I7d5329a07ab69039d0cb4a0d26d13bf5ae13185d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212141
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-06 05:29:20 +00:00
skia-autoroll
46e10981fe Roll ../src b81b70981d9b..e300e8fa69a2 (429 commits)
b81b70981d..e300e8fa69


Created with:
  gclient setdep -r ../src@e300e8fa69

The AutoRoll server is located here: https://autoroll.skia.org/r/chromium-skia-autoroll

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-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Release-All-CommandBuffer;skia.primary:Test-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Debug-All-CommandBuffer
TBR=robertphillips@google.com

Change-Id: If16efc2386455af08bb20562adf4b47f6e600050
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212142
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-06 04:40:49 +00:00
skia-autoroll
719a69ca5a Roll third_party/externals/swiftshader 10f7befc49a7..bb12b0e41a09 (14 commits)
https://swiftshader.googlesource.com/SwiftShader.git/+log/10f7befc49a7..bb12b0e41a09


git log 10f7befc49a7..bb12b0e41a09 --date=short --no-merges --format='%ad %ae %s'
2019-05-04 swiftshader.regress@gmail.com Regres: Update test lists @ 0e14a2e3
2019-05-03 chrisforbes@google.com Fix broken optimization of alpha VK_BLEND_OP_REVERSE_SUBTRACT
2019-05-03 bclayton@google.com Add traits for Reactor arguments and return types
2019-05-03 bclayton@google.com SetupRoutine: Do not use Bool as a function return type.
2019-05-03 bclayton@google.com Reactor: Do not start the unreachable block with Unreachable
2019-05-03 bclayton@google.com LLVMReactor: Finalize the debug info earlier
2019-05-03 bclayton@google.com LLVMReactor: Fix the signature of a few instrinsics
2019-05-03 bclayton@google.com LLVM: Fix for coroutines with array allocas
2019-05-03 chrisforbes@google.com Add input attachment support for VK_FORMAT_R5G6B5_UNORM_PACK16
2019-05-03 chrisforbes@google.com Add support for sampling VK_FORMAT_D16_UNORM
2019-05-03 chrisforbes@google.com Add input attachment support for VK_FORMAT_A2B10G10R10_UINT_PACK32
2019-05-03 chrisforbes@google.com Add support for VK_FORMAT_A2B10G10R10_UINT_PACK32 render targets
2019-05-03 chrisforbes@google.com Fix blitter clears of 2D views of 3D textures
2019-05-03 swiftshader.regress@gmail.com Regres: Update test lists @ 10f7befc


Created with:
  gclient setdep -r third_party/externals/swiftshader@bb12b0e41a09

The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-skia-autoroll

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:Test-Debian9-Clang-GCE-GPU-SwiftShader-x86_64-Debug-All-SwiftShader
TBR=robertphillips@google.com

Change-Id: I289b3bc4d231bd50dd5ee9dca76254aa9e898b83
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212140
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-06 04:35:09 +00:00
skia-recreate-skps
3fd1841161 Update SKP version
Automatic commit by the RecreateSKPs bot.

TBR=rmistry@google.com
NO_MERGE_BUILDS

Change-Id: I4d14ebc855de58ff5513d0d231cdfc44473145ef
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212100
Reviewed-by: <skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com>
Commit-Queue: <skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com>
2019-05-05 08:54:21 +00:00
Hal Canary
ac55784e33 Tests: re-sort gni, file name convention
Change-Id: I47c4b91805770bd68101e1503a8630243ef16c21
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212022
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Herb Derby <herb@google.com>
Auto-Submit: Hal Canary <halcanary@google.com>
2019-05-04 02:17:09 +00:00
Ben Wagner
feae867f84 Add harfbuzz to tools/install_dependencies.sh
No-Try: true
Change-Id: Ida3e485a7b3d218833b314f7b1cf81c9a44e3f6d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212033
Auto-Submit: Ben Wagner aka dogben <benjaminwagner@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2019-05-03 23:52:09 +00:00
Ben Wagner
8ac78aa6df Update the list of Skia dependencies
No-Try: true
Change-Id: I66652ee2ea6afa6854387f12b203908bb99aad4b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212027
Commit-Queue: Ben Wagner aka dogben <benjaminwagner@google.com>
Auto-Submit: Ben Wagner aka dogben <benjaminwagner@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2019-05-03 21:31:39 +00:00
Florin Malita
1b1b657488 Fix .bzl skottie_lib_hdrs
TBR=
Change-Id: Id50b30077045477e4a6b0dfef3b9d950ac8f40f7
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212023
Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Ben Wagner aka dogben <benjaminwagner@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
2019-05-03 19:46:55 +00:00
Ben Wagner
c5eb81bf44 Fixes for experimental/go-skia
No-Try: true
Change-Id: I7799cde0a361984ed958386d2568a959a6cf12ac
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212024
Commit-Queue: Ben Wagner aka dogben <benjaminwagner@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
Auto-Submit: Ben Wagner aka dogben <benjaminwagner@google.com>
2019-05-03 19:22:51 +00:00
Hal Canary
ccab1cc6f4 docs/examples/Colors
Change-Id: I61a295daacaf8a4b0350ae7fc7641059a990552a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212020
Commit-Queue: Hal Canary <halcanary@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: Hal Canary <halcanary@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2019-05-03 18:40:17 +00:00
Florin Malita
e4e9551cc8 Add explicit .bzl sources and headers lists for skottie/sksg
(to facilitate atomic repo updates)

Also remove some _INCLUDES (no longer used).

Change-Id: I9788838fb4b837b80a9ac637ab28760f9f4ccbf1
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211994
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Ben Wagner aka dogben <benjaminwagner@google.com>
2019-05-03 18:35:36 +00:00
Hal Canary
70aab82354 Shaper Tests: fix g3 build
Change-Id: Ib53a9e3224bedbbbdb6d4a99e7ed1856ff9a6717
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211997
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2019-05-03 18:10:46 +00:00
Ethan Nicholas
7e603db010 SkSL interpreter optimizations
Bug: skia:
Change-Id: If1e4916fc40fa5799dabd446e61b840ba3a32133
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211989
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Mike Reed <reed@google.com>
2019-05-03 17:32:36 +00:00
Hal Canary
56947a7fa3 Shaper Tests: make a macro, rename test resources.
Change-Id: Ic7bfd4784d9b2031fdcbf57e730b8c738b9ac9d3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211990
Auto-Submit: Hal Canary <halcanary@google.com>
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Herb Derby <herb@google.com>
2019-05-03 17:16:36 +00:00
Hal Canary
c2f260ad78 SkColors: Fix new color constants
Change-Id: I4ed40624f64aada1163585aa2a8bd0b643b2ccec
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211992
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
Auto-Submit: Hal Canary <halcanary@google.com>
2019-05-03 16:50:46 +00:00
Florin Malita
8b7768d206 [skottie] Relocate "GMs" to resources/
... to make them available on test bots.

TBR=mtklein@google.com

Change-Id: I107a042fbd20c673e964273291ce8c060c522a59
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211991
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Florin Malita <fmalita@chromium.org>
2019-05-03 16:14:46 +00:00
Herb Derby
09db9d2175 Reland "SkShaper JSON output with cluster visualization"
This is a reland of 53610832a0

Fixes conversion of size_t to int.

Original change's description:
> SkShaper JSON output with cluster visualization
>
> A simple JSON output for diagnostic purposes.
> If the run is not 1:1 code points to glyphs, then
> break the run into clusters.
>
> Change-Id: I06980e0bac2cdca8a69b5b5ba0759a021fd4eb3b
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209740
> Commit-Queue: Herb Derby <herb@google.com>
> Reviewed-by: Julia Lavrova <jlavrova@google.com>

Change-Id: I712293c4820eb23234d64fa019d28bac8b105637
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211986
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Herb Derby <herb@google.com>
2019-05-03 15:48:46 +00:00
Jim Van Verth
35a67eba61 Move GrMtlBufferManager functionality to better places
This is setting up for handling buffer recycling in the resource provider.

Bug: skia:8243
Change-Id: Ie850b68bdcac455128aa3e89ae8adf62f17bddea
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211988
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
2019-05-03 15:38:16 +00:00
Hal Canary
fd24b1fe84 Shaper : new unit test
Bug: skia:9050
Change-Id: I4edc6528c4d625f95ce42b1f96fb0c419ce45134
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211831
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Herb Derby <herb@google.com>
2019-05-03 15:02:35 +00:00
Hal Canary
dc19391eef SkShaper: new benchmarks
For the record on my machine:

       2897.46 μs   shaper_tifnagh
       2912.11 μs   shaper_hebrew
       2943.12 μs   shaper_english
       2955.57 μs   shaper_greek
       2966.55 μs   shaper_cyrillic
       3057.37 μs   shaper_lao
       3067.87 μs   shaper_nko
       3364.01 μs   shaper_han_traditional
       7876.46 μs   shaper_han_simplified
       9060.30 μs   shaper_ethiopic
      13202.39 μs   shaper_balinese
      24200.44 μs   shaper_buginese
      27776.86 μs   shaper_kana
      29691.41 μs   shaper_sundanese
      30949.22 μs   shaper_newtailue
      34719.73 μs   shaper_tibetan
      35545.17 μs   shaper_thai
      36623.54 μs   shaper_taitham
      42048.83 μs   shaper_mandaic
      48342.53 μs   shaper_khmer
      65015.62 μs   shaper_javanese
      82198.24 μs   shaper_tamil
      99166.26 μs   shaper_bengali
      99824.46 μs   shaper_hangul
     101777.10 μs   shaper_sinhala
     105471.68 μs   shaper_cherokee
     131298.10 μs   shaper_myanmar
     134949.71 μs   shaper_thaana
     143183.11 μs   shaper_devanagari
     177995.12 μs   shaper_syriac
     186761.23 μs   shaper_vai
     236699.95 μs   shaper_emoji

Change-Id: I312c8333be2408090797cddfe25ef2ccc6eaf82b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211823
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
2019-05-03 14:22:35 +00:00
Ethan Nicholas
6ad5289ea4 Revert "Child fragment processors are now written as separate functions"
This reverts commit 0364bde2c4.

Reason for revert: bot breakage, e.g. https://chromium-swarm.appspot.com/task?id=44942724fab4b710 and https://chromium-swarm.appspot.com/task?id=44938767e5fbb810
Original change's description:
> Child fragment processors are now written as separate functions
> instead of inline
> 
> Bug: skia:
> Change-Id: I3c6c876fea9cfcc311fc09c0fdf0375b776004aa
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/210632
> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
> Reviewed-by: Michael Ludwig <michaelludwig@google.com>

TBR=ethannicholas@google.com,michaelludwig@google.com

Change-Id: If33c1f554a090cd7541878e77712d5ce968ec70a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211943
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
2019-05-03 13:13:47 +00:00
Mike Reed
670c6ec66e Revert "SkShaper JSON output with cluster visualization"
This reverts commit 53610832a0.

Reason for revert: trying to fix flutter roll

Original change's description:
> SkShaper JSON output with cluster visualization
> 
> A simple JSON output for diagnostic purposes.
> If the run is not 1:1 code points to glyphs, then
> break the run into clusters.
> 
> Change-Id: I06980e0bac2cdca8a69b5b5ba0759a021fd4eb3b
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209740
> Commit-Queue: Herb Derby <herb@google.com>
> Reviewed-by: Julia Lavrova <jlavrova@google.com>

TBR=herb@google.com,jlavrova@google.com

Change-Id: I19f8e40032378a856453d059fb5dcdb2f117b75c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211940
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
2019-05-03 11:30:37 +00:00
skia-autoroll
fd105f1b76 Roll third_party/externals/angle2 3089f92e8a77..ddc4d33a259a (3 commits)
3089f92e8a..ddc4d33a25


git log 3089f92e8a77..ddc4d33a259a --date=short --no-merges --format='%ad %ae %s'
2019-05-02 jmadill@chromium.org Remove several underused platform configs.
2019-05-02 jmadill@chromium.org Remove fuchsia_x64 from CQ.
2019-05-02 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src 32af42616abe..c8b09744c6a1 (2 commits)


Created with:
  gclient setdep -r third_party/externals/angle2@ddc4d33a259a

The AutoRoll server is located here: https://autoroll.skia.org/r/angle-skia-autoroll

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:Build-Debian9-Clang-x86_64-Release-ANGLE;skia.primary:Perf-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC5i7RYH-GPU-IntelIris6100-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Perf-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-AlphaR2-GPU-RadeonR9M470X-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUC6i5SYK-GPU-IntelIris540-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-NUCD34010WYKH-GPU-IntelHD4400-x86_64-Debug-All-ANGLE;skia.primary:Test-Win10-Clang-ShuttleC-GPU-GTX960-x86_64-Debug-All-ANGLE
TBR=ethannicholas@google.com

Change-Id: I12bb8f476f1623341174cf95602ab5252c508f81
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211906
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-03 05:20:45 +00:00
skia-autoroll
590c017f45 Roll ../src 1745a957293f..b81b70981d9b (397 commits)
1745a95729..b81b70981d


Created with:
  gclient setdep -r ../src@b81b70981d

The AutoRoll server is located here: https://autoroll.skia.org/r/chromium-skia-autoroll

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-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Release-All-CommandBuffer;skia.primary:Test-Mac10.13-Clang-MacBookPro11.5-GPU-RadeonHD8870M-x86_64-Debug-All-CommandBuffer
TBR=ethannicholas@google.com

Change-Id: Ib0b213e12adbe0632b7588f6332615eaaf70b2c5
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211908
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-03 04:40:15 +00:00
skia-autoroll
58d7a99b6d Roll third_party/externals/swiftshader 20aab4e9c149..10f7befc49a7 (17 commits)
https://swiftshader.googlesource.com/SwiftShader.git/+log/20aab4e9c149..10f7befc49a7


git log 20aab4e9c149..10f7befc49a7 --date=short --no-merges --format='%ad %ae %s'
2019-05-03 capn@google.com Fix overwriting of filter factor
2019-05-02 chrisforbes@google.com Add support for sampling VK_FORMAT_B10G11R11_UFLOAT_PACK32
2019-05-02 chrisforbes@google.com Add support for sampling VK_FORMAT_D32_SFLOAT (without depth compare)
2019-05-02 capn@google.com Enable 3D texture sampling
2019-05-02 chrisforbes@google.com Add support for sampling VK_FORMAT_E5B9G9R9_UFLOAT_PACK32
2019-05-02 chrisforbes@google.com Refactor float sampling path to be more regular
2019-05-02 hliatis@google.com Add Gralloc usage flags
2019-05-02 chrisforbes@google.com Remove old isSRGBReadable/Writable queries
2019-05-02 chrisforbes@google.com Add support for sampling VK_FORMAT_A1R5G5B5_UNORM_PACK16
2019-05-02 amaiorano@google.com CMakeLists: add DEBUGGER_WAIT_DIALOG compiler definition to Debug builds of libEGL and libvk_swiftshader
2019-05-02 capn@google.com Implement sampler LOD bias
2019-05-02 capn@google.com Refactor duplicate mipmap LOD calculation
2019-05-02 capn@google.com Removed unused texture data
2019-05-02 swiftshader.regress@gmail.com Regres: Update test lists @ 20aab4e9
2019-05-02 capn@google.com Validate descriptor ID's
2019-05-02 capn@google.com Use sampler data from sampler descriptor
2019-05-02 capn@google.com Pass a separate sampler pointer to the sampling routine


Created with:
  gclient setdep -r third_party/externals/swiftshader@10f7befc49a7

The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-skia-autoroll

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:Test-Debian9-Clang-GCE-GPU-SwiftShader-x86_64-Debug-All-SwiftShader
TBR=ethannicholas@google.com

Change-Id: I118d2297a5f00d33f8bedfe2f1ebdaefd538e4bc
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211905
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-03 04:35:25 +00:00
Ben Wagner
666a9f9f8b Update includes for Wuffs.
Change-Id: Ibdcfc74ff5c1ee6e7bc6b9bdb13d3fd85ff9eece
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211881
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-02 21:48:33 +00:00
Ben Wagner
0b9b1f1847 IWYU for SkTLogic.h
Noticed that there was an include for <array> in there that didn't make
a lot of sense, so cleaned up the others which are hanging around from
older code which was in there.

Change-Id: I77acbb0914989e9bf67ab74dfd842a798ea592f3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/206172
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-02 21:17:37 +00:00
Ethan Nicholas
0364bde2c4 Child fragment processors are now written as separate functions
instead of inline

Bug: skia:
Change-Id: I3c6c876fea9cfcc311fc09c0fdf0375b776004aa
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/210632
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
2019-05-02 20:48:47 +00:00
skia-autoroll
752b740ed6 Roll skia/third_party/skcms 668026c511f3..853f66269c16 (2 commits)
https://skia.googlesource.com/skcms.git/+log/668026c511f3..853f66269c16

2019-05-02 brianosman@google.com Remove headers from gni source lists
2019-05-02 brianosman@google.com Remove non-explicit endian 16-bit pixel format aliases


The AutoRoll server is located here: https://autoroll.skia.org/r/skcms-skia-autoroll

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=luci.chromium.try:linux-blink-rel
TBR=ethannicholas@google.com,mtklein@google.com

Change-Id: I06c0b83cfa584618dafc270f14fdcf8460d87b91
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211840
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
2019-05-02 20:33:17 +00:00
Hal Canary
7c70143055 GMs: re-sort gni, script to regenerate gni, file name convention
Change-Id: I4b61c2a66f8d6b2d8c4a5ffd8c373b643415c70c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211598
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
2019-05-02 20:08:37 +00:00
Hal Canary
0a7b393e32 SkPaint: new constructor
Change-Id: I5033070bf5d52571869a687387b4f4fc6dddfdd2
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211597
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Mike Reed <reed@google.com>
2019-05-02 19:52:07 +00:00
Herb Derby
53610832a0 SkShaper JSON output with cluster visualization
A simple JSON output for diagnostic purposes.
If the run is not 1:1 code points to glyphs, then
break the run into clusters.

Change-Id: I06980e0bac2cdca8a69b5b5ba0759a021fd4eb3b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209740
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Julia Lavrova <jlavrova@google.com>
2019-05-02 19:22:13 +00:00