Need to compare all elements, then fold the result to a single bool.
Change-Id: I0ebfaa9d518f29a782701246ada247cb55c01c2e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216607
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
This saves texture reads when we need to scale in x or y but not both
when doing an async rescale and read.
Change-Id: I9e3550585875a1b42db7a2a7bb8bfacf3e840a01
Bug: skia:8962
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216520
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
This reverts commit d0d66fb8be.
Reason for revert: crashing on readpixels on some android bots
Original change's description:
> Remove call to copy in GrcontextPriv read/write pixel calls.
>
> Instead we just directly do all those copies as draws.
>
> Change-Id: I0cd9dfc6f96e35fbbc9c153a28a08eebf1d7b77f
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216356
> Commit-Queue: Greg Daniel <egdaniel@google.com>
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Robert Phillips <robertphillips@google.com>
TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com
Change-Id: I2b39564e0b20cf83e21744c91ee8ddb9d988ab22
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216608
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
In some cases, an overlap boundary edge can be nulled out by the creation
of a skeleton edge. In that case, we should avoid trying to create
parallel edge events from that dead edge.
This required making the tessellator verb maximum verb count modifiable
at runtime for testing, since the test case has more than 10 vertices.
Also added more logging.
BUG: 966696
Change-Id: I429735999f6297655311485bc68d732b1c48bfce
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216284
Commit-Queue: Stephen White <senorblanco@chromium.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Instead we just directly do all those copies as draws.
Change-Id: I0cd9dfc6f96e35fbbc9c153a28a08eebf1d7b77f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216356
Commit-Queue: Greg Daniel <egdaniel@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Per GLSL, constructing a matrix from a scalar produces a matrix
with the scalar value along the diagonal, and zero elsewhere.
Constructing a matrix from another matrix copies the overlapping
values, and fills in the remainder with the identity matrix.
Doing either of these with existing opcodes was going to be quite
verbose and tricky, so I just made new opcodes.
I've also got some (currently disabled) test cases for other
matrix behavior, all of which fail in various ways today.
Change-Id: Ia86a183395f1ac7e2f23ee1d6bb4af461f5ba93a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215823
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
This will move the cache to source ration onto the SkStrikeSpecStorage
instead of passing it along as an extra parameter.
Change-Id: Ie19665c8cd8f02798f9ea75d421a4c9337eb1f37
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216358
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Herb Derby <herb@google.com>
3fe06eb3aa..ded5f90326
git log 3fe06eb3aaaf..ded5f903261a --date=short --no-merges --format='%ad %ae %s'
2019-05-28 geofflang@chromium.org Vulkan: Make the Vulkan renderer thread safe.
2019-05-28 geofflang@chromium.org Vulkan: Use atomics in SerialFactory.
2019-05-28 jmadill@chromium.org D3D11: Suppress failing Texture test on AMD.
2019-05-28 geofflang@chromium.org Vulkan: Move command graph and garbage to ContextVk.
2019-05-28 jonahr@google.com Addition of dEQP "KHR" test variants to generate_stats
2019-05-28 geofflang@chromium.org GL: Allow BlitGL to copy from external textures.
2019-05-28 geofflang@chromium.org D3D: Use the ProgramD3D::GetExecutableTask context for loading program binaries
2019-05-28 syoussefi@chromium.org Vulkan: Have a cubemap as 2D-array view handy
2019-05-28 jonahr@google.com Cleanup Features headers.
2019-05-28 aleino@nvidia.com Use D3D11 GetDimensions driver workaround for 2D array textures
2019-05-28 aleino@nvidia.com Test D3D11 GetDimensions driver workaround for textureProjOffset
2019-05-28 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src 42abaa099aab..6c7db9c63098 (3 commits)
Created with:
gclient setdep -r third_party/externals/angle2@ded5f903261a
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=kjlubick@google.com
Change-Id: Ic6e1d6e41c7d87eeaa1c31ee560a544d7c0a3129
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216562
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
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/2ee83757a109f65cd4e831b257385a258f99c9d5 [autoninja] use custom status in non-windows too (tikuta@chromium.org)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I671e6f5d709d3699fb05fca25c07d2f8e5d0b5cb
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216569
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>
a4545bae03..b3e0873d64
Created with:
gclient setdep -r ../src@b3e0873d64
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=kjlubick@google.com
Change-Id: I901fad2d92f444842e104652f06c9e43a5789e20
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216563
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
https://swiftshader.googlesource.com/SwiftShader.git/+log/710fcd517ea4..ca8e3d7c153e
git log 710fcd517ea4..ca8e3d7c153e --date=short --no-merges --format='%ad %ae %s'
2019-05-28 bclayton@google.com Reactor: Fix printing of small integer values.
2019-05-28 bclayton@google.com SpirvShader: Correctly handle phi values in the loop merge
2019-05-28 bclayton@google.com SpirvShader: Simplify the logic around the generation of the inner loop.
2019-05-28 bclayton@google.com LLVMReactor: Mutex calls to llvm::orc::IRCompileLayer.
2019-05-28 bclayton@google.com Reactor: Add UInt4 constructors taking UInt
2019-05-28 bclayton@google.com Reactor: Fix all memory leaks with rr::DebugInfo.
2019-05-28 chrisforbes@google.com Fix accidentally considering Workgroup to be explicit layout
2019-05-28 bclayton@google.com Reactor: Add swizzle op for UInt4.
2019-05-28 capn@google.com Support images with linear tiling for transfers
2019-05-28 capn@google.com Implement Y′CbCr conversion
2019-05-28 capn@google.com Fix single-plane views of YCbCr formats
2019-05-28 capn@google.com Refactor writing texture level descriptor info
2019-05-28 capn@google.com Take the plane/aspect into account when copying/blitting
2019-05-28 capn@google.com Implement VkSamplerYcbcrConversion object
2019-05-28 capn@google.com Claim support for YCbCr 3- and 2-plane 420 format
2019-05-28 capn@google.com Allow additional info to be passed to object's Create functions
2019-05-28 capn@google.com Add 8-bit YCbCr 420 format info
2019-05-28 capn@google.com Refactor obtaining format aspect info
2019-05-28 capn@google.com Implement OpImageGather
2019-05-28 capn@google.com Refactor texel names
2019-05-28 capn@google.com Fix taking the sampling instruction type into account
2019-05-28 capn@google.com Fix interpolating z when using FragCoord builtin
2019-05-28 swiftshader.regress@gmail.com Regres: Update test lists @ 710fcd51
Created with:
gclient setdep -r third_party/externals/swiftshader@ca8e3d7c153e
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=kjlubick@google.com
Change-Id: I452dcc6f9eedb321519f7f0f3b5ebd6ed60eb96a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216561
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
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.
recipe_engine:
https://crrev.com/d90d37f751848195850d2af1a300d9c95dd2a2b8 Remove unused name_prefix complexity. (iannucci@chromium.org)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I05692518a156e8b4110d650cd661cead3e83c4bb
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216494
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>
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.
recipe_engine:
https://crrev.com/21625832430557848a1bb6336f10bd4a86e0ac78 [post_process] Make output_properties have decoded values. (iannucci@chromium.org)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I511bba6a950b88f245859bd99182ba1ef8a33758
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216481
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>
Also shuffle the search order to better match use frequency.
Change-Id: I8ba7f5474f0937aecb75215a1129b439f89a7dbf
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216357
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Ben Wagner <bungeman@google.com>
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.
recipe_engine:
https://crrev.com/03e509d455281ec684552595e49b77e66bca6c70 Add --no-emoji flag to simulation tests. (iannucci@chromium.org)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I7fcbdd4ccc835551b96cef22534aac1ec3dc60b3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216444
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>
The SkGlyphRunPainterInterface passed a strike cache in,
but it was only ever needed to get Descriptor. Pass a narrower
object in that just handles strike specifications.
glyphCache -> strike in several places
Change-Id: I98dd9e527149177e61ea26eba64b7868765ac64e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216349
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
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).
Please review the expectation changes, and LGTM+CQ.
More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
recipe_engine:
https://crrev.com/c2ae1f8b058dc210e2f8b1753aa7184973da36a6 Remove stdout and stderr from expectation files. (gbeaty@chromium.org)
R=stephana@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Idc32b8c2c426c0bc60b0c57476258b649d9620d1
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216355
Commit-Queue: Eric Boren <borenet@google.com>
Reviewed-by: Robbie Iannucci <iannucci@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
This also simplifies the internalStencilClear() paint configuration
to use GrDisableColorXP instead of a transparent black paint.
Without the fix to finalize(), the use of GrDisableColorXP would cause
GLPrograms unit tests to assert when run on a device that needed to
draw for its stencil clears. What would happen is that an initial
stencil clear would add an op with no FPs and the disabled-color XP.
Then the unit test would generate a program with an FP that didn't
support coverage as alpha and used the disabled-color XP. The FP
would disable the coverage-as-alpha optimization, but then the XP
would remove all FPs. This lead to processor sets that were considered
equal (same XP, no FPs), but had different results from finalize().
Change-Id: I6387234a6834028b417a63b9045b9098f72bec75
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216354
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: Michael Ludwig <michaelludwig@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
In preparation for text range selectors, update Shaper and the text
adapter logic to deal with multiple text "fragments".
- Shaper::Result is now an array of (blob,pos) tuples
- TextAdapter builds an arbitrary number of scene graph nodes, based on
the Shaper fragments
Change-Id: I0f2ed86da77e9aaf22b9cb138c3a5f8f957393fe
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216403
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
This reverts commit dbddfff933.
Reason for revert: Fix CL by removing use of GrDisableColorXP during
stencil clear, which confuses processor analysis. Will fix analysis
and restore GrDisableColorXP in later CL.
Original change's description:
> Revert "Update GrRTC to use GrFillRectOp::MakeGeneric"
>
> This reverts commit ffd4da3cd4.
>
> Reason for revert: Likely breaking Adreno 405 bot
>
> Original change's description:
> > Update GrRTC to use GrFillRectOp::MakeGeneric
> >
> > Change-Id: I4b137c545e726a49c7c5a64febdb83388f335e08
> > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215082
> > Commit-Queue: Michael Ludwig <michaelludwig@google.com>
> > Reviewed-by: Brian Salomon <bsalomon@google.com>
>
> TBR=bsalomon@google.com,robertphillips@google.com,michaelludwig@google.com
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> Change-Id: I495da5181e71b98a280c53787e8e3279a5d7b162
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216281
> Reviewed-by: Michael Ludwig <michaelludwig@google.com>
> Commit-Queue: Michael Ludwig <michaelludwig@google.com>
TBR=bsalomon@google.com,robertphillips@google.com,michaelludwig@google.com
Change-Id: I367d3724d52bbbbc7d3758cd982798265c0155a2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216342
Commit-Queue: Michael Ludwig <michaelludwig@google.com>
Auto-Submit: Michael Ludwig <michaelludwig@google.com>
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Change-Id: I6ead74a61884967f9a60eba614414bc42079d28b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216343
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
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/778c7f117aee162dece25d84e00504b2b828655e Make tryserver API only update the presentation of its own steps. (iannucci@chromium.org)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I50053de781960e097db4841202244aaa935d004c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216363
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>
40757b28bd..3fe06eb3aa
git log 40757b28bdc8..3fe06eb3aaaf --date=short --no-merges --format='%ad %ae %s'
2019-05-27 geofflang@chromium.org D3D: Make sure Lod0 functions are never referenced in non-fragment shaders.
2019-05-27 jmadill@chromium.org Vulkan: Handle dirty RTs with state messages.
2019-05-27 jmadill@chromium.org Fix KHR dEQP test data dir.
2019-05-27 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src b8fe7211c465..42abaa099aab (3 commits)
Created with:
gclient setdep -r third_party/externals/angle2@3fe06eb3aaaf
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=kjlubick@google.com
Change-Id: Ifc636c23e351a83cc724edf83ca749c0e96cfbcb
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216324
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
017b02e115..a4545bae03
Created with:
gclient setdep -r ../src@a4545bae03
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=kjlubick@google.com
Change-Id: I8a44b1504fe852263faa36a33b7715f246dea06b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216325
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
This reverts commit 186a295111.
Reason for revert: Metal bots failing
Original change's description:
> Change Metal to not take ownership of objects
>
> Prior to this change, Skia/Metal interfaces take ownership of the Metal
> objects passed in (that is, the caller should count passing the object
> to Skia as "freeing" the object).
>
> Change this behavior so that Skia/Metal retains its own separate
> ownership of the Metal objects.
>
> Make GrBackendTexture and GrBackendRenderTarget maintain their own
> references to the underlying MTLTexture by using the CFRetain/CFRelease
> interfaces. Do this by adding a private GrMtlBackendSurfaceInfo.
>
> Move GrMtlBackendSurfaceInfo (formerly GrMtlTextureInfo) out of the
> union in GrBackendTexture and GrBackendRenderTarget because unions
> cannot have nontrivial constructors and destructors (how fVkInfo isn't
> causing a compile error is unclear).
>
> Change-Id: Iae3719c0715825d86503d03c766e47f0f6015bdf
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215685
> Commit-Queue: Jim Van Verth <jvanverth@google.com>
> Reviewed-by: Jim Van Verth <jvanverth@google.com>
TBR=egdaniel@google.com,jvanverth@google.com,bsalomon@google.com,ccameron@chromium.org
# Not skipping CQ checks because original CL landed > 1 day ago.
Change-Id: Ie569fe1938857706b5413876a9480ef1eb3314ea
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216221
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
This reverts commit ffd4da3cd4.
Reason for revert: Likely breaking Adreno 405 bot
Original change's description:
> Update GrRTC to use GrFillRectOp::MakeGeneric
>
> Change-Id: I4b137c545e726a49c7c5a64febdb83388f335e08
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215082
> Commit-Queue: Michael Ludwig <michaelludwig@google.com>
> Reviewed-by: Brian Salomon <bsalomon@google.com>
TBR=bsalomon@google.com,robertphillips@google.com,michaelludwig@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.
Change-Id: I495da5181e71b98a280c53787e8e3279a5d7b162
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216281
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Michael Ludwig <michaelludwig@google.com>
When fitting the shape result to a box, scale both the font size and
line height by the same factor.
This produces more intuitive results.
Bug: skia:9129
Change-Id: I742a952b9615216a2b68c0432b41026751099cbc
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216220
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
e794ddc8a6..40757b28bd
git log e794ddc8a662..40757b28bdc8 --date=short --no-merges --format='%ad %ae %s'
2019-05-24 spang@chromium.org Use ConvertToGLBoolean instead of ternary operator in Context
2019-05-24 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/glslang/src 27ec91958501..66e46037d74f (1 commits)
2019-05-24 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src d9c00e1d2de1..b8fe7211c465 (5 commits)
2019-05-24 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-headers/src 9674a1a54754..7f4277e21b52 (1 commits)
2019-05-24 jiajia.qin@intel.com Fix the crash when the array size is not big enough.
2019-05-24 minkyu.jeong@samsung.com Add support for EXT_blend_minmax
Created with:
gclient setdep -r third_party/externals/angle2@40757b28bdc8
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=kjlubick@google.com
Change-Id: Id73a26bdeae666f930e6a22a44f40e0283ab0901
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216241
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
778c5dab5f..017b02e115
Created with:
gclient setdep -r ../src@017b02e115
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=kjlubick@google.com
Change-Id: I8578cc1c00ca0a1c8c2989ca89c3f99c6fdf865b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216242
Reviewed-by: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com>
Prior to this change, Skia/Metal interfaces take ownership of the Metal
objects passed in (that is, the caller should count passing the object
to Skia as "freeing" the object).
Change this behavior so that Skia/Metal retains its own separate
ownership of the Metal objects.
Make GrBackendTexture and GrBackendRenderTarget maintain their own
references to the underlying MTLTexture by using the CFRetain/CFRelease
interfaces. Do this by adding a private GrMtlBackendSurfaceInfo.
Move GrMtlBackendSurfaceInfo (formerly GrMtlTextureInfo) out of the
union in GrBackendTexture and GrBackendRenderTarget because unions
cannot have nontrivial constructors and destructors (how fVkInfo isn't
causing a compile error is unclear).
Change-Id: Iae3719c0715825d86503d03c766e47f0f6015bdf
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215685
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
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.
recipe_engine:
https://crrev.com/ac1ba2318d5dc0263f74e9aa00aabc95c1b856da [archive] Support extracting Unicode filenames (smut@google.com)
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I7de461c5cc3a36031120d70f7ea5abe42f60301a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/216042
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>
