AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
4108364efc
skia2/RELEASE_NOTES.txt

542 lines
21 KiB
Plaintext
Raw Normal View History

Introduce crowdsourced release notes file Bug: skia: Change-Id: I9d76b56ef88ec5caf305deed8bf839443427a260 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/229760 Commit-Queue: Heather Miller <hcm@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
2019-07-25 15:33:30 +00:00
Skia Graphics Release Notes
This file includes a list of high level updates for each milestone release.
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
* * *
add M86 branch release notes Bug: skia: Change-Id: I62331cab6cfadbaf9ab2f74e316cb7a2515249d0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/312305 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-08-20 19:59:08 +00:00
Milestone 87
------------
Revert "Remove ARC from Metal backend" This reverts commit 6541013b536ffb1e74c26462a5af54238244c62b. Reason for revert: TSAN issues with GrFence, and crash in GrMtlPipelineStateBuilder::CreatePipelineState. Original change's description: > Remove ARC from Metal backend > > Change-Id: I5ab28f6eda3b37d1b82c94c7cc6eaa2ce59157da > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/311113 > Reviewed-by: Adlai Holler <adlai@google.com> > Reviewed-by: John Stiles <johnstiles@google.com> > Commit-Queue: Jim Van Verth <jvanverth@google.com> TBR=jvanverth@google.com,bsalomon@google.com,adlai@google.com,johnstiles@google.com Change-Id: I031629b483fc46de8bd3751253e5391c2ce87853 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/312843 Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2020-08-24 17:16:46 +00:00
* <insert new release notes here>
add M86 branch release notes Bug: skia: Change-Id: I62331cab6cfadbaf9ab2f74e316cb7a2515249d0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/312305 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-08-20 19:59:08 +00:00
New version of SkImage::MakeFromYUVAPixmaps() Takes SkYUVAPixmaps. Still implemented in terms of SkYUVAIndex[4] internally. Replace all internal use cases except wacky_yuv_formats. Takes GrRecordingContext rather than GrContext. SkVideoDecoder updated to take GrRecordingContext. Bug: skia:10632 Change-Id: I6e9b9b6a4f11333bce6f87c1ebff0acb297f6540 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/316837 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Adlai Holler <adlai@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2020-09-15 15:24:28 +00:00
* Alternate SkImage::MakeFromYUVAPixmaps signature. Takes SkYUVAPixmaps, which specifies
planar configuration in a more structured manner. Currently limited to tri-planar
configurations without alpha but will be expanded. Older signature will become
deprecated when the new signature supports a broad range of planar configurations.
https://review.skia.org/316837
Reland "Use Microsoft's ComPtr to wrap D3D12 objects." This is a reland of 0ef049177f3c932e865177c71e2cb40e95a5b98b Original change's description: > Use Microsoft's ComPtr to wrap D3D12 objects. > > Change-Id: I4bd173428a2b65f0bc1994fb794ef9d4d68d5ba0 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/314957 > Reviewed-by: Brian Salomon <bsalomon@google.com> > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Jim Van Verth <jvanverth@google.com> Change-Id: Id0199db4061c67ed53da35e74dc31a004744be95 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/315655 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2020-09-08 19:51:14 +00:00
* Switch to using Microsoft::WRL::ComPtr for wrapping D3D12 objects.
https://review.skia.org/314957
Filter transformed atlas images correctly on GPU. The GPU backend was not considering the SkRSXform when optimizing the filter mode. This would cause rotated or scaled atlas images to look pixelated, regardless of the paint filter. Change-Id: I65f4bee6351e10349114869b823b2114bbcf70be Bug: skia:10662 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/313081 Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
2020-08-25 17:33:45 +00:00
* GPU backends now properly honor the SkFilterQuality when calling drawAtlas.
https://review.skia.org/313081
SkRuntimeEffect SkSL has a new signature for main() There is no more 'inout half4 color'. Effects return their output color. If an effect wants the input color, it must use the (already existing) approach of sampling a nullptr input shader. The change is guarded for Chromium (so we can update their runtime color filters in skia_renderer.cc). For the GPU backend, FPs can now override usesExplicitReturn to indicate that their emitCode will generate a return statement. If that's true, then writeProcessorFunction doesn't inject the automatic return of the output color, and emitFragProc will *always* wrap that FP in a helper function, even as a top-level FP. GrSkSLFP opts in to this behavior, so that the user-supplied return becomes the actual return in the FP's emitCode. Adapting the skvm code to this wasn't too bad: It looks fragile (what happens if there are multiple returns?), but that's not really possible today, without varying control flow. Bug: skia:10613 Change-Id: I205b81fd87dd32bab30b6d6d5fc78853485da036 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/310756 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: John Stiles <johnstiles@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-08-13 20:59:48 +00:00
* The signature of 'main' used with SkRuntimeEffect SkSL has changed. There is no longer an
'inout half4 color' parameter, effects must return their color instead.
Valid signatures are now 'half4 main()' or 'half4 main(float2 coord)'.
https://review.skia.org/310756
A new way to specify YUVA planar data from SkCodec to SkImage_Lazy Tunnels through SkImageGenerator as well. The new SkCodec interface doesn't assume three 8 bit planes. New SkYUVASpec more clearly defines chroma subsampling and siting of the planes. The intent is to use this for other YUVA APIs as well, in particular SkImage factories in the future. In this change we convert to the SkYUVASpec to SkYUVASizeInfo and SkYUVAIndex[4] representation. But the intent is to use the SkYUVASpec representation throughout the pipeline once legacy APIs are removed. orientation GM is replicated to test a variety of chroma subsampling configs. Bug: skia:10632 Change-Id: I3fad35752b87cac16c51b24824331f2ae7d458d3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/309658 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com>
2020-08-24 13:18:16 +00:00
* New YUVA planar interface in SkCodec. Chroma subsampling is specified in more structured way.
Doesn't assume 8bit planar values.
https://review.skia.org/309658
New helpers for SkYUVAInfo when used with pixmaps. SkYUVAPixmapInfo is a SkYUVAInfo with per-plane color types and row bytes. It describes a set of pixmaps that make up a planar image. Consolidates validity checks of the SkYUVAInfo with the color types and row bytes. It can provide SkImageInfos for each plane and also assist with configuring planes to share a common allocation. SkYUVAPixmaps is a collection of SkPixmaps that are valid for a SkYUVAInfo. It can either wrap existing SkPixmaps or allocate and own the memory. It consolidates validity checking of SkPixmaps with the SkYUVAInfo. Replaces sk_gpu_test::YUVAPixmaps. Minor tweaks to SkYUVAInfo naming, parameter order consistency, adds a hasAlpha() method. Bug: skia:10632 Change-Id: Ib0f48b8448fff22805fd0c04e07887d0b7338b76 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/312886 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com>
2020-08-27 15:00:04 +00:00
https://review.skia.org/312886
Reland "Add idea of DataType to SkYUVAPixmapInfo." This is a reland of ed6344458750c0389b7ccce588b96d0c25397b19 Original change's description: > Add idea of DataType to SkYUVAPixmapInfo. > > DataType describes the data type of YUVA channels > independent of how they are grouped into planes. > > Adds mapping functions between SkColorType/channel count > and DataType. > > SkYUVAPixmapInfo can be constructed from DataType and will > choose appropriate SkColorTypes for each plane. > > Valid SkYUVAPixmapInfos now have the same DataType for each > plane (could relax this in the future, esp for alpha plane). > > SkYUVAPixmapInfo::SupportedDataTypes specifies the supported > combinations of SkYUVAInfo::PlanarConfig and > kYUVAPixmapInfo::DataType supported by a GrContext (based on > supported texture formats). > > SkImageGenerator/SkCodec YUVA query API now takes a > SupportedDataTypes. > > Change-Id: I8791234638e6ba3396d1e7960b7bc210edc6dd57 > Bug: skia:10632 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/314276 > Reviewed-by: Leon Scroggins <scroggo@google.com> > Reviewed-by: Robert Phillips <robertphillips@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> Bug: skia:10632 Change-Id: I35b55b7477c11c822fdb3729a9f84acff1eb785d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/315284 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-09-01 19:01:15 +00:00
https://review.skia.org/314276
A new way to specify YUVA planar data from SkCodec to SkImage_Lazy Tunnels through SkImageGenerator as well. The new SkCodec interface doesn't assume three 8 bit planes. New SkYUVASpec more clearly defines chroma subsampling and siting of the planes. The intent is to use this for other YUVA APIs as well, in particular SkImage factories in the future. In this change we convert to the SkYUVASpec to SkYUVASizeInfo and SkYUVAIndex[4] representation. But the intent is to use the SkYUVASpec representation throughout the pipeline once legacy APIs are removed. orientation GM is replicated to test a variety of chroma subsampling configs. Bug: skia:10632 Change-Id: I3fad35752b87cac16c51b24824331f2ae7d458d3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/309658 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com>
2020-08-24 13:18:16 +00:00
Add image usage flags to GrVkImageInfo Bug: skia:9898 Change-Id: I46a66d4415af0bb013201ce756e8dbceb9e234cf Reviewed-on: https://skia-review.googlesource.com/c/skia/+/311437 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-08-21 15:26:12 +00:00
* Added VkImageUsageFlags to GrVkImageInfo struct.
add M86 branch release notes Bug: skia: Change-Id: I62331cab6cfadbaf9ab2f74e316cb7a2515249d0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/312305 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-08-20 19:59:08 +00:00
* * *
Implement Direct3D semaphores Change-Id: Iadfa14965bbe11cfea556ade5d46e264f0ace61a Bug: skia:9935 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298752 Commit-Queue: Jim Van Verth <jvanverth@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-06-25 17:10:29 +00:00
Milestone 86
------------
Remove unused GrFlushFlags. This also adds back default flush() calls which simply do a flush without any submit. Change-Id: Ia8c92bbdecd515d871abfa6364592f502e98656b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298818 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: John Stiles <johnstiles@google.com>
2020-06-30 17:42:46 +00:00
Remove 'in' variables from SkRuntimeEffect Runtime effects previously allowed two kinds of global input variables: 'in' variables could be bool, int, or float. 'uniform' could be float, vector, or matrix. Uniform variables worked like you'd expect, but 'in' variables were baked into the program statically. There was a large amount of machinery to make this work, and it meant that 'in' variables needed to have values before we could make decisions about program caching, and before we could catch some errors. It was also essentially syntactic sugar over the client just inserting the value into their SkSL as a string. Finally: No one was using the feature. To simplify the mental model, and make the API much more predictable, this CL removes 'in' variables entirely. We no longer need to "specialize" runtime effect programs, which means we can catch more errors up front (those not detected until optimization). All of the API that referred to "inputs" (the previous term that unified 'in' and 'uniform') now just refers to "uniforms". Bug: skia:10593 Change-Id: I971f620d868b259e652b3114f0b497c2620f4b0c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/309050 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: John Stiles <johnstiles@google.com>
2020-08-10 18:26:16 +00:00
* Remove support for 'in' variables from SkRuntimeEffect. API now exclusively refers to inputs
as 'uniforms'.
https://review.skia.org/309050
Add platform image encoder for using NDK APIs Bug: skia:10369 Add SkEncodeImageWithNDK, mirroring the CG and WIC versions, for encoding with the NDK APIs added to R. Rename SK_ENABLE_NDK_DECODING to SK_ENABLE_NDK_IMAGES and use it for both encoding and decoding. Move code for converting to/from NDK types into a common location. Update encode_platform.cpp to use NDK encoding APIs when available and to use both types of webp (lossy and lossless). Add tests specifically for the new implementation. Update NdkDecodeTest to use ToolUtils::equal_pixels for comparing pixels. Change-Id: Ic62f89af27372ccce90b8e028e01c388a135a68c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/308800 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-08-05 20:51:10 +00:00
* Add SkImageGeneratorNDK and SkEncodeImageWithNDK for using Android's NDK APIs to decode and
encode.
https://review.skia.org/308185
https://review.skia.org/308800
Reland "Add an SkImageGenerator that uses NDK APIs" This reverts commit 07438b0cdaa326d5e4b969e03bfb67e0c8066f13. Bug: skia:10369 Bug: skia:10371 This will allow Skia clients developing for Android 11+ to rely on Android's NDK APIs for decoding, which will allow them to decode without including their own decoding libraries (e.g. libjpeg-turbo). Using these APIs also provides support for static HEIF images. Run ImageGenSrc in kPlatform_Mode on Android to verify decoding visually. Add tests and a grayscale png. Update some test bots running Android R to specify ndk_api so they will run the new code. Change-Id: I4ca07d832dbd6a9d8cff0faea975fd70da00718f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/308185 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
2020-08-05 14:44:17 +00:00
remove (unused) DirectTo... methods Change-Id: Ie5eea2902c64fc44e4e6e53b1ee3b869d430d482 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/306331 Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
2020-07-28 21:24:21 +00:00
* SkImage:remove DecodeToRaster, DecodeToTexture
https://review.skia.org/306331
Reland "Add GrContext api to update compressed backend textures." This reverts commit ceebe424b1475ebdcf55f769e58b5e1a45b272c9. Reason for revert: relanding with fix Original change's description: > Revert "Add GrContext api to update compressed backend textures." > > This reverts commit 2c180304dc239a44770df7bf2649f3138f25922c. > > Reason for revert: attempted workaround did not fix techno spark so needs further investigation > > Original change's description: > > Add GrContext api to update compressed backend textures. > > > > Bug: chromium:1099255 > > Change-Id: I0c3f25ddb037e47e3b910fa89c3d8b3aa27b3114 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/302265 > > Reviewed-by: Brian Salomon <bsalomon@google.com> > > Reviewed-by: Robert Phillips <robertphillips@google.com> > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > Change-Id: Ib5433def02dc5dad97dcdbd4476ced6de2361e6a > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: chromium:1099255 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/302576 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:1099255 Change-Id: Ie238a56b7f12fea8b6e251a050e5e2f4b20d2ede Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304741 Reviewed-by: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-07-22 16:09:26 +00:00
* Add GrContext api to update compressed backend textures.
https://review.skia.org/302265
Rename GrMipMapped GrMipmapped Change-Id: Ia2cfbca8982b57399b6681cbb4501c2933ab4df7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304576 Auto-Submit: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-07-21 13:27:25 +00:00
* Rename GrMipMapped to GrMipmapped for consistency with new APIs.
Rename GrBackendTexture::fMipMapped -> fMipmapped Also mipMapped params to GrBackendTexture functions to mipmapped GrBackendTexture::hasMipMaps -> GrBackendTexture::hasMipmaps Misc test vars fMipMapped -> fMipmapped Change-Id: Ic0651d14fc106c21b0ab45529875b95ed8dc2dfd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304598 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-07-21 14:32:07 +00:00
Also rename GrBackendTexture::hasMipMaps() to GrBackendTexture::hasMipmaps()
Rename GrMipMapped GrMipmapped Change-Id: Ia2cfbca8982b57399b6681cbb4501c2933ab4df7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304576 Auto-Submit: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-07-21 13:27:25 +00:00
https://review.skia.org/304576
Rename GrBackendTexture::fMipMapped -> fMipmapped Also mipMapped params to GrBackendTexture functions to mipmapped GrBackendTexture::hasMipMaps -> GrBackendTexture::hasMipmaps Misc test vars fMipMapped -> fMipmapped Change-Id: Ic0651d14fc106c21b0ab45529875b95ed8dc2dfd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304598 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-07-21 14:32:07 +00:00
https://review.skia.org/304598
Rename GrMipMapped GrMipmapped Change-Id: Ia2cfbca8982b57399b6681cbb4501c2933ab4df7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/304576 Auto-Submit: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-07-21 13:27:25 +00:00
Add option for client to own semaphores after telling to skia to wait. Bug: chromium:1004772 Change-Id: Ifdad0b12925d13d3e896f6321c35312f9daa330c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/301216 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-07-08 15:44:25 +00:00
* Add option for clients to own semaphores after wait calls.
https://review.skia.org/301216
Remove unused GrFlushFlags. This also adds back default flush() calls which simply do a flush without any submit. Change-Id: Ia8c92bbdecd515d871abfa6364592f502e98656b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298818 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: John Stiles <johnstiles@google.com>
2020-06-30 17:42:46 +00:00
* Remove obsolete GrFlushFlags.
https://review.skia.org/298818
Add default flush calls to gpu backend. Change-Id: I7cd8ff975502128dbde86397218b7e5c115cdaa6 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299141 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-06-26 01:00:17 +00:00
* Adds default flush() calls to SkSurface, SkImage, and GrContext. These calls do
Remove unused GrFlushFlags. This also adds back default flush() calls which simply do a flush without any submit. Change-Id: Ia8c92bbdecd515d871abfa6364592f502e98656b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298818 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: John Stiles <johnstiles@google.com>
2020-06-30 17:42:46 +00:00
a basic flush without a submit. If you haven't updated Skia in a couple releases
Add default flush calls to gpu backend. Change-Id: I7cd8ff975502128dbde86397218b7e5c115cdaa6 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299141 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-06-26 01:00:17 +00:00
and still have flush() calls in your code that you expect to do a flush and
submit, you should update all those to the previously added flushAndSubmit() calls
instead.
https://review.skia.org/299141
Implement Direct3D semaphores Change-Id: Iadfa14965bbe11cfea556ade5d46e264f0ace61a Bug: skia:9935 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298752 Commit-Queue: Jim Van Verth <jvanverth@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-06-25 17:10:29 +00:00
* Enable BackendSemaphores for the Direct3D backend.
https://review.skia.org/298752
Reland "Add async rescale and read APIs to SkImage." This reverts commit 1caf3789f8c6e7e8d50f08cd56a5b725922aa044. Makes the image GMs detect an abandoned context just like the surface GMs. Bug: skia:10431 Change-Id: I56a3631a75e6b0383f96a73f461cfa314ee29afa Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299379 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-06-26 17:32:09 +00:00
* Added SkImage:asyncRescaleAndReadPixels and SkImage::asyncRescaleAndReadPixelsYUV420
https://review.skia.org/299281
Remove all internal uses of (and deprecate) SkSurface::getContext We are replacing GrContext with the GrDirectContext/GrRecordingContext pair. This starts making that change visible to clients (and weaning Skia off of GrContext). Change-Id: I00cc9bf208499984de855a1646229bd7557fc925 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/300706 Reviewed-by: Adlai Holler <adlai@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2020-07-06 19:50:15 +00:00
* Ganesh is moving towards replacing GrContext with the GrDirectContext/GrRecordingContext
pair. GrDirectContexts have _direct_ access to the GPU and are very similar to the old
GrContext. GrRecordingContexts are less powerful contexts that lack GPU access but provided
context-like utilities during DDL recording. SkSurfaces and SkCanvas will now only return
GrRecordingContexts. Clients requiring context features that need GPU access can then
check (via GrRecordingContext::asDirectContext) if the available recording context is actually
a direct context.
Remove unnecessary macros from SkString header. They have been replaced with equivalent constants (and given the requisite k prefix). Change-Id: I70907eec234e0861cc97aac1ca03086faca42f9a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/306160 Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Mike Reed <reed@google.com>
2020-07-28 13:23:54 +00:00
* Replace #defined values in SkString with equivalent constexprs.
http://review.skia.org/306160
Implement Direct3D semaphores Change-Id: Iadfa14965bbe11cfea556ade5d46e264f0ace61a Bug: skia:9935 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298752 Commit-Queue: Jim Van Verth <jvanverth@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-06-25 17:10:29 +00:00
* * *
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
Milestone 85
------------
Add GrContext::oomed() and implement for GL and VK. Surfaces to client whether GrContext has seen a GL_OUT_MEMORY, VK_ERROR_OUT_OF_HOST_MEMORY, or VK_ERROR_OUT_OF_DEVICE_MEMORY error. Bug: chromium:1093997 Change-Id: I8e9799a0f7d8a74df056629d7d1d07c0d0a0fe30 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298216 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-06-24 14:19:52 +00:00
* Added GrContext::oomed() which reports whether Skia has seen a GL_OUT_OF_MEMORY
error from Open GL [ES] or VK_ERROR_OUT_OF_*_MEMORY from Vulkan.
https://review.skia.org/298216
Add GrContext API to allow updating GrBackendSurfaceMutableState. This is currently only supported for the Vulkan backend Bug: skia:10254 Change-Id: I9274799098dc00dec5abcbcec95ce7cc23fec537 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293844 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-06-09 21:29:32 +00:00
Add SkSurface flush call that takes a GrBackendSurfaceMutableState. This new api will eventually replace the version that takes an SkSurface::BackendAccess. Change-Id: I48cd013725e14027f386b0b111223459944ac44a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/295567 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-06-11 15:51:06 +00:00
* Add option on SkSurface::flush to pass in a GrBackendSurfaceMutableState which
we will set the gpu backend surface to be at the end of the flush.
https://review.skia.org/295567
Add GrContext API to allow updating GrBackendSurfaceMutableState. This is currently only supported for the Vulkan backend Bug: skia:10254 Change-Id: I9274799098dc00dec5abcbcec95ce7cc23fec537 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293844 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-06-09 21:29:32 +00:00
* Add GrContext function to set mutable state on a backend surface. Currently this
is only used for setting vulkan VkImage layout and queue family.
https://review.skia.org/293844
Always call release procs on SkSurface GPU factories Change-Id: Id1ca19630aea88200dfe2444aca1f8e9f679a616 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293762 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-06-03 02:01:39 +00:00
* SkSurface factores that take GrBackendTexture or GrBackendRenderTarget now always
Add GrContext API to allow updating GrBackendSurfaceMutableState. This is currently only supported for the Vulkan backend Bug: skia:10254 Change-Id: I9274799098dc00dec5abcbcec95ce7cc23fec537 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293844 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-06-09 21:29:32 +00:00
call the release proc (if provided) on failure. SkSurface::replaceBackendTexture
Always call release procs on SkSurface GPU factories Change-Id: Id1ca19630aea88200dfe2444aca1f8e9f679a616 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293762 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-06-03 02:01:39 +00:00
also calls the release proc on failure.
https://review.skia.org/293762
Remove submit from async read pixels. User must submit. Bug: chromium:1087118 Change-Id: Iab152ac483787fbdcca448aee0c4b9d2b354a92b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/292840 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2020-06-02 15:47:26 +00:00
* SkSurface::asyncRescaleAndReadPixels and SkSurfaceasyncRescaleAndReadPixelsYUV420
now require explicit GrContext submit to guarantee finite time before callback
is invoked.
https://review.skia.org/292840
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
Add support for vk image sharing mode and handle queues xfers correctly. Change-Id: I9dbe6020d67cc452c9cbbdeace68f1d01275b419 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293559 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Jim Van Verth <jvanverth@google.com>
2020-06-02 15:10:41 +00:00
* Add VkSharingMode field to GrVkImageInfo.
https://review.skia.org/293559
Remove SkBitmapRegionDecoder and SkBRDAllocator Bug: skia:10154 Once ag/11371053 lands, Android will only be referencing the new version at android::skia::BitmapRegionDecoder. Remove the old code that remained for the transition. Change-Id: Ic532d705e8f2861f56dcaa0ac1c68dd2b0712035 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/291357 Commit-Queue: Leon Scroggins <scroggo@google.com> Auto-Submit: Leon Scroggins <scroggo@google.com> Reviewed-by: Derek Sollenberger <djsollen@google.com>
2020-05-21 15:14:31 +00:00
* Move SkBitmapRegionDecoder into client_utils/android.
[canvaskit] Make SkCanvas.drawColor/clear take SkColor4f CanvasKit converts 4 floats to an int, just to have it be converted back into 4 floats when it goes into the paint. Change-Id: I93cf1c596283b83cc9452fb205b7000ceed09bb9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/293538 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Reed <reed@google.com>
2020-06-03 15:19:49 +00:00
* SkCanvas.clear and SkCanvas.drawColor now accept SkColor4f in addition to SkColor.
Remove SkSurface::MakeFromBackendTextureAsRenderTarget from public API Still some more work to do to remove internal usage. Bug: skia:9832 Change-Id: Id0403d92debc26af2002630a4dfcf960c9343260 Docs-Preview: https://skia.org/?cl=292719 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/292719 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-05-28 21:44:05 +00:00
* Remove SkSurface::MakeFromBackendTextureAsRenderTarget.
This factory existed to work around issues with GL_TEXTURE_RECTANGLE that existed
in Chrome's command buffer. Those issues have since been resolved. Use
SkSurface::MakeFromBackendTexutre or SkSurface::MakeFromBackendRenderTarget instead.
https://review.skia.org/292719
Add submittedProc to GrFlushInfo. Bug: skia:10118 Change-Id: Iad848310d0f2fb22f19e9890209548fda103bd27 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/291078 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-05-26 15:26:45 +00:00
* Adds submittedProc callback to GrFlushInfo which will be called when the work
from the flush call is submitted to the GPU. This is specifically useful for knowing
when semahpores sent with the flush have been submitted and can be waiting on.
https://review.skia.org/291078
Reland "Implement submit API to GrContext." This reverts commit 9ee15d7b57075e2f8a533cfc9516aff0611a0ef3. Reason for revert: relanding with fixes Original change's description: > Revert "Implement submit API to GrContext." > > This reverts commit 40f288c72e4e836a2316cc058b5bb3aea92bde3f. > > Reason for revert: canvaskit breaking for some reason??? > > Original change's description: > > Implement submit API to GrContext. > > > > Change-Id: Ib813d42abb5f63e2ecdbf245d416658143853288 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/289033 > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > Reviewed-by: Brian Salomon <bsalomon@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com,penghuang@chromium.org,vasilyt@chromium.org > > Change-Id: Iee6c8342cccc601edf64ea011f1303e5d72559a9 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290917 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com,penghuang@chromium.org,vasilyt@chromium.org # Not skipping CQ checks because this is a reland. Change-Id: I5203676f88893cbbaba685301b8a713b40396b48 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290960 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-05-20 17:16:00 +00:00
* GrContext submit is now required to be called in order to send GPU work to the
actual GPU. The flush calls simply produces 3D API specific objects that are ready
to be submitted (e.g. command buffers). For the GL backend, the flush will still
send commands to the driver. However, clients should still assume the must call
submit which is where any glFlush that is need for sync objects will be called. There,
are flushAndSubmit() functions of GrContext, SkSurface, and SkImage that will act
like the previous flush() functions. This will flush the work and immediately call
submit.
https://review.skia.org/289033
Reland "Remove deprecated flush calls." This reverts commit 5e6d789ce47c8ce474b597073f7580602fcca467. Reason for revert: Relanding with fix landed in chrome Original change's description: > Revert "Remove deprecated flush calls." > > This reverts commit d8fd0bf5749c62302eb1d1c9f4eaab151d839341. > > Reason for revert: chrome roll > > Original change's description: > > Remove deprecated flush calls. > > > > Bug: skia:10118 > > Change-Id: I13758b5416784c296b8b5be9f3228230ac1be05f > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290540 > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > Reviewed-by: Brian Salomon <bsalomon@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com > > Change-Id: I49d35cdb258e632f645974c5ec62075d3392efe0 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: skia:10118 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290834 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com # Not skipping CQ checks because this is a reland. Bug: skia:10118 Change-Id: Ie24d7845a4ad75f95afe59037ea80a9f38082f13 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290918 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-05-19 22:06:26 +00:00
* Remove deprecated version of flush calls on GrContext and SkSurface.
https://review.skia.org/2290540
For vertices/patches with a shader and no texCoords, use positions This was another surprising (to me) inconsistency. Just because these geometric primitives allow for explicit local coords, doesn't mean we should require them (vs. all others that implicitly sample using local position). Change-Id: If3e7f6077bd15891b06cd2ffc969f1a649305d42 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290130 Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-05-15 17:04:48 +00:00
* SkCanvas::drawVertices and drawPatch now support mapping an SkShader without explicit
texture coordinates. If they're not supplied, the local positions (vertex position or
patch cubic positions) will be directly used to sample the SkShader.
https://review.skia.org/290130
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
* * *
Update release schedule, notes Bug: skia: Change-Id: I993eec22654edf93ea93e37282b0e08d19acd589 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281516 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-04-03 16:01:51 +00:00
Milestone 84
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
Add GrContext getter to SkSurface. Currently this is possible by getting the canvas off of surface and then getting the context off of the canvas. However, if in the future we wanted a surface that didn't have a canvas (maybe something that looked like a mutible image that only had a writePixels call), this would allow us to still get the GrContext. Bug: skia:10118 Change-Id: Ie8504bb708c1e4532edc8c7832ead86ae711e237 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/289479 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-05-13 18:11:40 +00:00
Reland "Reland "Add api on GrContext to update the data of GrBackendTextures."" This reverts commit ed219fe171df7bdd9ca7ab4edb25a529cad75682. Reason for revert: relanding with fix Original change's description: > Revert "Reland "Add api on GrContext to update the data of GrBackendTextures."" > > This reverts commit 83c6626946d03dce41a2aaeaf562a471bcaaa848. > > Reason for revert: technospark is failing to upload pixmaps to non base mip levels > > Original change's description: > > Reland "Add api on GrContext to update the data of GrBackendTextures." > > > > This reverts commit 93ca54e0ac5b90e91ef8b54a9d581d9b9e84eb26. > > > > Reason for revert: relanding with fix > > > > Original change's description: > > > Revert "Add api on GrContext to update the data of GrBackendTextures." > > > > > > This reverts commit ac09f7cd7a283292e81d57e94936dfd3cc8ca018. > > > > > > Reason for revert: breaking bots, may need to use swizzled color for > > > correctness test > > > > > > Original change's description: > > > > Add api on GrContext to update the data of GrBackendTextures. > > > > > > > > Change-Id: I680f12bf58fc7b66a6b2f3fa4c4723ae84d3f949 > > > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/288555 > > > > Reviewed-by: Brian Salomon <bsalomon@google.com> > > > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > > > > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > > > > > Change-Id: I47f41f536619ac13ca3ceeb216e7eaed9a9af255 > > > No-Presubmit: true > > > No-Tree-Checks: true > > > No-Try: true > > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/288630 > > > Reviewed-by: Greg Daniel <egdaniel@google.com> > > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > > > # Not skipping CQ checks because this is a reland. > > > > Change-Id: I82283b2437e523b80acead71c5f7c651180620db > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/288631 > > Reviewed-by: Greg Daniel <egdaniel@google.com> > > Reviewed-by: Brian Salomon <bsalomon@google.com> > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > Change-Id: I5d6614db8db59a69ded511726507a186596cfbd4 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/288907 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com # Not skipping CQ checks because this is a reland. Change-Id: I35fa1ae7dc47a697bbc358655f5c821942033234 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/288909 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-05-13 19:32:04 +00:00
* Add api on GrContext, updateBackendTexture that will upload new data to a
GrBackendTexture.
https://review.skia.org/288909
Add GrContext getter to SkSurface. Currently this is possible by getting the canvas off of surface and then getting the context off of the canvas. However, if in the future we wanted a surface that didn't have a canvas (maybe something that looked like a mutible image that only had a writePixels call), this would allow us to still get the GrContext. Bug: skia:10118 Change-Id: Ie8504bb708c1e4532edc8c7832ead86ae711e237 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/289479 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-05-13 18:11:40 +00:00
* Add GrContext getter to SkSurface.
https://review.skia.org/289479
A flush and submit apis to public headers. We add a new flushAndSubmit() to replace the current flush() call. In the future all other flush(GrFlushInfo) type calls will not do a submission of work to the GPU. Instead an explicit submit call will have to be made. flushAndSubmit will do a flush and submit. Also adds a no-op submit call. This allows us to stage the flush submit changes by updating all clients that use non simple flushes to add a submit call immediately after each flush. Then we can change the logic of where the submission happens from flush to submit without breaking folks. Bug: skia:10118 Change-Id: I4f02189a21912d52b888597c7734b4ca0baee792 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/289478 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-05-13 18:07:40 +00:00
* Deprecate GrContext and SkSurface flush() call and replace ith with flushAndSubmit().
This only effects the default flush call that takes no parameters.
https://review.skia.org/289478
Update release schedule, notes Bug: skia: Change-Id: I993eec22654edf93ea93e37282b0e08d19acd589 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281516 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-04-03 16:01:51 +00:00
Add finisehd proc to backend texture creation. The callback lets the caller know when the data uploads to the texture from the create call are finished. This is important since the caller cannot delete the backend texture till the gpu is finished on vulkan and d3d. This change also removes the hard sync in vulkan during creation. Change-Id: I660d142219474e22b1337d2b0c81cda66fe18a4b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/286517 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2020-05-06 15:40:03 +00:00
* GrContext::createBackendTexture functions that initialize the texture no longer
guarantee that all the data has been uploaded and the gpu is done with the texture.
Instead the client can assume the upload work has been submitted to the gpu and they
must wait for that work to finish before deleting the texture. This can be done via
their own synchronization or by passing in a finish proc into the create calls which
will be called when it is safe to delete the texture (at least in terms of work
done during the create).
https://review.skia.org/286517
Update release schedule, notes Bug: skia: Change-Id: I993eec22654edf93ea93e37282b0e08d19acd589 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281516 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-04-03 16:01:51 +00:00
remove unused maskfilter helpers: Combine, Compose Change-Id: I8c6bf2c3539905c9c7bc9c29454322dbf944fe96 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/287823 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org>
2020-05-05 18:40:42 +00:00
* Remove unused SkMaskFilter helpers: compbine, compose
Note: shadermaskfilter will likely be removed next (clipShader should serve)
Restore SkCanvas::kPreserveLCDText_SaveLayerFlag Will use this in Chrome to support LCDText in saveLayer/restore with opacity. This partly reverts https://skia-review.googlesource.com/c/skia/+/181841. Bug: 1076019 Change-Id: Id870fb1dcc95c9b319797e936725b4447a97d1d5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/285956 Commit-Queue: Florin Malita <fmalita@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by: Florin Malita <fmalita@chromium.org> Auto-Submit: Xianzhu Wang <wangxianzhu@chromium.org>
2020-05-01 16:34:57 +00:00
* Add back SkCanvas::kPreserveLCDText_SaveLayerFlag to indicate that saveLayer()
will preserve LCD-text. All text in the layer must be drawn on opaque background
to ensure correct rendering.
Reland "Move SkFrontBufferedStream into Android-only dir" This reverts commit b25f30348bfee972b509fd4a13a0de99e89caa6a. Bug: skia:10154 Original message: > Add client_utils for code that is specifically for a single client. > Move SkFrontBufferedStream into its android/ subdir. Rename the class > to android::skia::FrontBufferedStream. Temporarily leave in > SkFrontBufferedStream until Android updates to the new API. > > Add a new optional target for client_utils/android. It is built in dev > builds for testing, and when building for the Android framework. Deliberately do not include client_utils in Google3, since the whole point is to only include where necessary. Change-Id: I48938c56aabb98e1ed820240d43ffcd0fdce7956 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/285104 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Derek Sollenberger <djsollen@google.com> Reviewed-by: Mike Reed <reed@google.com>
2020-04-24 17:00:48 +00:00
* Add the new directory client_utils/ for code that is specific to a single client and
should be considered separate from Skia proper. Move SkFrontBufferedStream into the
subdir android/.
add SkColorSpace param to SkPixmap::erase() We don't have a way to erase with a non-sRGB color. Update the P3 gm to test this, removing the SkBitmap erase case where there's no option for even an SkColor4f, let alone non-sRGB. I'm not sure it's really important to have one when we've got this on pixmap. Updated release notes. Change-Id: Ie98270d3f83e041593b4c6b2da8e325b36d4ff18 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/285341 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
2020-04-24 12:44:50 +00:00
* SkBitmap and SkPixmap's erase() methods now treat their color parameters
consistently with the rest of Skia, with all SkColors and any untagged
SkColor4fs interpreted as sRGB, not as a color in the bitmap's color space.
SkPixmap::erase(SkColor4f) now takes an SkColorSpace, so you can pass
pixmap.colorSpace() if you want the old behavior.
SkMatrix44 is deprecated: use SkM44 instead Change-Id: I9a269b9c0c3cda29d06827d016db3a3f963a91fd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/283504 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com>
2020-04-14 19:44:21 +00:00
* SkCamera.h and SkMatrix44.h are DEPRECATED.
SkCamera.h is deprecated Change-Id: I57bd99f2bddb7ac182f7830245c9d342d2adda62 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/283430 Reviewed-by: Derek Sollenberger <djsollen@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Mike Reed <reed@google.com>
2020-04-14 15:35:40 +00:00
Use SkM44 if you want to have 3d transformations.
Take MorphologyImageFilter radii as SkScalar. The morphology image filters work on pixels, so took the radii as integers. However, these radii will be mapped through the CTM, so any rounding should happen as late as possible or the effect will be overly discretized. Bug: skia:10110 Change-Id: I62ab6c37d0b4612690addc48c9bc473099ac36b9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/282636 Commit-Queue: Ben Wagner <bungeman@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com>
2020-04-09 17:00:44 +00:00
* Changed Dilate and Erode image filters to take SkScalar for radius instead of int. While
the image filters themselves are defined in terms of discrete pixels, the radii provided by
the user are mapped through the CTM so taking ints forced over discretization. After mapping
through the CTM the radii are now rounded to pixels.
https://review.skia.org/281731
https://review.skia.org/282636
Update gpu flush semaphore contract. From my scanning of clients this should not change how any of our main users interact with semaphores. The biggest change here is that we don't give up on submitted all the semaphores if a creation fails. We just submit the semaphores we do have. It also makes it explicit that the client is responsible for deleting any initialized semaphore regardless if we were able to submit the semaphores or not or if the semaphore was wrapped or created by Skia. The motivation for this change is to more align the current API with how things will work when we separate flushing and submit into different calls. Bug: skia:10118 Change-Id: I3e8b5d3a9852ddb2b5dc972fee21bf46ded7a36f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/282265 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-04-08 16:52:51 +00:00
* Updated the contract of GrContext and SkSurface flush calls in regards to semaphores. Made it
clear that the caller is responsible for deleting any initialized semaphores after the flush
call regardless if we were able to submit them or not. Also, allows skia to only submit a
subset of the requested semaphores if we failed to create some.
https://review.skia.org/282265
Remove secret wireframe mode from drawVertices Now, drawing vertices without colors or a shader simply fills with the paint color, like anyone would expect. Bug: skia:9984 Change-Id: I9f7cf72f89948756b9c2c6e055a11c6b64fc22ff Reviewed-on: https://skia-review.googlesource.com/c/skia/+/282043 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-04-07 17:48:15 +00:00
* SkCanvas::drawVertices will now always fill the triangles specified by the vertices. Previously,
vertices with no colors and no (texture coordinates or shader) would be drawn in wireframe.
https://review.skia.org/282043
Update release schedule, notes Bug: skia: Change-Id: I993eec22654edf93ea93e37282b0e08d19acd589 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/281516 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-04-03 16:01:51 +00:00
* * *
fix up release notes file for milestone 83 Bug: skia: Change-Id: Ia83f2b69948eeb747ecc171fb8677c8e22cc8ba7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/277176 Reviewed-by: Heather Miller <hcm@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-03-16 14:50:49 +00:00
Milestone 83
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
Fix up release notes file Bug: skia: Change-Id: I124abbb54f13e70e5d9c6a71f43cea5adf0ce49a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/250038 Auto-Submit: Heather Miller <hcm@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-22 19:45:45 +00:00
remove localmatrix parameter from Blend and Lerp compose shaders Bug: skia:10080 Change-Id: I936d6d696c86c50d5b51dc84894127c38ad753d4 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/279048 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org>
2020-03-25 17:13:20 +00:00
* Remove localmatrix option from SkShaders::[Blend, Lerp]
Flesh out backend textures and rendertargets to support Direct3D. Add GrD3DTextureInfo and GrD3DBackendSurfaceInfo, and uses those to initialize GrBackendTexture and GrBackendRendertarget. Bug: skia:9935 Change-Id: I4571c1b3aa8115250ff748deb8cf4a95f80f1237 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/278036 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2020-03-20 15:23:39 +00:00
* Fill out Direct3D parameters for backend textures and backend rendertargets.
Add SkBudgeted parameter to SkImage::makeTextureImage(). Also strengthens/adds some guarantees about this function: * Always returns the same image if the original is texture-backed and compatible with GrMipMapped (WRT HW MIP support) * If a new texture backed image is returned it is always an uncached texture that is not shared with another image or owned by an image generator. Adds a GrImageTexGenPolicy that allows control through image/bitmap GrTextureProducers of whether a new texture must be made and whether that texture should be budgeted or not. Increases unit test coverage of this API. Bug: skia:8669 Change-Id: Ifc0681856114a08fc8cfc57ca83d22efb1c1f166 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274938 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-03-18 14:06:13 +00:00
* SkImage::makeTextureImage() takes an optional SkBudgeted param
Remove drawBitmap et al. from SkDevice SkCanvas keeps its non-exotic drawBitmap calls for now but they immediately wrap the bitmap in SkImage::MakeFromBitmap. This will add a copy for mutable bitmaps, but this can be worked around by using images directly (preferred), or marking the bitmap as immutable. In practice, mutable bitmaps do not seem to be widely used so this is deemed acceptable for now. Several GMs are updated to mark the bitmaps as immutable in order to avoid pixel churn in how CPU vs. GPU mipmaps are generated. As we move towards explicit mipmap generation, this distinction will become less critical, so I'm just avoiding it for now. See skbug.com/9337 for mipmap proposal. The image diffs encountered in this CL directly related to generating mipmaps for bitmaps already cached on the GPU vs. not on the GPU yet. It was decided to copy if mutable vs. never copying the bitmap because it preserves the image guarantees the devices and canvas' rely on, and imposing the need to inspect an image for mutability is undesired. Bug: skia:10037, skia:9337 Change-Id: If152e6cae1e155ccf6bd2cd755895f3c7a0135a5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276004 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-03-12 14:52:15 +00:00
Revert "Revert "Revert "Revert "Don't build GL on Metal, Vulkan, Dawn, Direct3D bots"""" Updated to use sentinel GL context even when GL backend is not built. This reverts commit 1171d314efc7e131b039c94126d26575cfb9e70c. Change-Id: Ia94bbe4865ddd4e898446c13886877c539f0eb0b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/277976 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-03-19 19:54:28 +00:00
* Made non-GL builds of GPU backend more robust.
https://review.skia.org/277456
* MoltenVK support removed. Use Metal backend instead.
https://review.skia.org/277612
fix up release notes file for milestone 83 Bug: skia: Change-Id: Ia83f2b69948eeb747ecc171fb8677c8e22cc8ba7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/277176 Reviewed-by: Heather Miller <hcm@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-03-16 14:50:49 +00:00
* * *
Milestone 82
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
fix up release notes file for milestone 83 Bug: skia: Change-Id: Ia83f2b69948eeb747ecc171fb8677c8e22cc8ba7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/277176 Reviewed-by: Heather Miller <hcm@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-03-16 14:50:49 +00:00
Remove drawBitmap et al. from SkDevice SkCanvas keeps its non-exotic drawBitmap calls for now but they immediately wrap the bitmap in SkImage::MakeFromBitmap. This will add a copy for mutable bitmaps, but this can be worked around by using images directly (preferred), or marking the bitmap as immutable. In practice, mutable bitmaps do not seem to be widely used so this is deemed acceptable for now. Several GMs are updated to mark the bitmaps as immutable in order to avoid pixel churn in how CPU vs. GPU mipmaps are generated. As we move towards explicit mipmap generation, this distinction will become less critical, so I'm just avoiding it for now. See skbug.com/9337 for mipmap proposal. The image diffs encountered in this CL directly related to generating mipmaps for bitmaps already cached on the GPU vs. not on the GPU yet. It was decided to copy if mutable vs. never copying the bitmap because it preserves the image guarantees the devices and canvas' rely on, and imposing the need to inspect an image for mutability is undesired. Bug: skia:10037, skia:9337 Change-Id: If152e6cae1e155ccf6bd2cd755895f3c7a0135a5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276004 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-03-12 14:52:15 +00:00
* Removed drawBitmap and related functions from SkDevice; all public drawBitmap functions on
SkCanvas automatically wrap the bitmap in an SkImage and call the equivalent drawImage function.
Drawing mutable SkBitmaps will now incur a mandatory copy. Switch to using SkImage directly or
mark the bitmap as immutable before drawing.
SkVertices: Remove volatile/nonvolatile bit No one was using this, and it added significant complexity to GrDrawVerticesOp. Bug: skia:9984 Change-Id: I23f38b3f5d853a8e531f13b6931cd57b5985a2c8 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276407 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-03-11 16:26:56 +00:00
* Removed "volatile" flag from SkVertices. All SkVertices objects are assumed to be
volatile (the previous default behavior).
Remove drawBitmap et al. from SkDevice SkCanvas keeps its non-exotic drawBitmap calls for now but they immediately wrap the bitmap in SkImage::MakeFromBitmap. This will add a copy for mutable bitmaps, but this can be worked around by using images directly (preferred), or marking the bitmap as immutable. In practice, mutable bitmaps do not seem to be widely used so this is deemed acceptable for now. Several GMs are updated to mark the bitmaps as immutable in order to avoid pixel churn in how CPU vs. GPU mipmaps are generated. As we move towards explicit mipmap generation, this distinction will become less critical, so I'm just avoiding it for now. See skbug.com/9337 for mipmap proposal. The image diffs encountered in this CL directly related to generating mipmaps for bitmaps already cached on the GPU vs. not on the GPU yet. It was decided to copy if mutable vs. never copying the bitmap because it preserves the image guarantees the devices and canvas' rely on, and imposing the need to inspect an image for mutability is undesired. Bug: skia:10037, skia:9337 Change-Id: If152e6cae1e155ccf6bd2cd755895f3c7a0135a5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276004 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-03-12 14:52:15 +00:00
* Removed exotic legacy bitmap functions from SkCanvas (drawBitmapLattic, drawBitmapNine); the
exotic SkImage functions still exist.
Reland "Split building encoding from decoding" This reverts commit 9d4b788807fcca010399c0987aed41b3af5d938f. Copyright issue has been fixed in https://skia-review.googlesource.com/c/skia/+/275998. Original description: Bug: skia:9756 In CanvasKit, a large part of the binary is for encoding. Clients would be happier with a smaller binary and no webp/jpeg encoding. Make this an option by splitting up the GN arguments. Split SK_HAS_WEBP_LIBRARY into SK_CODEC_DECODES_WEBP (to match the existing SK_CODEC_DECODES_RAW) and SK_ENCODE_WEBP. Same for JPEG and PNG. Update CanvasKit compile script to disable webp and jpeg encoding. Update debugger compile script to disable all encoding. Change IsPng signature to match other SkCodecs. TBR=djsollen@google.com Change-Id: Ic847bae0154e0a2922100b3f2ee14a077ee5635a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276007 Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
2020-03-09 18:23:30 +00:00
* Make it possible to selectively turn on/off individual encoders/decoders,
using skia_use_(libpng/libjpeg_turbo/libwebp)(decode/encode).
Reland "Move GrGpuResource GrSurface and GrTexture into src." This reverts commit f6ed96d1c23b79130ca7344c984b07ef9d94fb7b. Reason for revert: google3 change landed Original change's description: > Revert "Move GrGpuResource GrSurface and GrTexture into src." > > This reverts commit e5a06ce678aad7640411f99f70f220f82ad49908. > > Reason for revert: Need to make change in google3 first > > Original change's description: > > Move GrGpuResource GrSurface and GrTexture into src. > > > > Must land https://chromium-review.googlesource.com/c/chromium/src/+/2087980 > > before this can land. > > > > Bug: skia:7966 > > Change-Id: I60bbb1765bfbb2c96b2bc0c9826b6b9d57eb2a03 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275077 > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > Reviewed-by: Robert Phillips <robertphillips@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > Change-Id: Id39e0a351e49a87209de88a6ad9fadb0219db72c > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: skia:7966 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275216 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com Change-Id: I746ce739cb084cefc46f9dab24ef773e7c3cc621 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia:7966 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275436 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-03-05 19:14:18 +00:00
* Removed GrGpuResource, GrSurface, and GrTexture from public api. These were not
Remove SkImage getTexture call. Change-Id: Ib9f3d8c58ee5736bfee03eaedff20f24b51d5ff9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275625 Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-03-06 18:07:10 +00:00
meant to be public, and we now can move them into src. Also removed getTexture
function from SkImage.h
Reland "Move GrGpuResource GrSurface and GrTexture into src." This reverts commit f6ed96d1c23b79130ca7344c984b07ef9d94fb7b. Reason for revert: google3 change landed Original change's description: > Revert "Move GrGpuResource GrSurface and GrTexture into src." > > This reverts commit e5a06ce678aad7640411f99f70f220f82ad49908. > > Reason for revert: Need to make change in google3 first > > Original change's description: > > Move GrGpuResource GrSurface and GrTexture into src. > > > > Must land https://chromium-review.googlesource.com/c/chromium/src/+/2087980 > > before this can land. > > > > Bug: skia:7966 > > Change-Id: I60bbb1765bfbb2c96b2bc0c9826b6b9d57eb2a03 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275077 > > Commit-Queue: Greg Daniel <egdaniel@google.com> > > Reviewed-by: Robert Phillips <robertphillips@google.com> > > TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com > > Change-Id: Id39e0a351e49a87209de88a6ad9fadb0219db72c > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: skia:7966 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275216 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Commit-Queue: Greg Daniel <egdaniel@google.com> TBR=egdaniel@google.com,bsalomon@google.com,robertphillips@google.com Change-Id: I746ce739cb084cefc46f9dab24ef773e7c3cc621 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia:7966 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275436 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2020-03-05 19:14:18 +00:00
remove bones from SkVertices Bug: skia:9984 Change-Id: I7c4f33ee82087eb1b0a60243183f0c68e310f10d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274547 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
2020-03-02 19:57:09 +00:00
* Removed Bones from SkVertices
Add GrContextOption for GL error checking, shader, and program success. Small behavior change: will not check for shader success and program linking success over the command buffer in a debug build. Also fail gracefully if stencil renderbuffer allocation fails. Bug: skia:9938 Bug: chromium:1040186 Change-Id: I623f09d306261d28070078268f6242f92d65fd5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/273276 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-02-25 17:13:29 +00:00
* Added a field to GrContextOptions that controls whether GL errors are checked after
GL calls that allocate textures, etc. It also controls checking for shader compile
success, and program linking success.
Add a program iterator to SkDeferredDisplayList Bug: skia:9455 Change-Id: I201218174b5d6efb72f7c06766a49f6711325bd5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272530 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-02-24 13:38:34 +00:00
* Made SkDeferredDisplayList.h officially part of the public API (i.e., moved it to
include/core). Also added a ProgramIterator to SkDeferredDisplayList which allows
clients to pre-compile some of the shaders the DDL requires.
Add createBackendFormat and createFBO0 helper methods to SkSurfaceCharacterization These make it easier for clients to create new surface characterizations that differ only a little from an existing surface characterization. Change-Id: Iebd0b32ae941d3f91427927108d092cb5864b09f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/270444 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-02-13 18:24:47 +00:00
* Added two new helper methods to SkSurfaceCharacterization: createBackendFormat and
createFBO0. These make it easier for clients to create new surface characterizations that
differ only a little from an existing surface characterization.
Remove SkTMin and SkTMax Use std::min and std::max everywhere. SkTPin still exists. We can't use std::clamp yet, and even when we can, it has undefined behavior with NaN. SkTPin is written to ensure that we return a value in the [lo, hi] range. Change-Id: I506852a36e024ae405358d5078a872e2c77fa71e Docs-Preview: https://skia.org/?cl=269357 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/269357 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2020-02-07 15:36:46 +00:00
* Removed SkTMax and SkTMin.
Replace usage of SkTClamp with SkTPin I prefer the SkTClamp spelling, but there were 4 uses vs. ~200. And eventually we'll replace all of these with std::clamp anyway. Change-Id: I418fce5080d089745c3f9eb42c36e3fc7962c895 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/269155 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-02-06 19:51:46 +00:00
* Removed SkTClamp and SkClampMax.
Remove SkScalarClampMax and SkScalarPin Change-Id: Ic96a0ea2cd1bfd59ee3f236543e1d6dd102544ec Reviewed-on: https://skia-review.googlesource.com/c/skia/+/269142 Commit-Queue: Mike Klein <mtklein@google.com> Auto-Submit: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-02-06 17:52:25 +00:00
* Removed SkScalarClampMax and SkScalarPin.
Remove SkMin32/SkMax32 Use std::max and std::min instead Change-Id: I7fd2626ea9ea8ea09c709ff962523ca3de2f8a16 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/269136 Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-02-06 16:25:43 +00:00
* Removed SkMax32 and SkMin32.
Remove SkMaxScalar and SkMinScalar Use std::max and std::min instead Change-Id: Icf3796609e5cb511687fb50bd31229ae4b6b9b39 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/268841 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-02-05 18:34:09 +00:00
* Removed SkMaxScalar and SkMinScalar.
SkColorSetA now warns if the result is unused. Change-Id: Ie0d2ec6e1ca1ddeb83338df232570443502e6cdb Reviewed-on: https://skia-review.googlesource.com/c/skia/+/268697 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-02-04 22:27:22 +00:00
* SkColorSetA now warns if the result is unused.
Allow decoding without color conversion Bug: b/135133301 Especially for the Android NDK, which may not be able to report the default profile, we should be able to decode the image without doing any color correction. That way if the client wants to figure out the color profile on their own (e.g. by parsing the data manually), they can retrieve the un-corrected pixels and treat them appropriately. Note that internally, we still require a destination profile for F16, but we will use the source profile if there is one, or SRGB for source and destination otherwise. Regardless, this will result in no color correction. Change-Id: Id1157bb99d3119a114b25f64b38620480a027f34 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/268157 Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
2020-02-03 17:39:54 +00:00
* An SkImageInfo with a null SkColorSpace passed to SkCodec::getPixels() and
related calls is treated as a request to do no color correction at decode
time.
Remove CGFontRef parameter from SkMakeTypefaceFromCTFont Since the Blink side CTFontRefs are no longer created from CGFontRefs after [1], we do not need to pass CGFontRefs to Skia any longer for keeping them alive. Clean up related plumbing which previously passed through the CGFontRef to the SkTypefaceMac constructor. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1964073 Bug: chromium:1033478 Change-Id: Ic85cae6289fa9b4886201c92defb54fcc5622720 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/266816 Reviewed-by: Dominik Röttsches <drott@chromium.org> Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2020-01-31 16:13:37 +00:00
Re-land: Support adding attributes to PDF document structure nodes. Originally landed: https://skia-review.googlesource.com/c/skia/+/268878 Reverted: https://skia-review.googlesource.com/c/skia/+/271858 The issue was with compilation when PDF support is disabled. See the diff between patchsets 1 and 2. This is an important part of writing a tagged PDF. Many of the nodes in the document structure tree need additional attributes, just like in HTML. This change aims to add support for a few useful attributes, not to be comprehensive. Bug: chromium:1039816 Change-Id: I15f8b6c41d4fdaa4b6e21775ab6d26ec57eb0f5d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/271916 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by: Mike Reed <reed@google.com>
2020-02-19 18:59:29 +00:00
* Add new APIs to add attributes to document structure node when
creating a tagged PDF.
Remove CGFontRef parameter from SkMakeTypefaceFromCTFont Since the Blink side CTFontRefs are no longer created from CGFontRefs after [1], we do not need to pass CGFontRefs to Skia any longer for keeping them alive. Clean up related plumbing which previously passed through the CGFontRef to the SkTypefaceMac constructor. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1964073 Bug: chromium:1033478 Change-Id: Ic85cae6289fa9b4886201c92defb54fcc5622720 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/266816 Reviewed-by: Dominik Röttsches <drott@chromium.org> Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2020-01-31 16:13:37 +00:00
* Remove CGFontRef parameter from SkCreateTypefaceFromCTFont.
Use CTFontManagerCreateFontDescriptorFromData instead of
CGFontCreateWithDataProvider to create CTFonts to avoid memory use issues.
Allow decoding without color conversion - part 2 Bug: b/135133301 Follow-on to 196f319b. - Add SkCodec::getICCProfile to match the SkAndroidCodec version. - Update comments on getPixels() regarding how the SkColorSpace on the SkImageInfo is treated. - Add two new images that have ICC profiles that do not map to an SkColorSpace. Add a test to verify that they have the un-transformed color we expect. - Stop uploading ColorCodecSrc images decoded to a null SkColorSpace to Gold. Though they may be correct, they do not match other images they're compared against. The new test above verifies that we do not do color conversion with a null SkColorSpace. Change-Id: I08635e4262f16500fab32ef97511d305c2c06483 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/269236 Reviewed-by: Derek Sollenberger <djsollen@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
2020-02-06 20:47:58 +00:00
* Added SkCodec:: and SkAndroidCodec::getICCProfile for reporting the native
ICC profile of an encoded image, even if it doesn't map to an SkColorSpace.
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
Update SkSurface::ReplaceBackendTexture to take ContentChangeMode In SkSurface::ReplaceBackendTexture, it used to always retain a copy of the the current content. The ContentChangeMode is added as a parameter to allow the callers to decide whether retain or discard the current content. Bug: 1014246 Change-Id: I8736e7795b4755bbde94009ffe4d943c75112b98 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/271016 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Khushal Sagar <khushalsagar@chromium.org>
2020-02-14 23:10:40 +00:00
* SkSurface::ReplaceBackendTexture takes ContentChangeMode as a parameter,
which allow callers to specify whether retain a copy of the current content.
Remove drawBitmap et al. from SkDevice SkCanvas keeps its non-exotic drawBitmap calls for now but they immediately wrap the bitmap in SkImage::MakeFromBitmap. This will add a copy for mutable bitmaps, but this can be worked around by using images directly (preferred), or marking the bitmap as immutable. In practice, mutable bitmaps do not seem to be widely used so this is deemed acceptable for now. Several GMs are updated to mark the bitmaps as immutable in order to avoid pixel churn in how CPU vs. GPU mipmaps are generated. As we move towards explicit mipmap generation, this distinction will become less critical, so I'm just avoiding it for now. See skbug.com/9337 for mipmap proposal. The image diffs encountered in this CL directly related to generating mipmaps for bitmaps already cached on the GPU vs. not on the GPU yet. It was decided to copy if mutable vs. never copying the bitmap because it preserves the image guarantees the devices and canvas' rely on, and imposing the need to inspect an image for mutability is undesired. Bug: skia:10037, skia:9337 Change-Id: If152e6cae1e155ccf6bd2cd755895f3c7a0135a5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/276004 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2020-03-12 14:52:15 +00:00
* Enforce the existing documentation in SkCanvas::saveLayer that it ignores
any mask filter on the restore SkPaint. The 'coverage' of a layer is
Enforce saveLayer ignoring restore paint's mask filter Documentation specifies that only the alpha, color filter, image filter and blend mode are used when restoring the saved layer. The coverage that would be sent to the mask filter is ill-defined, whereas the explicit boundary of the separate clip image gets around this. This also removes the issue of the order in which the mask filter and image filter would be applied. Some GMs exercised the save layer with mask filter, but those have been updated to either expect the MF to be ignored, or modified to no longer use saveLayer+MF. While Android and Flutter expose a saveLayer with general Paint API, their documentation matches SkCanvas so hopefully no one is actually relying on the mask filter in those cases. It does not appear as if Chrome ever tries to use a saveLayer+MF. Bug: skia:9561 Change-Id: I67a350fbf565683ea60ea88f83a91270e7dcd187 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274508 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com>
2020-03-03 15:39:41 +00:00
ill-defined, and masking should be handled by pre-clipping or using the
auxiliary clip mask image of the SaveLayerRec.
Update SkSurface::ReplaceBackendTexture to take ContentChangeMode In SkSurface::ReplaceBackendTexture, it used to always retain a copy of the the current content. The ContentChangeMode is added as a parameter to allow the callers to decide whether retain or discard the current content. Bug: 1014246 Change-Id: I8736e7795b4755bbde94009ffe4d943c75112b98 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/271016 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Khushal Sagar <khushalsagar@chromium.org>
2020-02-14 23:10:40 +00:00
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
* * *
Milestone 81
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
Add functions for GL_NV_fence to GrGLInterface. Also add macro for ALL_COMPLETED We plan to use this with ANGLE ES2 contexts. Change-Id: I6c967fa9b9aefc41c31cd806aece8533f846f940 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/265756 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-01-22 14:59:56 +00:00
* Added support for GL_NV_fence extension.
Update release notes files Docs-Preview: https://skia.org/?cl=258417 Bug: skia: Change-Id: I4716231ad260f2f7f29776f7cc455c6837ec6a43 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/258417 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-12-05 20:27:42 +00:00
Make SkImageInfo::validRowBytes consider alignment SkSurface will already reject a rowBytes that does not align on a pixel boundary. Push that check into SkImageInfo. This will make SkBitmap, SkPixmap, SkMallocPixelRef, and SkImage_Raster, which already call validRowBytes, make the same check. If an SkSurface cannot use a non-pixel-aligned rowBytes, then an SkCanvas wrapping an SkBitmap should not either. Update MallocPixelRefTest to use a rowBytes that is still valid. Change-Id: I848d94dbeab8b58b92877104dd67ea23a9d19ca8 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/265599 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Mike Reed <reed@google.com> Reviewed-by: Herb Derby <herb@google.com>
2020-01-22 15:44:00 +00:00
* Make SkImageInfo::validRowBytes require rowBytes to be pixel aligned. This
makes SkBitmap match the behavior of raster SkSurfaces in rejecting
non-aligned rowBytes.
Add SkImage::MakeRasterFromCompressed and make MakeFromCompressed fall back to decompression for unsupported compression types (take 2) This is intended to make using compressed textures easier since developers can just provided compressed data and it will be uploaded to the GPU in some way, shape or form. TBR=reed@google.com Change-Id: I8c672ccc7db5cd098f629c3469ae7cbdc7436392 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/266939 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2020-01-27 21:11:57 +00:00
* Added an SkImage::MakeRasterFromCompressed entry point. Also updated
SkImage::MakeFromCompressed to decompress the compressed image data if
the GPU doesn't support the specified compression type (i.e., macOS Metal
doesn't support BC1_RGB8_UNORM so such compressed images will always be
decompressed on that platform).
Add SkImage::CompressionType::kBC1_RGBA8_UNORM support This is mainly so we can test the compression code on macOS. Bug: skia:9680 Change-Id: Ie0a2eacfe9100ee4ce4cc94c878d3032d6985832 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/264480 Reviewed-by: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2020-01-15 17:56:52 +00:00
* Added support for BC1 RGBA compressed textures
Add CachingHint to makeRasterImage, default to kDisallow When a client calls makeRasterImage, they're likely to hold on to that image, and throw away the source (GPU or Lazy). In that situation, it makes no sense to leave the buffer in the resource cache. Change-Id: Ic0be8ceb643ed9196526469516be0d200bea42fd Bug: b/144232910 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/263347 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2020-01-09 15:21:44 +00:00
* Added CachingHint to SkImage::makeRasterImage
[canvaskit] Create an SkImage from a frame of an SkAnimatedImage Flutter on the web wants to be able to extract arbitrary frames from an animated image and pass those into functions like: drawAtlas, drawImage, drawImageRect This should allow that to happen w/o having to add lots of variants like drawAnimatedImage. If this sticks, is drawAnimatedImage still useful? (maybe it saves a copy?) Change-Id: I99d7045c5dea61d0a1bd6d335c88e7517f2c4fc2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/263020 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com> Reviewed-by: Mike Reed <reed@google.com>
2020-01-08 18:29:31 +00:00
* Added SkAnimatedImage::getCurrentFrame()
Add method to delay drawable acquisition from MTKView Change-Id: Iad9fadd113d0c97e8ece51df19c7824252d7eb9c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/263197 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2020-01-08 17:15:44 +00:00
* Add support to create an SkSurface from an MTKView, with delayed acquisition of
the MTLDrawable.
Entry point: SkSurface::MakeFromMTKView
Remove SkIRect::EmptyIRect Replace with SkIRect::MakeEmpty. Docs-Preview: https://skia.org/?cl=262382 Change-Id: I001cc76f4af223490be3734bb461ab4fca36d710 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/262382 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2020-01-06 20:20:18 +00:00
* Removed SkIRect::EmptyIRect(). Use SkIRect::MakeEmpty() instead.
https://review.skia.org/262382/
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
* Moved SkRuntimeEffect to public API. This is the new (experimental) interface to custom SkSL
shaders and color filters.
Move SkRuntimeEffect.h to include/effects Change-Id: I0b11d4210c6e663cfb4854fc33e1396fd79fe9a4 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/261780 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2020-01-02 16:55:24 +00:00
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
* Added BC1 compressed format support. Metal and Vulkan seem to only support the BC
formats on desktop machines.
Add BC1_RGB8_UNORM support This is working in the GL and Vulkan back ends. MacOS only supports the RGBA8 variants. For mobile devices, probably only nVidia GPUs will support this. Bug: skia:9680 Change-Id: I9d886b72232a031603e93e46059a97a8aa288b3c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/261093 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2019-12-20 16:32:27 +00:00
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
* Added compressed format support for backend texture creation API.
This adds the following new entry points:
GrContext::compressedBackendFormat
GrContext::createCompressedBackendTexture
The latter method comes in variants that allow color-initialized and
compressed texture data initialized.
Add interface(s) for compressed backend texture support This is just the API level interface(s) and the backend virtual calls needed to implement them. Bug: skia:9680 Change-Id: I6c655751e23ce18c6a184c925e8d472902e8a478 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/259976 Reviewed-by: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2019-12-17 19:44:37 +00:00
Support clipRegion on GPU backend Bug: skia:9580 Change-Id: I663549dafc4239248e265bee8d6927bf5b259303 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/259804 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2019-12-13 15:48:33 +00:00
* Added SkMatrix::MakeTrans(SkIVector)
https://review.skia.org/259804
Update release notes files Docs-Preview: https://skia.org/?cl=258417 Bug: skia: Change-Id: I4716231ad260f2f7f29776f7cc455c6837ec6a43 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/258417 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-12-05 20:27:42 +00:00
new section and formatting Bug: skia: Change-Id: I759f2eb1ee26245ba3d3c971e030793b9c52aa5f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/267820 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2020-01-30 18:57:13 +00:00
* * *
Update release notes files Docs-Preview: https://skia.org/?cl=258417 Bug: skia: Change-Id: I4716231ad260f2f7f29776f7cc455c6837ec6a43 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/258417 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-12-05 20:27:42 +00:00
Milestone 80
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
Update release notes files Docs-Preview: https://skia.org/?cl=258417 Bug: skia: Change-Id: I4716231ad260f2f7f29776f7cc455c6837ec6a43 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/258417 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-12-05 20:27:42 +00:00
Always destroy resources instead of simply "abandoning" them in Vulkan backend. This also now requires that when using a vulkan GrContext, the backing VkDevice, Queue, and Instance must be alive when the GrContext is first destroyed or abandoned. Additionally any GrBackendTextures created from the GrContext must be deleted before destroying or abandoning the GrContext. Bug: skia:9603 Change-Id: Ibf194d2ffdcddd61e34214d7d59a2d9a33c880e5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/257921 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2019-12-04 20:17:54 +00:00
* For Vulkan backend, we now require that the VkDevice, Queue, and Instance outlive
either the destruction or abandoning of the GrContext. Additionally, all
GrBackendTextures created via GrContext::createBackendTexture calls must be deleted
before destroying or abandoning the GrContext.
https://review.skia.org/257921
Remove SkSize& SkSize::operator=(const SkISize&) Having it means you cannot write: SkSize size; float x, y; size = {x , y}; clang allows it but GCC does not, claiming it is ambiguous between the implicitly generated SkSize& SkSize::operator=(const SkSize&) and SkSize& SkSize::operator=(const SkISize&) clang gives the same error if the former is explicitly declared default. Change-Id: I3b64436ef6aa669b3d87e7f37057c5dcb4add987 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/257880 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2019-12-04 15:33:52 +00:00
* Removed SkSize& SkSize::operator=(const SkISize&)
https://review.skia.org/257880
Fix for 420/422 chroma subsampling of odd dimension images when converting from YUV to RGB on GPU. Change-Id: I872e1e85f3efccce7bf93d2f3d5f2a841b8a5b0f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/257680 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Jim Van Verth <jvanverth@google.com>
2019-12-04 16:05:35 +00:00
* SkISize width() and height() now constexpr
https://review.skia.org/257680
Test texture domain effect with local matrix. Add a common way to make rect op for testing that uses a GP with a local matrix. Change-Id: I958d1230bd5067b2e4b60fcd374e2f7718681e43 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/255782 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com>
2019-11-22 21:56:36 +00:00
* Added SkMatrix::MakeTrans(SkVector) and SkRect::makeOffset(SkVector).
https://review.skia.org/255782
Clamp RGB outputs of GrYUVtoRGBEffect. The matrices we're using can produce very slightly out of range color channels. This gives surprising results when in shader blending is used for color burn and color dodge. After this change we clamp the RGB values to 0..1 before applying premul. Adds a GM modeled on a blink layout test that shows the problem using SkImageMakeFromYUVAPixmaps. Bug: skia:9619 Change-Id: I446d39763a7f5a2f7c5f61d94d163927d851baa3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/253879 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2019-11-11 18:47:25 +00:00
* Added SkImageInfo::MakeA8(SkISize) and added optional color space parameter to
SkImageInfo::MakeN32Premul(SkISize).
Fix up release notes file Bug: skia: Change-Id: I124abbb54f13e70e5d9c6a71f43cea5adf0ce49a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/250038 Auto-Submit: Heather Miller <hcm@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-22 19:45:45 +00:00
[canvaskit] Expand SkAnimatedImage With these things exposed, I think Flutter will not need CanvasKit to expose the very complex SkCodec API. Change-Id: Iace1b496d1dcb8842181466e860e8f212aba7b48 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/253542 Reviewed-by: Leon Scroggins <scroggo@google.com>
2019-11-08 14:55:15 +00:00
* Added dimensions() and getFrameCount() to SkAnimatedImage
https://review.skia.org/253542
Reland "Fully embrace skcms types in SkColorSpace API" This reverts commit 6af9b1c673b1f017ecf5b845a1328a6c539f3c0d. Change-Id: I7954951497e57475ab6f1c7f946b47aa17f1ac8c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/252817 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2019-11-05 16:41:16 +00:00
* Removed SkMatrix44 version of toXYZD50 from SkColorSpace. Switched to skcms types in
transferFn, invTrasnferFn, and gamutTransformTo functions.
https://review.skia.org/252596
SkColorMatrix cleanup Remove the RGB/YUV helpers (use SkYUVMath instead), along with the unused get20/set20. Change-Id: Id83467a1b7f33657869e0a933af75387a4e36a88 Bug: skia:9543 Change-Id: Id83467a1b7f33657869e0a933af75387a4e36a88 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/252188 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Mike Reed <reed@google.com>
2019-11-01 21:10:28 +00:00
* Removed rotation and YUV support from SkColorMatrix
https://review.skia.org/252188
Add BT2020 (non-constant-luminance) YUV color space Change-Id: I8ef13b78a5f2f49ff9c59db285b3e0e7ee708c9b Bug: chromium:960620 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/252160 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2019-11-01 14:02:24 +00:00
* Added kBT2020_SkYUVColorSpace. This is BT.2020's YCbCr conversion (non-constant-luminance).
https://review.skia.org/252160
Remove legacy async read pixels APIs. Chrome is now using the newer version. Bug: skia:8962 Change-Id: I8c38bde63e1e032281d19b0cc7cb8cf301061a9e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/251198 Commit-Queue: Greg Daniel <egdaniel@google.com> Auto-Submit: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2019-10-28 13:16:47 +00:00
* Remove old async read pixels APIs
https://review.skia.org/251198
Make SkBlendMode_AsCoeff() public Change-Id: Ie704f0f83b1034ff8ee1a5d3cd194e555cae20a3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/252600 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Commit-Queue: Mike Klein <mtklein@google.com> Auto-Submit: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2019-11-04 20:01:52 +00:00
* Expose SkBlendModeCoeff and SkBlendMode_AsCoeff for Porter-Duff blend modes.
https://review.skia.org/252600
Fix up release notes file Bug: skia: Change-Id: I124abbb54f13e70e5d9c6a71f43cea5adf0ce49a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/250038 Auto-Submit: Heather Miller <hcm@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-22 19:45:45 +00:00
* * *
update formatting, add links Bug: skia: Change-Id: I1f74e95781d575570d560003f706eb073b1b0a01 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249118 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-10-17 04:03:36 +00:00
Fix up release notes file Bug: skia: Change-Id: I124abbb54f13e70e5d9c6a71f43cea5adf0ce49a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/250038 Auto-Submit: Heather Miller <hcm@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-22 19:45:45 +00:00
Milestone 79
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
update formatting, add links Bug: skia: Change-Id: I1f74e95781d575570d560003f706eb073b1b0a01 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249118 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-10-17 04:03:36 +00:00
Simple iterator for textblobs bug: skia:9503 Change-Id: I5b4e2434c6085d048d6d0fc42b2b928439b1066b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/246296 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2019-10-04 14:18:34 +00:00
* SkTextBlob::Iter to discover the glyph indices and typefaces in each run
https://skia-review.googlesource.com/246296
Starting to hack up HDR transfer function support Brings over skcms' encoding scheme, etc. Change-Id: Ib8abec911acd1c50df3b201b4a9bde01b1cb123b Bug: chromium:960620 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249000 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2019-10-16 17:58:42 +00:00
* Added support for PQ and HLG transfer functions to SkColorSpace.
update formatting, add links Bug: skia: Change-Id: I1f74e95781d575570d560003f706eb073b1b0a01 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249118 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-10-17 04:03:36 +00:00
https://skia-review.googlesource.com/c/skia/+/249000
Starting to hack up HDR transfer function support Brings over skcms' encoding scheme, etc. Change-Id: Ib8abec911acd1c50df3b201b4a9bde01b1cb123b Bug: chromium:960620 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249000 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
2019-10-16 17:58:42 +00:00
Add new public api on GrContext for computing image size. This api will replace ComputeTextureSize. Change-Id: I9befd8706f44250805315bdb13a559cf81c7142d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/247337 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
2019-10-09 16:38:22 +00:00
* Added new api on GrContext ComputeImageSize. This replaces the hold static helper
ComputeTextureSize.
update formatting, add links Bug: skia: Change-Id: I1f74e95781d575570d560003f706eb073b1b0a01 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/249118 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Heather Miller <hcm@google.com> Auto-Submit: Heather Miller <hcm@google.com>
2019-10-17 04:03:36 +00:00
https://skia-review.googlesource.com/c/skia/+/247337
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
Reland "Reland "SkSurface asynchronous read APIs allow client to extend pixel lifetime"" This is a reland of 6fc04f88a89ed2c9a1b4aa48bcd28602a69a457b Original change's description: > Reland "SkSurface asynchronous read APIs allow client to extend pixel lifetime" > > This is a reland of ce240cc6fd8ec95bd051c7df2173dad2ae8f6ad6 > > Original change's description: > > SkSurface asynchronous read APIs allow client to extend pixel lifetime > > > > Previously the pixel data passed to the client was only valid during > > the client's callback. This meant if the client wanted to defer > > processing of the data a copy was necessary. > > > > Now we pass an object to the callback and the pixel lifetime is tied > > to the lifetime of that object. > > > > The object may be holding a GPU transfer buffer mapped. We don't assume > > that the object will be released on the direct GrContext thread. So > > when the object is destroyed it posts a message to a new type, > > GrClientMappedBufferManager, hanging off the direct context. The direct > > context will periodically check for messages and unmap and then unref > > buffers so that they can be reused. Currently this is done in > > GrContext::performDeferredCleanup() and GrDrawingManager::flush(). > > > > The old API is kept around for backwards compatibility but it is > > reimplemented as a bridge on top of the new mechanism. > > > > Also a utility function to SkImageInfo is added to directly make a new > > info with a specified dimensions rather than passing the width and > > height separately to makeWH(). > > > > Bug: chromium:973403 > > Bug: skia:8962 > > > > Change-Id: Id5cf04235376170142a48e90d3ecd13fd021a2a6 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245457 > > Reviewed-by: Brian Osman <brianosman@google.com> > > Commit-Queue: Brian Salomon <bsalomon@google.com> > > Bug: chromium:973403, skia:8962 > Change-Id: I5cecd36276c8b6dc942cf549c7095db2df88530c > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245678 > Reviewed-by: Brian Salomon <bsalomon@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> Bug: chromium:973403, skia:8962 Change-Id: Ie584c1c3ef8021c976f71b708e53871c693cc450 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/246057 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2019-10-03 17:26:54 +00:00
* New versions of SkSurface async-rescale-and read APIs that allow client to extend
the lifetime of the result data. Old versions are deprecated.
https://review.skia.org/245457
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
* Add SkColorInfo. It's dimensionless SkImageInfo.
https://review.skia.org/245261
* Added SkPixmap-based createBackendTexture method to GrContext. This allows clients to create
backend resources (initialized with texture data) that Skia/Ganesh doesn't know about/track.
https://review.skia.org/244676
Revert "Revert "add explicit src and dst colorspaces to filterColor4f"" This reverts commit 355111bf7cf53c11bfc6dd4c6924fc9895d3fd8f. Change-Id: I0aa45bf92f437a0d680aa788ef4dc5c0299c6fe0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/244882 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com> Auto-Submit: Mike Reed <reed@google.com>
2019-09-28 01:05:24 +00:00
* Add explicit src and dst colorspace parameters to SkColorFilter::filterColor4f()
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
https://review.skia.org/244882
* Remove Vulkan/Metal float32 RGBA texture support
https://review.skia.org/244881
Revert "Revert "add explicit src and dst colorspaces to filterColor4f"" This reverts commit 355111bf7cf53c11bfc6dd4c6924fc9895d3fd8f. Change-Id: I0aa45bf92f437a0d680aa788ef4dc5c0299c6fe0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/244882 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com> Auto-Submit: Mike Reed <reed@google.com>
2019-09-28 01:05:24 +00:00
Set up prototype interface for MakeFromCAMetalLayer Change-Id: I87b0fc76ef48c1a21498e576853a6c3b4a6a98f9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/242563 Commit-Queue: Jim Van Verth <jvanverth@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2019-09-19 17:03:09 +00:00
* Add SkSurface::MakeFromCAMetalLayer
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
https://review.skia.org/242563
* Added kAlpha_F16_SkColorType, kRG_F16_SkColorType and kRGBA_16161616_SkColorType.
This is intended to help support HDR YUV uses case (e.g., P010 and P016). As such,
the addition is focused on allowing creation of SkPixmaps and SkImages and not
SkSurfaces (i.e., who wants to render to render to these?)
https://review.skia.org/241357
Introduce crowdsourced release notes file Bug: skia: Change-Id: I9d76b56ef88ec5caf305deed8bf839443427a260 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/229760 Commit-Queue: Heather Miller <hcm@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
2019-07-25 15:33:30 +00:00
Pull path nested types out to root level Change-Id: I66a854fa157c5c22d0caaf8bc6f5989baa876a5c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/241079 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2019-09-13 12:26:56 +00:00
* Start to move nested SkPath types (e.g. Direction, Verb) up to root level in SkPathTypes.h
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
https://review.skia.org/241079
make rectcontour and nestedrects private Change-Id: I37b81f3cd96acc310ce78244d427eeb9c7999061 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/241078 Reviewed-by: Mike Reed <reed@google.com> Commit-Queue: Mike Reed <reed@google.com> Auto-Submit: Mike Reed <reed@google.com>
2019-09-12 21:09:12 +00:00
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
* Remove isRectContour and ksNestedFillRects from public
https://review.skia.org/241078
* Added kRG_88_SkColorType. This is intended to help support YUV uses case (e.g., NV12).
As such, the addition is focused on allowing creation of SkPixmaps and SkImages and not
SkSurfaces (i.e., who wants to render to RG?)
https://review.skia.org/239930
https://review.skia.org/235797
Make the size of program/pipeline caches configurable in GrContextOptions Change-Id: I988fd8cc7e78e2124f20b7d8a815f160bb166756 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/239756 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-09-06 14:16:02 +00:00
* Make the size of program/pipeline caches configurable via
GrContextOptions::fRuntimeProgramCacheSize
https://review.skia.org/239756
doc: cleanup release notes No-try: true Change-Id: Ice2803974c7451aff200c75a609414cf9f0a17fe Reviewed-on: https://skia-review.googlesource.com/c/skia/+/245374 Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-01 19:15:36 +00:00
* Added kAlpha_16_SkColorType and kRG_1616_SkColorType. This is intended to help support HDR YUV
uses case (e.g., P010 and P016). As such, the addition is focused on allowing creation of
SkPixmaps and SkImages and not SkSurfaces (i.e., who wants to render to render to these?)
https://review.skia.org/239930
* Add GrContext::precompileShader to allow up-front compilation of previously-cached shaders.
https://review.skia.org/239438
* * *
Introduce crowdsourced release notes file Bug: skia: Change-Id: I9d76b56ef88ec5caf305deed8bf839443427a260 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/229760 Commit-Queue: Heather Miller <hcm@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
2019-07-25 15:33:30 +00:00
Milestone 78
update release notes file, site for 84 branch Bug: skia: Change-Id: Icdc056e7972cad814fa633b7c9b021f4534d60c9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/290119 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Heather Miller <hcm@google.com>
2020-05-15 14:53:49 +00:00
------------
Introduce crowdsourced release notes file Bug: skia: Change-Id: I9d76b56ef88ec5caf305deed8bf839443427a260 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/229760 Commit-Queue: Heather Miller <hcm@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
2019-07-25 15:33:30 +00:00
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* SkDrawLooper is no longer supported in SkPaint or SkCanvas.
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
https://review.skia.org/230579
https://review.skia.org/231736
remove code now that loopers are dead (w.r.t. canvas and paint) We can't remove the loopers themselves, as they are still used by android and chrome (they just don't ever pass them to skia). Eventually each of those clients will resolve this, but for now we just keep the classes (and tests) in skia. Bug: skia:4783 Change-Id: I5f507e6bb82280f2bc7c0b21eebe59c287aa9265 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/230579 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Mike Reed <reed@google.com>
2019-07-29 21:52:48 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* SkPath::Iter::next() now ignores its consumDegenerates bools. Those will so
go away entirely
https://review.skia.org/235104
never check for degenerates in path::iter This does not appear to be used externally, and only internally in 3 sites in GPU, which we can handle explicitly. For now you can still pass bools to Iter::next() but they are ignored. After this lands, I will update the callers to remove those. FWIW: none of our other tests or gms seem to notice this change... Bug: skia:9339 Change-Id: If40077b1ebb3d47cfce0ec43996ff272318e4a62 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235104 Reviewed-by: Chris Dalton <csmartdalton@google.com> Commit-Queue: Mike Reed <reed@google.com>
2019-08-16 17:30:34 +00:00
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* SkImage: new factories: DecodeToRaster, DecodeToTexture
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
https://review.skia.org/234476
expose direct methods for decoding to an image These are meant to contrast MakeFromEncoded(), and emphasize that it is deferred/cached, while the new methods are not. Change-Id: I83ac22394cb14cdc84ff8507a514bf708734b84f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234476 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Reed <reed@google.com>
2019-08-14 15:08:41 +00:00
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* SkImageFilter API refactor started:
- Provide new factory API in include/effects/SkImageFilters
- Consolidated enum types to use SkTileMode and SkColorChannel
- Hide filter implementation classes
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
- Hide previously public functions on SkImageFilter that were intended for
internal use only
https://review.skia.org/230198
https://review.skia.org/230876
https://review.skia.org/231256
Reland: SkColorfilters::HSLAMatrix Introduce an SkColorFilter_Matrix flavor operating in HSLA space. (CPU-only for now) TBR=mtklein Change-Id: I3a8686775593ccc9ce0e46159a17bec7f4ab4228 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/231736 Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org>
2019-08-01 21:20:19 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* SkColorFilters::HSLAMatrix - new matrix color filter operating in HSLA
space.
https://review.skia.org/231736
Rework how backend-specific formats are retrieved from GrBackendFormat. Change-Id: If1047c477ff3f2cc4aaf19d2e2e838f2dbcb0126 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/233160 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
2019-08-08 16:08:24 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* Modify GrBackendFormat getters to not return internal pointers. Use an enum
class for GL formats.
https://review.skia.org/233160
Expose GrContext::dump() when SK_ENABLE_DUMP_GPU is defined. Change-Id: I499228454bc387de2c988b8f607320b7e5ea57b0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/233557 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
2019-08-09 13:41:48 +00:00
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* Expose GrContext::dump() when SK_ENABLE_DUMP_GPU is defined.
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
https://review.skia.org/233557
Reland: Enable YCbCr sampler support on platforms other than Android Previously YCbCr Vulkan samplers were supported only on Android for external images, while Vulkan requires YCbCr sampler for I420 YUV image formats such as VK_FORMAT_G8_B8_R8_3PLANE_420_UNORM, VK_FORMAT_G8_B8R8_2PLANE_420_UNORM. This CL: - Adds VK_FORMAT_G8_B8_R8_3PLANE_420_UNORM and VK_FORMAT_G8_B8R8_2PLANE_420_UNORM as supported Vulkan formats - Updates GrVkYcbcrConversionInfo to add fFormat field and allow fExternalFormat=0. - Removes assertions format=VK_FORMAT_UNDEFINED for all images that have ycbcr info. Bug: chromium:981022 Change-Id: Id4d81b20d9fda4d9ad0831f77e6025eed3db2bfd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/233776 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> Auto-Submit: Sergey Ulanov <sergeyu@chromium.org>
2019-08-12 05:46:33 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* Vulkan backend now supports YCbCr sampler for I420 Vulkan images that are
not backed by external images.
https://review.skia.org/233776
Hook up SkHeifCodec for ImageDecoder animation bug: 78868457 bug: 120414514 test: local test with OpenGL Rendrerer Tests animation demo and heifs files Change-Id: I09a7667a57f545927dbe9ac24c1a6b405ff0006d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/232839 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com> Reviewed-by: Derek Sollenberger <djsollen@google.com> Auto-Submit: Chong Zhang <chz@google.com>
2019-08-14 15:29:29 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* Add SkCodec::SelectionPolicy for distinguishing between decoding a still
image or an image sequence for a container format that has both (e.g. HEIF).
https://review.skia.org/232839
Remove unused dstColorSpace parameter from makeTextureImage Old version still exists until client code is migrated. Change-Id: I087b6b977f586d334f9a20954e7ed7b5e4b7ea5e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234579 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2019-08-14 17:19:04 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* SkImage::makeTextureImage and SkImage::MakeCrossContextFromPixmap no longer
take an SkColorSpace parameter. It was unused.
https://review.skia.org/234579
https://review.skia.org/234912
Add SkImage::reinterpretColorSpace This reinterprets the contents of an image as though they were in a different color space. Helpful for tools and debug visualization. Bug: chromium:795132 chromium:985500 Change-Id: Ia8739bbd73d72249b8bee9d51dfa11d560a2a1f5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234328 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2019-08-14 20:14:51 +00:00
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
* SkImage::reinterpretColorSpace - to reinterpret image contents in a new
color space.
https://review.skia.org/234328
Remove MakeCrossContextFromEncoded, unused argument from ...FromPixmap Change-Id: I962e923a4994eb549a9c1002323f33d05b936b84 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234912 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
2019-08-15 16:13:53 +00:00
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* Removed SkImage::MakeCrossContextFromEncoded.
RELEASE_NOTES: formatting cleanup, add some urls No-Try: true Change-Id: Ic8f8416fa10d37205ab27a8e8da173869d158ab1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235862 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 18:52:02 +00:00
https://review.skia.org/234912
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
Implement fences and semaphores for Metal. GrFence is implemented by a single MTLSharedEvent where we increase the value with each new invocation. GrSemaphore uses a MTLEvent (the assumption here is that we are signaling and waiting on the same device) with an associated value that defaults to 1. For generating a large number of GrSemaphores at once it should be possible to use the same MTLEvent but with different assigned values. Bug: skia:8243 Change-Id: Ic7de2d9d295fbe51e67bc7c3c4354257cb0774d4 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/233416 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2019-08-28 18:35:55 +00:00
* Add Metal support for GrFence, GrSemaphore, and GrBackendSemaphore
https://review.skia.org/233416
RELEASE_NOTES: note change to SkMallocPixelRef No-Try: true Change-Id: I04b5193bab1524debb5a9ad2baf655ddd4209288 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/235857 Reviewed-by: Heather Miller <hcm@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-08-20 15:28:37 +00:00
* SkMallocPixelRef: remove MakeDirect and MakeWithProc from API.
https://review.skia.org/234660
remove unused sk_sp comparison operators These unused comparison operators are the only users of <functional> in SkRefCnt.h, for std::less. <functional> is an expensive header to compile, and SkRefCnt.h is popular, so it helps to cut dependencies like this. Mostly we just need to add #include <functional> in a few places that were picking it up via SkRefCnt.h. In SkPixmapPriv.h, it looked simpler to template the argument, since everything was inline anyway. Change-Id: I7c125bb26a04199847357c729a1b178256c6ef8d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/236942 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@google.com>
2019-08-27 13:12:15 +00:00
Add Color4f variant for DrawEdgeAAQuad Bug: chromium:795132,chromium:985500 Change-Id: Idbb4d45b29d2c0d0fd54b05b807086ecf0b8cf26 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/237492 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com>
2019-08-27 19:25:49 +00:00
* Remove 4-parameter variant of SkRect::join() and intersect(), and
noemptycheck variants of intersect().
https://review.skia.org/235832
https://review.skia.org/237142
remove unused sk_sp comparison operators These unused comparison operators are the only users of <functional> in SkRefCnt.h, for std::less. <functional> is an expensive header to compile, and SkRefCnt.h is popular, so it helps to cut dependencies like this. Mostly we just need to add #include <functional> in a few places that were picking it up via SkRefCnt.h. In SkPixmapPriv.h, it looked simpler to template the argument, since everything was inline anyway. Change-Id: I7c125bb26a04199847357c729a1b178256c6ef8d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/236942 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Mike Klein <mtklein@google.com>
2019-08-27 13:12:15 +00:00
* Remove unused sk_sp comparison operators.
https://review.skia.org/236942
Add Color4f variant for DrawEdgeAAQuad Bug: chromium:795132,chromium:985500 Change-Id: Idbb4d45b29d2c0d0fd54b05b807086ecf0b8cf26 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/237492 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com>
2019-08-27 19:25:49 +00:00
* Add SkColor4f variant to experimental_DrawEdgeAAQuad for SkiaRenderer.
https://review.skia.org/237492
Remove maxCount resource cache limit Change-Id: I0f1064c8433d69e24ca3d5b970c99d539dc1dadd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/238442 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-09-03 14:29:20 +00:00
* Deprecated maxCount resource cache limit for Ganesh.
This hasn't been relevant for a long time.
Move SkSL caching out of GR_TEST_UTILS, trim persistent cache options Combines the two boolean options into a single tri-state enum. Old GLSL option is still present (temporarily) until Chrome is switched over. Also add a type tag for cached program binaries, so we can safely detect cache entries of the wrong type. Change-Id: I0ddeefa9180b27bc2c46e2e7b77e6c9cdf4a730a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/238856 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-09-03 18:59:26 +00:00
* Changed GrContextOptions' fDisallowGLSLBinaryCaching to fShaderCacheStrategy,
and allow caching SkSL.
https://review.skia.org/238856
Add GL_QCOM_TILED_RENDERING as an optional interface Change-Id: I29b53e674cd788ac188295060028bbb7805858a4 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/246281 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-10-07 17:51:07 +00:00
* Use GL_QCOM_TILED_RENDERING to explicitly discard stencil
Fix up release notes file Bug: skia: Change-Id: I124abbb54f13e70e5d9c6a71f43cea5adf0ce49a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/250038 Auto-Submit: Heather Miller <hcm@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
2019-10-22 19:45:45 +00:00
* Added RELEASE_NOTES.txt file
https://review.skia.org/229760
Add back door support for GLSL tessellation shaders Implements tessellation support at the Ganesh level, and adds back door methods for supplying raw GLSL strings directly to the OpenGL driver. Adds a new gm to verify tessellation is works in GL. Change-Id: Idfc285b955cbe5e8e6bf0475be8b518b0cc6ed2c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/261196 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
2019-12-27 21:56:38 +00:00
* Implemented internal support for OpenGL tessellation.
Reference in New Issue Copy Permalink