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

374 lines
15 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
* * *
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
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
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
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
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
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 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
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