Age | Commit message (Collapse) | Author |
|
6630269822 am: 644cd98ceb
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/12200927
Change-Id: Id9a64b48207fdc10b142974d8ab78847f545f010
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/12200927
Change-Id: I9f61dd5b8ed77b66c3e00ef385ec795703864b7c
|
|
Bug: b/160958008
Test: SystemWebViewShell.apk
- Tap 3 dot menu
- Tap "Animation test"
- Check "Rounded corner" checkbox
Roll Skia from b43313646796 to 411090f66974 (1 revision)
https://skia.googlesource.com/skia.git/+log/b43313646796..411090f66974
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-next-autoroll
Please CC scroggo@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
Change-Id: I64fe28313866b775b2ecf78b111b3fab7623a99b
|
|
Bug: b/160958008
Test: SystemWebViewShell.apk
- Tap 3 dot menu
- Tap "Animation test"
- Check "Rounded corner" checkbox
Change-Id: I67851cb4be1a95f9f1d813846f5d4ba4b1535d9c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304176
Reviewed-by: Chris Dalton <csmartdalton@google.com>
|
|
614cb53c20
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/11933884
Change-Id: I17c97f9d054e2d1eda6272063a6fbabd82982cf7
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/11933884
Change-Id: I22d771f7903f274c9a56eb35fec2ac19270c46cc
|
|
https://skia.googlesource.com/skia.git/+log/cf8c53436c77..b43313646796
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-next-autoroll
Please CC stani@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
Bug: 159177162
Test: Presubmit checks and Skia's test infrastructure.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Merged-In: I217b84628a0d652336bb170804f094d4f9cf6c85
Change-Id: I0cba53ebbf069ac135bba4e206843a8e761daf2e
|
|
In the old code, the cache was checked and perged on every reattach
of a strike. That was left out of the multi-threaded code. Add the
check and purge back in to maintain cache size.
* Removed minimum cache size from setCacheSizeLimit.
Bug: skia:10046
Change-Id: I2438c83f04b6da8133c161a29604c3c3d7f58cd8
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/277066
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
(cherry picked from commit 3b946c16180ca2df5b239d410829c046bb74ceb2)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/297599
Auto-Submit: Derek Sollenberger <djsollen@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
|
|
bc4f4f3ba6
Change-Id: Ibca8cf0fa83d8c3bea0a153dd0d32dec9df3e261
|
|
Change-Id: Id6620657fd6dc8f41579adaf90604357d8383856
|
|
https://skia.googlesource.com/skia.git/+log/7eb1d296c06b..cf8c53436c77
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-next-autoroll
Please CC egdaniel@google.com,nifong@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: 147686308
Merged-In: I36c0eb43dd1b65826478f3e3c285268b618175dd
Change-Id: I68c33de98e667563f87e868becc8940ff8453e81
|
|
rvc-dev am: e8302fb6aa am: cb9390e70b
Change-Id: I6892d085df4f08abb40f2b58d1e4e8c87fd94494
|
|
rvc-dev am: e8302fb6aa
Change-Id: Ic5b898cf0baeb6601f30abeb18a1710ca0912548
|
|
|
|
https://skia.googlesource.com/skia.git/+log/3e4e9ab7c20e..7eb1d296c06b
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-next-autoroll
Please CC westont@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: 147686308
Merged-In: Ib1da283eb4184b99b924dceb5b8f6bd557047d8b
Change-Id: I6816805651b951bec62dd377ec9ce4821bdc6e3d
|
|
This fixes in issue on some mobile gpu's where the larger negative value
caused precision issues when trying to interpolate.
Change-Id: Id76e6f96be2a7e46720794f54c24dafe567c5836
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/284956
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
(cherry picked from commit 1204c4bc3826414e34c8fb93aca7376cbd8fc36c)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/285416
|
|
Change-Id: Id1ee9770d011af29b5a0d4e9d2a9fd1452e00586
|
|
Change-Id: I7610babee4e66756fa5a5ee43d95fdf979f51d4d
|
|
We cherry-pick the appropriate part of
https://skia-review.googlesource.com/c/skia/+/279136 to rvc-dev
to make sure we don't have a nullptr dereference. Since that
CLs is quite large, we chose to just make this small change
instead of introduce a large change.
Bug: 154661346
Test: TreeHugger
Change-Id: I01ac23547a345b1aed8048cd83cb7a1f1e36e331
Merged-In: I7c990413b7a966f6a330289552d4520be0f3d3d1
|
|
Bug: skia:9438
Change-Id: I1326f55fa8546d141c43ef4c6fcee6f65ed0e600
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281016
Reviewed-by: Eric Boren <borenet@google.com>
|
|
d9cee2a69b
Change-Id: Ia266e9e2901e1ef9d9df604b933bfb2f346d1417
|
|
Change-Id: I156602e0d65210cf5da74c5b347e86b077f25334
|
|
https://skia.googlesource.com/skia.git/+log/178b86076996..3e4e9ab7c20e
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-next-autoroll
Please CC stani@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Test: Built on Android
Bug: 142426387
Change-Id: I47782f5fc677b361f243942181af968d73d6eae8
|
|
Test: Built on Android
Bug: b/142426387
Change-Id: I7ca32ffedd8896cbb105872cd1b3c746ff8ffe9d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/282837
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Stan Iliev <stani@google.com>
(cherry picked from commit 1db4d14e595ddfd51f89b0d76c5f2f762e2db856)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/283076
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|
|
Change-Id: Ifc1b9b2e061959515eb6ca3527c541b75e566ddb
|
|
Change-Id: Ib90bcceb6a28d879a79eb4d456d7a2921ee27eb5
|
|
Bug: 150353577
Test: Infeasible
Cherry-pick of 9eb78748448b853074c16d99da8bff42b6791782 in Skia.
If malloc fails, don't let SkJpegCodec attempt to decode into it.
Return kInternalError, which is what we return for OOM.
Change-Id: I6750e907c5e649d9e12ba75e84c8d9e91a66dd61
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/273809
Auto-Submit: Leon Scroggins <scroggo@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Merged-In: Iaec5e7d0f8603124acb8238bd4d12457f09e9090
Change-Id: I41527dc95c5565bc8985f0003260840e69555caa
|
|
https://skia.googlesource.com/skia.git/+log/77521d9e06e8..178b86076996
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/android-master-autoroll
Please CC scroggo@google.com on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Test: Presubmit checks will test this change.
Change-Id: Ic3a55116f9f74be47ca5e8c9c662932d0932a39f
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
|
|
When per-character 3D is enabled, text properties can be animated in
3 dimensions.
- position and scale become 3-value vectors
- in addition to existing "r" (really rz), rotation gains "rx" and "ry"
- instead of specializing for 3D, expand the existing structures to
handle both 3D and 2D modes
- also ensure that sksg::Transform does not flatten to SkMatrix
Change-Id: I426a7ee1ff38c1702deb85e9f1db80f6069f36d6
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272648
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
|
|
Change-Id: Ie3c08d79ae36f28ff48f825699a2542154ca3096
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272652
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
|
|
This is a reland of a6cd7c0b1f989840efe203979943cbb59d4d131b
Original change's description:
> Multi-threaded strike cache
>
> Allow multiple threads to share the same strike. The old
> system of removing the cache from the linked list is no longer.
> The strikes stay in the list and can be found by other threads.
>
> * Removed strike size verification. There was no way to get the
> locks to work out. The whole point of the change was to have multiple
> threads muting the structure at the same time.
>
> * Strikes are now refed instead of being checked out. Therefore,
> ExclusiveStrikePtr is now just wraps an sk_sp, and should be renamed
> in a future CL.
>
> Change-Id: I832642332a3106e30745f9cdd3156ae72d41fd0b
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272057
> Reviewed-by: Ben Wagner <bungeman@google.com>
> Commit-Queue: Herb Derby <herb@google.com>
Change-Id: Id02381de93ff82bca58f09e07a457883d57d5565
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272436
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
|
|
GrGLGpu.
Our contract is that we will call the finishedCallbacks at some point. So if we
are either abandoning or just deleting the GrGLGpu make sure we call any outstanding
callbacks.
Change-Id: I1425e951185d350a1faa567f0342822c41aafb65
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272650
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
|
|
I've been increasingly concerned about the CPU and memory costs of
creating and holding on to many Compiler instances. Given that they're
use infrequently, having a shared (guarded) instance seems fine.
We can still look at reducing the startup cost of a single instance, but
this removes a large blocker to wider usage of SkRuntimeEffect.
Change-Id: Ia6dc721c73fdf8c9c4c7a8c1af5f350d2c028b22
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272466
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
|
|
Fix a compilation error in SkiaDawn by adding an onBindPipeline()
implementation and renaming onDraw() to onDrawMeshes(). onBindPipeline()
does nothing because onDraw() previously did not use the drawBounds
parameter.
Change-Id: Icf3d49296359bfdee8936cc63d6b5d33b597fb4a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272511
Commit-Queue: Stephen White <senorblanco@chromium.org>
Reviewed-by: Stephen White <senorblanco@chromium.org>
|
|
Bug: skia:9935
Change-Id: Ib6548f413ca3a8befb553d2d47354b400c9162b9
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272520
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
|
|
SkSLSlide was triggering this assert - now it prints an error every
frame, but doesn't crash.
Bug skia:9941
Change-Id: I4c02a89c8d824acc71ab595af99e1df2f5fc980d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272639
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Tyler Denniston <tdenniston@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Auto-Submit: Brian Osman <brianosman@google.com>
|
|
Change-Id: I90e0918b1ce6df7f54fae67916754289eb9a263b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272638
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
|
|
Change-Id: Id26530bbd54626a74bfb4fccd4c066fa39346411
Bugs: skia:9892
getGlyphPositionAtCoordinate should return correct code point index
Change-Id: Id26530bbd54626a74bfb4fccd4c066fa39346411
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272347
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Julia Lavrova <jlavrova@google.com>
|
|
This reverts commit ad3b2c98865dac6d7947fcf92c0084df0eb818fa.
Reason for revert: clang
Original change's description:
> Move SkDeferredDisplayList.h into include\core
>
> Clients will need official access to this class for the compilation iterator.
>
> This CL also hides some of the cruft we don't want exposed.
>
> Bug: skia:9455
> Change-Id: I408c19f9ecd6880a5a7853def591407b0ca43e4e
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272343
> Commit-Queue: Robert Phillips <robertphillips@google.com>
> Reviewed-by: Brian Salomon <bsalomon@google.com>
TBR=bsalomon@google.com,robertphillips@google.com
Change-Id: Ica80434e7423fb202355eb77a614ece1c4d54726
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:9455
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272641
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
|
|
Clients will need official access to this class for the compilation iterator.
This CL also hides some of the cruft we don't want exposed.
Bug: skia:9455
Change-Id: I408c19f9ecd6880a5a7853def591407b0ca43e4e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272343
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
|
|
This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (depot_tools) into this repository.
depot_tools:
https://crrev.com/7e4c38a48b04677803505d759ba4b0623141f661 (sque@google.com)
Roll out new goma client binary VERSION=191
https://crrev.com/1eb16f67982b134323bd89c9f639ba1908700b88 (brucedawson@chromium.org)
Fix build summary to classify *.mojom*.obj as .obj
More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
TBR=borenet@google.com
Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I7d66192a4f9c73972393a2399653fd27e6f8be0f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272616
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>
|
|
Change-Id: I7dedf2c8724256cf6889d8a608b425cb9188c030
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272597
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
|
|
Change-Id: I2d44eb5b1984212fb461977f5e9ee95717046a8e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272467
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
|
|
private copy of it
Change-Id: I4c390fa4e2eae71203c688433569a7afb5e02642
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272535
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Reed <reed@google.com>
|
|
Bug: skia:9799
Change-Id: I94069d58bde1762ef19122605147bd75232b6ed8
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/270446
Commit-Queue: Ben Wagner aka dogben <benjaminwagner@google.com>
Reviewed-by: Weston Tracey <westont@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
|
|
I'm about to add more of these, wanted to land this first.
Change-Id: Ic7ec30f531456075b1f9032f1f07cc54d234af95
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272527
Auto-Submit: Brian Osman <brianosman@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
|
|
Needed for https://skia-review.googlesource.com/c/skia/+/270446
Bug: skia:9799
Change-Id: I0b5814aa3bae73ca6c231b5ced7a0c94e0efa6dc
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/270447
Commit-Queue: Ben Wagner aka dogben <benjaminwagner@google.com>
Reviewed-by: Weston Tracey <westont@google.com>
|
|
Need to migrate clients from private/ to core/ include
Unexperimentalize concat44() methods on SkCanvas
Change-Id: I64b8816722a9d93316cb8b8691d2d9a3e36f167f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272464
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Mike Reed <reed@google.com>
|
|
Uniforms in the general case can be multiple floats, so
we expose a small struct to provide information about all
uniforms (including the built in ones like "dt") as well
as effectUniforms and particleUniforms which return Float32Arrays
bound to the WASM memory containing those unifroms.
Thus, by modifying the Float32Array, one can directly affect
the particle/effect properties.
This allows us to expose sliders on particles.skia.org
(https://skia-review.googlesource.com/c/buildbot/+/272398)
Change-Id: Ie390f3d2dc571ee4ebaab59a7fa1b7b2dc24d871
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272377
Reviewed-by: Brian Osman <brianosman@google.com>
|
|
This reverts commit 7fba244ea9d446eb8e4b36f9c3acc161b3db6dcf.
Reason for revert: See if this is blocking the Chrome roll
Original change's description:
> Remove copies for wrap modes in GrTextureProducer.
>
> GrTextureEffect now supports implementing wrap modes in shaders
> for subsets and texture types with HW sampling restrictions.
>
> Change-Id: I5c93ade044465e13c5f56f7437fbbe288db0a8a8
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/271056
> Reviewed-by: Greg Daniel <egdaniel@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>
TBR=egdaniel@google.com,bsalomon@google.com
Change-Id: I14397bec8ff4ba165c28faa8f44497f47d865862
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272522
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
|