| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Mac viewer builds with bazel Some of the required objective-c files were compiled with ARC but they do MRC and so didn't compile. objc_library has a non_arc_srcs list that compiles files with -fno-objc-arc but using this through the skia_objc_library macro causes an obscure crash in bazel. So it is reimplemented in the macro using a nested objc_library that augments its copts. Change-Id: Ie04f2531ff741ac0d209a6bdca2f793c488e63ab Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1051339 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> | 9 个月前 | |
Prepare more call-sites for std::span Change-Id: Ieccf7ff710b32994fa16cc2fde3f428619925e54 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1099836 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Eric Boren <borenet@google.com> | 7 个月前 | |
Fix blendmodes_canvas GM in CanvasKit Due to minification stuff, a "not blend mode" mode snuck into Object.keys() and was throwing off the GM. By making this a fixed list, it'll be more readable anyway. Change-Id: I26aaae309976dadef2e6cafc9f91fe630d2bd851 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1087596 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Auto-Submit: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Julia Lavrova <jlavrova@google.com> | 8 个月前 | |
Mark some private constructors explicit This is considered good practice by the style guide. https://google.github.io/styleguide/cppguide.html#Implicit_Conversions I don't expect this to break any clients, but I've made it a separate commit from the ones touching the public API. I asked Gemini CLI to help me with this one. I had it use a regex to find candidates and then it narrowed down the pool and applied the edits. Change-Id: I3b519698508b186438a4a185c92472dccfa15313 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1075876 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 8 个月前 | |
Reapply "Move SkJSON.h to //modules/jsonreader" This reverts commit b5d22cf4f59a02cc806e96e1c811ccff508b36ec. http://review.skia.org/927040 http://review.skia.org/927697 was landed first in Skia, then https://crrev.com/c/6081356 landed in Chromium Original commit message: JSON parsing doesn't need to be part of Skia's core functionality, so we can move it to either //modules or //tools. Since it *is* part of Skottie's functionality, //modules seemed like the best home for it. SkJSONReader has lots of little-endian specific code that has not been ported to big-endian code. While core Skia doesn't have great support for big-endian systems, moving this out should allow core Skia to compile on big-endian systems without hitting the static_assert(false) statements Change-Id: Ie8d4e0a41ad2a638250c297dd2b9cf47e2effe24 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/928137 Reviewed-by: Florin Malita <fmalita@google.com> | 1 年前 | |
Reland "Optional PathOps" This reverts commit 3e8a27e6f9c32dffb88c61f50305115c4c4e72db. Reason for revert: reland with fixes Original change's description: > Revert "Optional PathOps" > > This reverts commit 9f8ffd1e2056d5b686d9f81b740e995df5140c30. > > Reason for revert: NativeFonts + ASAN failures > > Original change's description: > > Optional PathOps > > > > Move PathOps out of core :skia, into its own lib (:pathops), and only > > use with clients that need it (:pdf and :xps). > > > > To avoid dependency cycles, also move :pdf and :xps out of :skia, and > > introduce another lib for their shared SkClipStackUtils functionality. > > > > Change-Id: I19f42917ea37e34dae2d2d72a4714797db9b565c > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1011859 > > Reviewed-by: Ben Wagner <bungeman@google.com> > > Commit-Queue: Florin Malita <fmalita@google.com> > > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Change-Id: I23be2c95b8a52f352eeecad967afc4ce60273d5e > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1012860 > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Auto-Submit: Florin Malita <fmalita@google.com> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Change-Id: I4b31f0f79d712e8664147fe4b1bf9b76f5226909 Cq-Do-Not-Cancel-Tryjobs: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1013977 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Florin Malita <fmalita@google.com> | 1 年前 | |
Roll skcms from 026f8a969740 to bf2d52b98a42 (1 revision) https://skia.googlesource.com/skcms.git/+log/026f8a969740..bf2d52b98a42 2025-11-26 sergiog@microsoft.com [rust bmp] Add enum name to Skcms_Signature enum If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skcms-skia-autoroll Please CC fmalita@google.com,kjlubick@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Cq-Include-Trybots: skia/skia.primary:Canary-Chromium Tbr: fmalita@google.com,kjlubick@google.com Change-Id: I1073d3fde057a7cb51a6e03e29332f4acd1625e9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1111556 Commit-Queue: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: skia-autoroll <skia-autoroll@skia-public.iam.gserviceaccount.com> | 7 个月前 | |
Make SkSerialProcs return pointer-to-const This should let us more cleanly fold https://review.skia.org/1084056 into Chromium. The #if guard should let us roll it out incrementally as Chromium, Android and others will need to update some of their callsites. Change-Id: I8b29f5e7306cbfcc7c5be9d7bc04fe0e732a3437 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1109339 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Florin Malita <fmalita@google.com> | 7 个月前 | |
Adapt remaining callers to be compatible with std::span After this lands, we need only test that our clients can build with the std::span alias, after which we can remove the flag. Change-Id: I80eb1c0356eea816865e51b2236b7b10b1dff1f2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1104616 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Eric Boren <borenet@google.com> | 7 个月前 | |
Add explicit keyword to many public single argument constructors This is considered good practice by the style guide. https://google.github.io/styleguide/cppguide.html#Implicit_Conversions I asked Gemini CLI to help me with this one. I had it use a regex to find candidates and then it narrowed down the pool and applied the edits. This could break clients if they were relying on the auto-conversion. Change-Id: I25f9b294d34c2c967504c59f0389fddc27237dea Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1075696 Reviewed-by: Michael Ludwig <michaelludwig@google.com> | 8 个月前 | |
Add predefined encoded origin inverse function Updates GMs and tests to use the inverse. Adds a unit test to ensure it matches the actual computed inverse from SkEncodedOriginToMatrix. Updates Ganesh/Graphite's YUV image shaders to use the direct inverse instead of recomputing the inverse from the original origin matrix. Change-Id: I803f77be44e0985ddfb7d7df004d861dad0fc206 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1091176 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> | 8 个月前 | |
Reapply "DM builds with graphite enabled and ganesh disabled" Fixes no-gpu build. Fixes chrome build. This reverts commit 479696a3c5bfeb263a18c9260cb581b20ea4b556. Cq-Include-Trybots: luci.skia.skia.primary:Canary-Chromium Change-Id: Idbbb7759b229d5c28de197151dbb260f26e7c829 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1084176 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Commit-Queue: Brian Salomon <briansalomon@gmail.com> Reviewed-by: Greg Daniel <egdaniel@google.com> | 8 个月前 | |
Prepare more call-sites for std::span Change-Id: Ieccf7ff710b32994fa16cc2fde3f428619925e54 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1099836 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Eric Boren <borenet@google.com> | 7 个月前 | |
Prepare more call-sites for std::span Change-Id: Ieccf7ff710b32994fa16cc2fde3f428619925e54 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1099836 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Eric Boren <borenet@google.com> | 7 个月前 | |
Add gni file group for svg writer We'd like to avoid hard-coded file paths (e.g the ones in Chromium) [1] [1] https://source.chromium.org/chromium/chromium/src/+/main:skia/BUILD.gn;l=398-401;drc=a47c17cf38abc308d3cd89abf2cb60dab55766cc Change-Id: I9b49296a681bfc6e1260cc0ef2c6e5eb1f18439f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1100316 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 7 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 9 个月前 | ||
| 7 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 1 年前 | ||
| 1 年前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 7 个月前 |