Age | Commit message (Collapse) | Author |
|
mainline-permission-release
Change-Id: I0c7759a471f19f087a547981f3628f0a036fe4a0
|
|
am: db7ba19b63
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24664224
Change-Id: I52b6ff4000626f08c849eb1487136bf096e1a04e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
e30d6c3f13 am: db2a1e985f
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24664223
Change-Id: I5fa79d9d89ea8d0eaf5a798764ad69776f23a11c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24664224
Change-Id: I96e38318784401b609e8a0eff02dd078151e85cc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
e30d6c3f13
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24664223
Change-Id: Ic698ac22fdd2a28115f282c6890577b1c40d56cf
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Reduces per-test run time.
Test: this
Bug: b/298000572
Bug: b/297313207
Change-Id: Ibc561074c92af242d2e70cd24e0ffd8c7eb2cee5
Merged-In: If337d987262416988e83941c672302ec321654d7
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/746361
Commit-Queue: Nolan Scobie <nscobie@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
|
|
Reduces per-test run time. This manual approach to splitting feels a bit
hacky, but it works and I couldn't find any parameterized test macros.
Test: this
Bug: b/298000572
Bug: b/246008281
Change-Id: I221b9e6179bc429cc6b1471ca2e5dcfc90718272
Merged-In: I2d19944a47e8970ff5cee40a0db148367afd4342
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/745557
Reviewed-by: Leon Scroggins <scroggo@google.com>
Commit-Queue: Nolan Scobie <nscobie@google.com>
|
|
mainline-permission-release
Change-Id: I43272ad65c3f6d47527724e4c7bf6f1d707369f1
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24265941
Change-Id: I2987f81160951b75d05b5c78544ef668c0a9999a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24265940
Change-Id: I3b10a4b971c56d978fcdebe72f2b382096821054
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24265941
Change-Id: Iaf6c08679e53acadd2120baf565c5a801a81db27
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24265940
Change-Id: Ia9200651cbc56971b66a1119d1241f3930388762
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
mainline-permission-release
Change-Id: Idb2a327434d5a1f42e9c9085202c51f59273a6f4
|
|
Conflicts:
- tests/SkSLTest.cpp: Manually update these tests, since most of these
lines have changed in `main`
In this cherry-pick, Use kNextRelease, which corresponds to the U API
level, since CtsEnforcement::kApiLevel_U does not exist in this branch.
Bug: 293367611
Test: atest CtsSkQPTestCases
Change-Id: I17ef93ace98ebd882baa4102681c259968c74230
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/729805
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Nolan Scobie <nscobie@google.com>
Reviewed-by: John Stiles <johnstiles@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
Merged-In: I41b0b20166d4dae68b2faea39f4bc9437383edb4
|
|
When CtsEnforcement was introduced in
Ida2c58a30c70d9a4ae0f64ec943662ba1162e406, this test was enforced at
kApiLevel_T. But this test didn't exist in T. As I understand it, we
should start enforcing the test on the version in which it was
introduced, which would be U in this case.
Update the test to only be enforced starting in Android U. Use
kNextRelease, which corresponds to the U API level, since
CtsEnforcement::kApiLevel_U does not exist in this branch.
Bug: 293367611
Test: atest CtsSkQPTestCases
Change-Id: If85169a2940af76f66b5b21d167213ea6ee910b1
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/704896
Reviewed-by: John Stiles <johnstiles@google.com>
Commit-Queue: Leon Scroggins <scroggo@google.com>
Merged-In: Ie6c0c897b04a482a3aa74b9fa73421a48639230c
|
|
am: da7b2e8ae9 am: e3bcf4cc39
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24061444
Change-Id: I42a27dafa7241b6872c6801d14c176077633d4da
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: da7b2e8ae9
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/24061444
Change-Id: Id590a925c852470514d6b49789d22942e4c7357b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/d64928640a1b..ae193d0cc7e3
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-u-release-autoroll
Please CC rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/289111414
Bug: b/186777432
Bug: b/290581028
Change-Id: I52ae114bd1a95a66814883911b331cb4865eb829
|
|
mainline-permission-release
Change-Id: Ic132af78185cbf983986422d77cdba9825a10e55
|
|
am: 929d127dfb
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/23858210
Change-Id: Ia7499c58dce4230cc76f1e266a4646ef72f022dc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/d64928640a1b..ae193d0cc7e3
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-u-release-autoroll
Please CC rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/289111414
Bug: b/186777432
Change-Id: I52ae114bd1a95a66814883911b331cb4865eb829
|
|
This reverts commit 7ef621bf45c855c9ca20c29ef85a4ee2d7755805.
Revert "Add alph8 gainmap decoding with BigmapRegionDecoder."
This reverts commit cf1ccd4e23d718852d7b02d4f0fb3657fc2e7188.
Reason for revert: We shouldn't do gray8 conversion in skia decoderegion side because this function is not only used for gainmap case but also for the recycled image.
Bug: b/289111414
Change-Id: I25eec7cf26d51c5add8171f567a128fdbfb467c6
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/718301
Commit-Queue: Sally Qi <sallyqi@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
|
|
am: 6b9ad739c3
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/23660131
Change-Id: I9708e1e22ad4f344586c0ab7ef109bd6d00c0456
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/7ef621bf45c8..d64928640a1b
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-u-release-autoroll
Please CC rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/186777432
Change-Id: I69d2838b93c6616551542536b88c6ad386d4222a
|
|
Enough time has passed since we started writing Version 1.0 into the
files that it's now safe to break any files that didn't have this.
Additional changes in the cherry-pick:
- Fix the incorrect return value in extract_gainmap.
- If SkJpegXmp::getGainmapInfoHDRGM specifies nullptr for its
SkGainmapInfo, then only perform the version check.
- Make SkJpegXmp::getGainmapInfoHDRGM's not-just-version-check path
fail if HdrCapacityMax or GainMapMax are absent.
- Add more XMP tests.
Bug: skia:14031
Change-Id: I74048a7f03c5cbdc3302574e5aa78a0ebd81e937
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/700196
Commit-Queue: ccameron chromium <ccameron@chromium.org>
Commit-Queue: Christopher Cameron <ccameron@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
(cherry picked from commit 1b8ff701e6f90d4ac78e5f16523b938731803b61)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/710176
Reviewed-by: Kevin Lubick <kjlubick@google.com>
|
|
am: cbd8639a61
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/23219979
Change-Id: Ib0070a09fdcebfae7b31f06235b07c2f0680be92
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/b76c00e60cf9..7ef621bf45c8
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-u-release-autoroll
Please CC nscobie@google.com,rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/271174821
Bug: b/186777432
Change-Id: I4db62f076fa22df600c29060c61f7877d872ebbb
|
|
- We don't need to recompute output colortype and colorspace here if the
codec info is kGray8.
- If we recompute output colortype here, the desired colortype may
change if hwui side change f16 colortype back to k32 one to match the
recyled bitmap's color type if necessary.
- for kGray8 color type, no need to recompute output colorspace since
color correction is not supported for kGray
Bug: b/271174821
Change-Id: Ic170b15ae5abc33a28c9301076cedb216d46dc81
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/697316
Commit-Queue: Nolan Scobie <nscobie@google.com>
Commit-Queue: Sally Qi <sallyqi@google.com>
Reviewed-by: Nolan Scobie <nscobie@google.com>
|
|
Bug: b/271174821
Change-Id: I854ace468bb385dcd4391f7bb7801caa65563e87
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/696480
Commit-Queue: Nolan Scobie <nscobie@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
|
|
am: 1304bf0028
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/23139639
Change-Id: I999d40d9d0e1cc74cfd5a8a28bd36e72f4bc6ece
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 4f489e8142
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/23128715
Change-Id: I680491da588c55db6ddc7899f3f02b8dc05224e0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/8f929978fd7c..b76c00e60cf9
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-u-release-autoroll
Please CC nscobie@google.com,rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/279811818
Bug: b/186777432
Change-Id: I4dbb58f7c9a35063bfb9f1bc1496af360c4b2831
|
|
Bug: b/279811818
Change-Id: I421fe4157624bfaaeda1c40e6d2d39ef74591c21
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/694651
Commit-Queue: Leon Scroggins <scroggo@google.com>
Reviewed-by: Leon Scroggins <scroggo@google.com>
|
|
https://skia.googlesource.com/skia.git/+log/234093826ebe..8f929978fd7c
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-u-release-autoroll
Please CC nscobie@google.com,rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/278122024
Bug: b/186777432
Change-Id: I4cf2704914d68592a3e3dc0e01c1c10983eb9644
|
|
mainline-permission-release
Change-Id: I3a6f1724f9ed9344c3c3916a1acc2f0f347bd913
|
|
am: e3ab186a07 am: df43f60c4e am: d17c1053f9 -s ours am: a75de6da08 -s ours
am skip reason: Merged-In I84d250c27ce407259c52d12e2e011c0c0a7ae3b0 with SHA-1 3ace9b2024 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: Ib8b8f99dc1c1bf233553bc9618c84dc6c17716fe
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: e3ab186a07 am: 92ef3a428a -s ours am: ff685e5754 -s ours
am skip reason: Merged-In I84d250c27ce407259c52d12e2e011c0c0a7ae3b0 with SHA-1 3ace9b2024 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: I63bdf86cd06e877f70c994397a575e534cce2555
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: e3ab186a07 am: df43f60c4e am: d17c1053f9 -s ours
am skip reason: Merged-In I84d250c27ce407259c52d12e2e011c0c0a7ae3b0 with SHA-1 3ace9b2024 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: If1b17e0e8c78ef4fcef52db82c8e87e4c32cec90
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
df43f60c4e
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: Ia26d9807df324d2a8a46392ab914765c5096ccb7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: e3ab186a07 am: 92ef3a428a -s ours
am skip reason: Merged-In I84d250c27ce407259c52d12e2e011c0c0a7ae3b0 with SHA-1 3ace9b2024 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: I56d951b24493aa2ffbec08c2d250fcc6cceef2b8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: If8fc29fd6a860a345204b7ade4e221f8afdd01e6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22896125
Change-Id: I7f07d0b0f59e688bde3d9ecb1062ca031aeb58f6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
In particular, PQ's transfer function requires really high precision,
so using halfs instead of floats creates artifacts on some GPUs.
Bug: b/278122024
Change-Id: If403103f6f420588869aad00b228e81dd838e94e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/687083
Commit-Queue: Alec Mouri <alecmouri@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
(cherry picked from commit 1c2d2876434874d9bbdbaf25daca5a7c48dacbfa)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/688178
|
|
am: 1cde1274be
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22937298
Change-Id: I4c87eb2ad3f8d51c4b66a6873aec3b0c43c102f4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
https://skia.googlesource.com/skia.git/+log/cee2c5ed5fae..234093826ebe
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-u-release-autoroll
Please CC nscobie@google.com,rmistry@google.com,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/main/autoroll/README.md
Test: Presubmit checks will test this change.
Exempt-From-Owner-Approval: The autoroll bot does not require owner approval.
Bug: b/186777432
Bug: b/280099119
Change-Id: Ib542b4e6a089192355415a0ef97d77c7d09620b7
|
|
The gainmap spec has changed how it represents vectors. Previously,
they were represented as "1,2,3". Now they are true XMP lists, with
three entries.
The gainmap spec has been changed to now require that the base image
have XMP and that that XMP specify the gainmap version (and that that
version be 1.0).
Update writing and parsing of vectors, and add more tests for this.
Update the writing to include the version. Do not yet require that
the version be written, because that will briefly break the feature
(old images will stop working).
Also, fix the interpretation of gamma. The definition given in
SkGainmapInfo is the inverse of what the XMP spec intended.
Bug: skia:14031
Change-Id: I34bef228fcd05823efbe76bbcf9f09a324c96fe3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/670657
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Christopher Cameron <ccameron@google.com>
(cherry picked from commit 6b8e2cd9a6bc442a9815a885461586ed29dcabf0)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/686036
Reviewed-by: John Reck <jreck@google.com>
|
|
Previously, a function's parameter list did not count against its
stack size limit.
Bug: 278113033
Test: SkQP
Change-Id: If49dce98f3155f3144a766c26b5a3a39401ce1b2
Merged-In: I84d250c27ce407259c52d12e2e011c0c0a7ae3b0
|
|
am: 3e819a2cc6
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22790538
Change-Id: Id3b9d5f18345ddc2552f13d8e5f3589aae92ea4c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
|
|
am: 1315177043
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/skia/+/22788051
Change-Id: I855f33a94564ba42ef237b01f2ec67c18b3637cb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|