| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
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 年前 | |
Silence unuseful printfs in tests ... making it easier to notice when something important is printed. ... all test results are checked as before. This an example of what has been silenced [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte xml parse error, line -1 SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) Change-Id: I54e8d7e40f0c44ccb80a5d2cc391bb4ce57e3cf5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1051676 Reviewed-by: Eric Boren <borenet@google.com> Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> | 9 个月前 | |
Silence unuseful printfs in tests ... making it easier to notice when something important is printed. ... all test results are checked as before. This an example of what has been silenced [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte [ToolUtils::equal_pixels] row 0 does not match byte for byte xml parse error, line -1 SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) SystemFontUse: OnlyCustom BasePath: resources/fonts/ Fonts: resources/fonts/fonts.xml FallbackFonts: (null) Change-Id: I54e8d7e40f0c44ccb80a5d2cc391bb4ce57e3cf5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1051676 Reviewed-by: Eric Boren <borenet@google.com> Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Mike Reed <mike@reedtribe.org> | 9 个月前 | |
SkXMLParser::parse: Optimize memory-backed streams Parsing XML in small chunks can hit some O(n^2) paths in libexpat that can result in extremely slow JPEG loading. See root cause analysis from Snild Dolkow (snild@sony.com) in the skia issue. All XMP metadata sent to SkXMLParser is memory-backed. Bug: b/295364874, skia:296854546 Change-Id: I802da43faa87d298157e36aa5e45ad01b1310455 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/745020 Commit-Queue: Christopher Cameron <ccameron@google.com> Reviewed-by: Brian Osman <brianosman@google.com> | 2 年前 | |
Mark more single argument constructors explicit I had a more expansive search pattern and found a few more candidates this time around. Gemini CLI did most of the work here and it does not appear we were implicitly using the conversion. Change-Id: Ia51d127a099dca4f76f7e13012d9f19dd1038e50 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1076816 Auto-Submit: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Florin Malita <fmalita@google.com> | 8 个月前 | |
[base] Move some files from include/private to include/private/base These ones were arbitrarily chosen among files that did not have any dependencies on include/core: include/private/SingleOwner.h include/private/SkDeque.h include/private/SkMalloc.h include/private/SkMutex.h include/private/SkOnce.h include/private/SkSemaphore.h include/private/SkStringView.h include/private/SkTFitsIn.h include/private/SkThreadAnnotations.h include/private/SkThreadID.h include/private/SkTo.h Change-Id: I32b945b459922e39e3e40c9f4af134b269d4958a Bug: skia:13983 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/626078 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Herb Derby <herb@google.com> Owners-Override: Kevin Lubick <kjlubick@google.com> | 3 年前 | |
[base] Move many files into base Moved from include/private into include/private/base: - SkAlignedStorage.h - SkContainers.h - SkFloatBits.h - SkPathEnums.h - SkTArray.h - SkTDArray.h - SkTypeTraits.h - SkVx.h - SkCPUTypes.h* Moved from include/core into include/private/base: - SkMath.h - SkSpan.h We still want SkSpan.h to be a part of the public API, so include/core/SkSpan.h still exists, it just includes include/private/base/SkSpan_impl.h We want to remove SkMath.h from the public API (most client uses have been removed already), but for now it too has a shim header file in include/core/SkMath.h As per the design of base, there are no includes inside of base on Skia files not also in base. SkLeftShift moved out of SkTypes.h and into SkMath.h, as that seemed like a better fit. U8CPU and U16CPU moved into their own file, SkCPUTypes.h while we decide where their future lies. Suggested review order: - include/private/base/BUILD.bazel - include/private/BUILD.bazel - include/BUILD.bazel - gn/*.gni (these are generated from the BUILD.bazel files) - BUILD.gn - public.bzl - Everything else, mostly done with find and replace and cleaned up with the rewrite_includes.py script. Change-Id: Icce0d0c445863c6908b63ec31b9595de400988dc Bug: skia:13983 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/629317 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com> Owners-Override: Kevin Lubick <kjlubick@google.com> | 3 年前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 1 年前 | ||
| 9 个月前 | ||
| 9 个月前 | ||
| 2 年前 | ||
| 8 个月前 | ||
| 3 年前 | ||
| 3 年前 |