| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Use helper for checking optional Also moved build-guards to just around the android-specific part, so the code can always be built by the bots. Change-Id: I202515e51f6ff3b1ed5c7b48755b04c217598825 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1053076 Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> | 9 个月前 | |
SkCodec: Remove source format setter and getter This functionality isn't used (and the related member will be moved into the transform structure). Bug: skia:456675139 Change-Id: I378acfc8fbc90f34ec03a2d9aa8a96f1b3ae9463 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1112940 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Christopher Cameron <ccameron@google.com> Owners-Override: Kaylee Lubick <kjlubick@google.com> | 7 个月前 | |
Reland "Fix bazel release build to not compile debug code" This reverts commit c4077516d5e515897afbfd195212617a1195c88d. Reason for revert: Fix upstream in http://cl/794548367 Original change's description: > Revert "Fix bazel release build to not compile debug code" > > This reverts commit 2ece3e23a46cd87018b53c8e990322a47e0c2272. > > Reason for revert: I believe this is blocking the G3 roll > > Original change's description: > > Fix bazel release build to not compile debug code > > > > * Bazel release builds now set release / debug flags appropriately. > > > > * Other bazel users should do this similarly (e.g., g3). > > > > Change-Id: I98070ac1e66c71c9362e88408be61f912de2aed4 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1036776 > > Auto-Submit: Nicolette Prevost <nicolettep@google.com> > > Reviewed-by: Kaylee Lubick <kjlubick@google.com> > > Commit-Queue: Kaylee Lubick <kjlubick@google.com> > > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Change-Id: I5ea605f00ea34dc0b9b0c644471a9eacfae48c93 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1037496 > Auto-Submit: Robert Phillips <robertphillips@google.com> > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Change-Id: Ibf9e72d10f1aefdd6f03d576083abbdd13ad8b9f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1037576 Auto-Submit: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Nicolette Prevost <nicolettep@google.com> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> | 10 个月前 | |
2026.03.30 arkweb_144代码蓝黄同步 Signed-off-by: jiujiaoxiaogula <sujiahao10@huawei.com> Change-Id: Id6b8f8e08796118f0ec7d7f82b8f2430ef3448e1 | 3 个月前 | |
Make SkImage and SkImageGenerator return const SkData from refEncoded This behavior was implied (a shared pointer to SkData should always be treated as read-only) and is now made explicit. This also updates several places in our decoders where we promise not to modify the SkData that is passed in (since we will likely be caching it) and ask that the client not modify it either. This is partially guarded by SK_DISABLE_LEGACY_NONCONST_ENCODED_IMAGE_DATA so we can update Chromium and potentially other places. Change-Id: Id174dff4afa148745ad32dd7d5cd848931b77f9d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1084056 Reviewed-by: Florin Malita <fmalita@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 个月前 | |
[rust png] Support for using SkPngRustEncoder from SkXPSDevice.cpp. This Skia CL unblocks landing a PDFium CL at https://pdfium-review.googlesource.com/135730 by avoiding the following linking error from https://ci.chromium.org/ui/p/pdfium/builders/try/win_xfa_skia_rust/10/overview and https://logs.chromium.org/logs/pdfium/buildbucket/cr-buildbucket/8701086421222190401/+/u/compile_with_siso/stdout lld-link: error: undefined symbol: bool __cdecl SkPngEncoder::Encode(class SkWStream *, class SkPixmap const &, struct SkPngEncoder::Options const &) >>> referenced by .\..\..\third_party\skia\src\xps\SkXPSDevice.cpp:634 >>> obj/skia/skia/SkXPSDevice.obj:(private: long __cdecl SkXPSDevice::createXpsImageBrush(class SkPixmap const &, class SkMatrix const &, enum SkTileMode const (&)[2], unsigned char, struct IXpsOMTileBrush **)) Bug: chromium:381900683 Change-Id: I6ec0c48fb69b66bcd1de68ef8fea58a2aca57ccd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1073497 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Łukasz Anforowicz <lukasza@google.com> | 8 个月前 | |
[graphite] Stage SampleCount enum with uint assignment This allows SampleCount to appear like an enum but accept existing deprecated assignments from integer values. Once clients are updated, this can be reverted. Bug: b/452351267 Change-Id: I46c587f69adf6e6681bf0e9daf09c46e1da4540c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1104417 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com> | 7 个月前 | |
pathops Simplify() to return its result Change-Id: Ic73b31cca5df78d2dad3b7f10dc0f25196a36049 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1059763 Owners-Override: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Florin Malita <fmalita@google.com> | 9 个月前 | |
Remove old SkFontMgr_New_Fuchsia Remove the old SkFontMgr_New_Fuchsia which did not take an SkFontScanner. Change-Id: I990c3fecbfa09e0a5da7d8a18b1995196308577d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1113976 Reviewed-by: Florin Malita <fmalita@google.com> Reviewed-by: Julia Lavrova <jlavrova@google.com> Reviewed-by: Ben Wagner <bungeman@google.com> | 7 个月前 | |
Make SkData in SkJpegDecoder be pointer-to-const Requires https://crrev.com/c/7199994 to land in Chromium first. Change-Id: I6916f13481b9e6ee479fd203596504b714ffb594 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1109338 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 7 个月前 | |
Compile include/ headers on their own In https://review.skia.org/1024156, we removed IWYU logic, but this also removed the ability for us to test compiling headers on their own (which other build systems do). This caused a missing #include to land in Skia but break upstream things [1]. By adding in the way to generate .cpp files for headers, this will allow us to catch that type of error. Unlike before, where there was a hand-curated list of headers, I did my best to include all public headers (and then those from select/core) which would have caught the previous error. If we like this, I can add other subdirectories to src to the list. I also did my best to avoid duplication of header files, so they are listed in one place and not two (e.g. 2 different BUILD.bazel rules) so it's hopefully easier to maintain. Change-Id: Ib5589c6ac06709d5a472bc7c524cc7a3b2a506b0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1029376 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 10 个月前 | |
[rust png] Support for using SkPngRustEncoder from SkSVGDevice.cpp. Before this CL, SkSVGDevice.cpp would have a hard-coded dependency on libpng-based SkPngEncoder. After this CL, a libpng/Rust-agnostic callback is plumbed from the public SkSVGCanvas::Make API into SkSVGDevice.cpp. Bug: chromium:381900683 Change-Id: Ibaa422b2983a644066f6d6bb6430c2faa587e638 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1082557 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Łukasz Anforowicz <lukasza@google.com> Reviewed-by: Florin Malita <fmalita@google.com> | 8 个月前 | |
[vulkan] Update Skia's own Vulkan headers to v1.4.313.0 Includes support for more recent extensions, include VK_EXT_layer_settings. Update is done by copying files from third_party/external/vulkan-headers. Bug: skia:415228636 Change-Id: I15cc12d26cb971fc371e35109a1f0b74d105d33c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/988939 Commit-Queue: Shahbaz Youssefi <syoussefi@google.com> Auto-Submit: Shahbaz Youssefi <syoussefi@google.com> Reviewed-by: Nicolette Prevost <nicolettep@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> | 1 年前 | |
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 个月前 | |
Delete all filegroups that supported legacy (non-buffet) Bazel build We don't need most of the flags and all these file groups which had made a big "ball o mud" build. Change-Id: I690f38c3555e390b7c8d459b4649a270c3e03ce2 Bug: b/40045064 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/889266 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <bungeman@google.com> | 1 年前 | |
Make borenet@ and kjlubick@ OWNERS of include/*.bzl files We already have OWNERS for the BUILD.bazel files, so this seems like a natural extension of that. Change-Id: I938063d15ae3d1a09985f37988978ec64e429340 No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1021337 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Auto-Submit: Kaylee Lubick <kjlubick@google.com> | 11 个月前 |