AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
e267a5bb7a
skia2/modules/pathkit/CHANGELOG.md

43 lines
1.0 KiB
Markdown
Raw Normal View History

[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
# PathKit Changelog
All notable changes to this project will be documented in this file.
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
## [Unreleased]
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[canvaskit,pathkit] Update to emsdk 2.0 There was a break that was solved by adding -lGL to our link steps. I discovered a few extra flags to aid in debugging builds and I've left those in (they aren't too noisy IMO). This changes the base dockerfile to use the official emscripten one. Code size delta for full build is +5 kb For future reference, emsdk decides which "library JS" files to pull in using https://github.com/emscripten-core/emscripten/blob/a83ba99d60a0bedaa5cfaecf22e2ce8c750e0147/tools/building.py#L1553 Those JS files live in src (e.g. https://github.com/emscripten-core/emscripten/blob/a83ba99d60a0bedaa5cfaecf22e2ce8c750e0147/src/library_html5_webgl.js#L222) and define functions that the C++ code can call. I'd like to follow-up on what -lEGL is doing. Also, since the new image no longer has depot_tools, we need to make docker/skia-wasm-release/Dockerfile install it. Change-Id: I5a38e61e5080e9c4cb1e0a7e031509bcb107ff86 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/311726 Reviewed-by: Nathaniel Nifong <nifong@google.com>
2020-08-20 14:47:34 +00:00
### Changed
[canvaskit] Build with emsdk 2.0.20 Change-Id: I4c3bdc8dd0d0dc8b12b2d430d5ea9a20d170b82b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/412876 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2021-05-27 12:34:47 +00:00
- Now compile with emsdk 2.0.20
[canvaskit,pathkit] Update to emsdk 2.0 There was a break that was solved by adding -lGL to our link steps. I discovered a few extra flags to aid in debugging builds and I've left those in (they aren't too noisy IMO). This changes the base dockerfile to use the official emscripten one. Code size delta for full build is +5 kb For future reference, emsdk decides which "library JS" files to pull in using https://github.com/emscripten-core/emscripten/blob/a83ba99d60a0bedaa5cfaecf22e2ce8c750e0147/tools/building.py#L1553 Those JS files live in src (e.g. https://github.com/emscripten-core/emscripten/blob/a83ba99d60a0bedaa5cfaecf22e2ce8c750e0147/src/library_html5_webgl.js#L222) and define functions that the C++ code can call. I'd like to follow-up on what -lEGL is doing. Also, since the new image no longer has depot_tools, we need to make docker/skia-wasm-release/Dockerfile install it. Change-Id: I5a38e61e5080e9c4cb1e0a7e031509bcb107ff86 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/311726 Reviewed-by: Nathaniel Nifong <nifong@google.com>
2020-08-20 14:47:34 +00:00
[pathkit] Publish 0.7.0 to npm Change-Id: I181e38e97888e9b319f62f1bbd6f4e832248d7dc Reviewed-on: https://skia-review.googlesource.com/c/skia/+/303923 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2020-07-20 14:20:56 +00:00
## [0.7.0] 2019-07-20
[canvaskit] Update to build with emscripten 1.38.47 Bug: skia:9052 Change-Id: Ic1aec2c9b8ace556f5b77862582894f59fd26b64 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/247302 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-10-09 14:46:14 +00:00
### Changed
Upgrade CanvasKit emscripten to version 1.39.16. Changes are largely mechanical. Non-mechanical changes to support newer versions of emscripten are enumerated below, in format ${EMSCRIPTEN_VERSION}: ${RELEVANT_CHANGE}. - 1.39.9: TOTAL_MEMORY has been renamed INITIAL_MEMORY. - 1.39.12: passing of linker flags to wasm-ld has changed in a way that requires supplying `--no-entry` to avoid error message "wasm-ld: error: entry symbol not defined (pass --no-entry to suppress): main". - 1.39.16: The factory function created by using `MODULARIZE` build option now returns a Promise instead of the module instance. As such, the ready.js workaround is removed. Note this is a breaking API change for CanvasKit, which now uses just `then()` and not `ready().then()`. - 1.38.33: `emsdk install` hasn't required the `-64bit` suffix on version names since `1.38.33`, so we remove them. E.g. `emsdk install sdk-1.39.6-64bit` simply becomes `emsdk install sdk-1.39.16`. cf. https://github.com/emscripten-core/emscripten/blob/master/ChangeLog.md Bug: NONE Change-Id: Iabec4bd5ad7db2e0715ad42c2e4cf7d67b192b4c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/291182 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2020-05-21 13:59:44 +00:00
- Updated to emscripten 1.39.16
[npm] Support npm publish flow See go/npm-publish and b/160246223 Change-Id: I5a92a19ae6be814ca9a98679cfe39184f7cdb184 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/303580 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2020-07-17 15:11:28 +00:00
- Support wombat-dressing-room. go/npm-publish
Upgrade CanvasKit emscripten to version 1.39.16. Changes are largely mechanical. Non-mechanical changes to support newer versions of emscripten are enumerated below, in format ${EMSCRIPTEN_VERSION}: ${RELEVANT_CHANGE}. - 1.39.9: TOTAL_MEMORY has been renamed INITIAL_MEMORY. - 1.39.12: passing of linker flags to wasm-ld has changed in a way that requires supplying `--no-entry` to avoid error message "wasm-ld: error: entry symbol not defined (pass --no-entry to suppress): main". - 1.39.16: The factory function created by using `MODULARIZE` build option now returns a Promise instead of the module instance. As such, the ready.js workaround is removed. Note this is a breaking API change for CanvasKit, which now uses just `then()` and not `ready().then()`. - 1.38.33: `emsdk install` hasn't required the `-64bit` suffix on version names since `1.38.33`, so we remove them. E.g. `emsdk install sdk-1.39.6-64bit` simply becomes `emsdk install sdk-1.39.16`. cf. https://github.com/emscripten-core/emscripten/blob/master/ChangeLog.md Bug: NONE Change-Id: Iabec4bd5ad7db2e0715ad42c2e4cf7d67b192b4c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/291182 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2020-05-21 13:59:44 +00:00
### Breaking
- `PathKitInit(...)` now directly returns a Promise. As such, `PathKitInit(...).ready()`
has been removed.
[pathkit][canvaskit] roll versions This removes the buggy .then() constructor, primarily. Docs-Preview: https://skia.org/?cl=195126 Bug: skia: Change-Id: Ie20512267ce3f822eb0d68e71fc07f3d3245c1e9 Reviewed-on: https://skia-review.googlesource.com/c/195126 Auto-Submit: Kevin Lubick <kjlubick@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-02-25 21:04:02 +00:00
## [0.6.0] 2019-02-25
[*kit] fulfil promise immediately I think this would only commonly happen for cases which files aren't being loaded (debug asmjs), but could happen if multiple ready() calls are made (which is discouraged). Bug: skia: Change-Id: If848aad8603f997661502563136ac590cc9a6d5a Reviewed-on: https://skia-review.googlesource.com/c/181409 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-01-07 03:27:54 +00:00
### Fixed
- Potential bug in `ready()` if already loaded.
[pathkit][canvaskit] roll versions This removes the buggy .then() constructor, primarily. Docs-Preview: https://skia.org/?cl=195126 Bug: skia: Change-Id: Ie20512267ce3f822eb0d68e71fc07f3d3245c1e9 Reviewed-on: https://skia-review.googlesource.com/c/195126 Auto-Submit: Kevin Lubick <kjlubick@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-02-25 21:04:02 +00:00
### Removed
- Deprecated `PathKitInit.then()` see 0.5.1 notes.
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
## [0.5.1] 2019-01-04
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
### Changed
- `PathKitInit(...).then()` is no longer the recommended way to initialize things.
It will be removed in 0.6.0. Use `PathKitInit(...).ready()`, which returns a real Promise.
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
## [0.5.0] 2018-12-17
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
Updated PathKit to use same FOSS license as Skia proper.
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
[pathkit] Add .ready() to initialize See https://github.com/kripken/emscripten/issues/5820 and https://skia-review.googlesource.com/c/skia/+/181177 for more context. (Problem was discovered in CanvasKit, and this ports the same fix to PathKit). Bug: skia: Change-Id: Ic8b2fb399210631a571595a5b3d11d1736c00def Reviewed-on: https://skia-review.googlesource.com/c/181178 Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-01-04 19:38:29 +00:00
## [0.4.2] 2018-11-07
[canvaskit] Expand canvas2d API Made addPath take one more arg to allow for append/expand (which makes emulating the HTML canvas easier). Add Gold test for various lineTo/pathTo, etc. Make CanvasKit.Color() choose a better value for alpha when omitted (was 0, should be 1). Add some parsing logic to deal with colors/font sizes. Fonts are going to be rather complex it seems. Moves some arc-related logic to the JS side, (although this should preserve the behavior of CanvasKit.arc() to behave like the Canvas implementation) Make Examples and tests to a side-by-side comparison to HTML canvas where applicable. Add a Changelog for PathKit. There was a bug (I thought), but turns out I was wrong. The Changelog will be for future bug fixes. Bug: skia: Change-Id: I1bd603fdb518232604b098e24543e3453015b504 Reviewed-on: https://skia-review.googlesource.com/c/170446 Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-20 17:51:16 +00:00
Beginning of changelog.
Reference in New Issue Copy Permalink