| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Separate general defaults from Skia defaults In the build there are some defaults which actually apply to every use of a built-in target type, but there are some (particularaly warnings) which apply only to targets controlled by Skia. Currently these unwanted defaults are magically known to exist and removed wherever they are not wanted. Instead, create 'skia_' prefixed target templates and apply these defaults to those instead. Change-Id: I3a2afb53c7205a2e2748d1cfad46319f2e93d3b3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/385516 Commit-Queue: Ben Wagner <bungeman@google.com> Reviewed-by: Mike Klein <mtklein@google.com> | 5 年前 | |
Build everything with c++20 And clean up configs such as skia_use_cpp20. Bug: 311238827, chromium:388070065 Change-Id: I8149243f3f051c2432a1910febe1462d6292a360 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1082056 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Victor Vianna <victorvianna@google.com> Auto-Submit: Victor Vianna <victorvianna@google.com> | 8 个月前 | |
Reland "Delete GN support for building for fuchsia" This reverts commit 3539a4ea51922728f4ba06fb5ea9d8af45849e8f. Reason for revert: Fixed upstream Client CLs: - https://github.com/flutter/flutter/pull/171800 - https://github.com/flutter/flutter/pull/171874 - https://github.com/flutter/flutter/pull/172104 Bug: b/427681157 Original change's description: > Revert "Delete GN support for building for fuchsia" > > This reverts commit b6e7fee4ae9b9108da8264f24a3dd04462b912ed. > > Reason for revert: Breaking Fuchsia tests > > Bug: b/427681157 > Original change's description: > > Delete GN support for building for fuchsia > > > > This mostly was added in [1] for building SkQP for Fuchsia. > > There are no automated CI tests that build for Fuchsia in > > the Skia suite and I am not aware of any dev building these > > locally. Thus, I'm declaring these things unused. > > > > This CL *does not* delete SkFontMgr_fuchsia.cpp, which is > > used in chromium (e.g. [2]) which has its own GN rules for > > building Skia for Fuchsia [3]. Skia's ports.gni file still > > lists those files [4] so clients can use them in their own GN build. > > > > If this breaks things, it'll be illuminating and unexpected. > > > > [1] https://skia-review.googlesource.com/c/skia/+/244369 > > [2] https://source.chromium.org/chromium/chromium/src/+/main:skia/ext/fontmgr_fuchsia_unittest.cc;l=20;drc=f6492524f906d0728ddd5d41daf071f4b89300ba;bpv=1;bpt=1 > > [3] https://source.chromium.org/chromium/chromium/src/+/main:skia/BUILD.gn;l=469-484;bpv=1;bpt=0 > > [4] https://github.com/google/skia/blob/ef104904659ff58bca9e3332606009c26c7b5540/gn/ports.gni#L112-L116 > > > > Bug: b/427681157 > > Change-Id: I2f855c9c875a1844422178999958442be8dcb035 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1017296 > > Commit-Queue: Kaylee Lubick <kjlubick@google.com> > > Auto-Submit: Kaylee Lubick <kjlubick@google.com> > > Reviewed-by: Thomas Smith <thomsmit@google.com> > > Bug: b/427681157 > Change-Id: If4caf816db8e69c71ebd362fedbe8e9a39ddae48 > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1018617 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Reviewed-by: Thomas Smith <thomsmit@google.com> Bug: b/427681157 Change-Id: I6485b6404f9e573ac87f5bbdb64cf2060ffebb9c No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1019617 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Thomas Smith <thomsmit@google.com> | 11 个月前 | |
Remove exports_files_legacy We had been using these to export all files in G3 so our legacy rules could refer to them from the top level (not normally allowed because a new BUILD.bazel file acts as the root directory for all subfolders). Since those rules were deleted (see http://cl/650343844) we can delete these too. This was mostly done via mechanical find and replace. Some places actually needed to export files, so in those cases exports_files_legacy was changed to the built-in exports_files [1]. [1] https://bazel.build/reference/be/functions#exports_files Change-Id: I7952941c4b15f9e6af2a6647d303d5138087d431 Bug: b/256865569 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/874396 Reviewed-by: Nicolette Prevost <nicolettep@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 1 年前 | |
Reland "Delete GN support for building for fuchsia" This reverts commit 3539a4ea51922728f4ba06fb5ea9d8af45849e8f. Reason for revert: Fixed upstream Client CLs: - https://github.com/flutter/flutter/pull/171800 - https://github.com/flutter/flutter/pull/171874 - https://github.com/flutter/flutter/pull/172104 Bug: b/427681157 Original change's description: > Revert "Delete GN support for building for fuchsia" > > This reverts commit b6e7fee4ae9b9108da8264f24a3dd04462b912ed. > > Reason for revert: Breaking Fuchsia tests > > Bug: b/427681157 > Original change's description: > > Delete GN support for building for fuchsia > > > > This mostly was added in [1] for building SkQP for Fuchsia. > > There are no automated CI tests that build for Fuchsia in > > the Skia suite and I am not aware of any dev building these > > locally. Thus, I'm declaring these things unused. > > > > This CL *does not* delete SkFontMgr_fuchsia.cpp, which is > > used in chromium (e.g. [2]) which has its own GN rules for > > building Skia for Fuchsia [3]. Skia's ports.gni file still > > lists those files [4] so clients can use them in their own GN build. > > > > If this breaks things, it'll be illuminating and unexpected. > > > > [1] https://skia-review.googlesource.com/c/skia/+/244369 > > [2] https://source.chromium.org/chromium/chromium/src/+/main:skia/ext/fontmgr_fuchsia_unittest.cc;l=20;drc=f6492524f906d0728ddd5d41daf071f4b89300ba;bpv=1;bpt=1 > > [3] https://source.chromium.org/chromium/chromium/src/+/main:skia/BUILD.gn;l=469-484;bpv=1;bpt=0 > > [4] https://github.com/google/skia/blob/ef104904659ff58bca9e3332606009c26c7b5540/gn/ports.gni#L112-L116 > > > > Bug: b/427681157 > > Change-Id: I2f855c9c875a1844422178999958442be8dcb035 > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1017296 > > Commit-Queue: Kaylee Lubick <kjlubick@google.com> > > Auto-Submit: Kaylee Lubick <kjlubick@google.com> > > Reviewed-by: Thomas Smith <thomsmit@google.com> > > Bug: b/427681157 > Change-Id: If4caf816db8e69c71ebd362fedbe8e9a39ddae48 > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1018617 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Reviewed-by: Thomas Smith <thomsmit@google.com> Bug: b/427681157 Change-Id: I6485b6404f9e573ac87f5bbdb64cf2060ffebb9c No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1019617 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Thomas Smith <thomsmit@google.com> | 11 个月前 | |
Make gn directory importable in python This will make it possible to do >>> from gn import gn_to_bp instead of: >>> sys.path.append('gn') >>> import gn_to_bp in the new Android autoroller BUG=skia:5538 Change-Id: I4fe851247ab6dd5bc553b8315ad91281819fbc7e Reviewed-on: https://skia-review.googlesource.com/9897 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com> | 9 年前 | |
Treat fontations ffi.rs.h as explicit output Currently, the GN build sometimes fails if GN thought it did not have to rebuild libfontations_ffi.a but the bazel-bin folder did not have the header ffi.rs.h that was also required. Now the bazel_build.py script can copy multiple files and not just into the immediate GN folder. I also added documentation to that script to help rationalize the functionality. Change-Id: Iadf4475fa6ccdad80e08789e4276de045350a7f2 Bug: b/40045269 Cq-Include-Trybots: luci.skia.skia.primary:Build-Debian10-Clang-x86_64-Debug-Fontations,Build-Mac-Clang-x86_64-Debug-Fontations Reviewed-on: https://skia-review.googlesource.com/c/skia/+/738779 Reviewed-by: Dominik Röttsches <drott@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com> | 2 年前 | |
[webp] Use the same RP blender for all lines Since the pipeline stages are not changing, we can reuse the same SkRasterPipeline for all lines. Shows a modest perf improvement. Before: min median mean max stddev samples config bench 186µs 186µs 187µs 190µs 1% ▁▇▁▇▂▁█▁▁█ nonrendering webp_blend_bench After: min median mean max stddev samples config bench 173µs 174µs 176µs 185µs 2% ▁█▂▁▃▂▁▃▁▁ nonrendering webp_blend_bench Change-Id: I4af1796ba709cc7ca18405eaa38bc9760b439580 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/934277 Reviewed-by: Daniel Dilan <danieldilan@google.com> Commit-Queue: Florin Malita <fmalita@google.com> | 1 年前 | |
[minor] mark scripts as executable if a filename ends with .py and the file begins with '#!.*python.*', make it executable. Change-Id: I41de516ff37343d3b0979bde9fd61813aec7365c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/254439 Commit-Queue: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> Auto-Submit: Hal Canary <halcanary@google.com> Reviewed-by: Ben Wagner <bungeman@google.com> | 6 年前 | |
fix print functions for python3 gn scripts Change-Id: I478b10c23aae0c363b0d7342f25663ca8fc0d0fa Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274637 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Klein <mtklein@google.com> | 6 年前 | |
2026.03.30 arkweb_144代码蓝黄同步 Signed-off-by: jiujiaoxiaogula <sujiahao10@huawei.com> Change-Id: Id6b8f8e08796118f0ec7d7f82b8f2430ef3448e1 | 3 个月前 | |
Reland "Set GN's script_executable to "python3"" This is a reland of commit 6f0d2bf49458fb1b0fdc44e775e81c903da65b29 Includes a fix for iOS codesigning script to be python3 compatible. Original change's description: > Set GN's script_executable to "python3" > > In some environments, GN would end up running scripts with python2. > This change requires that python3 be in the path, but is the safest > way to guarantee that all scripts are run correctly. > > Change-Id: Ifc2601ff9c9f3705ac7f1057a26a87a61211ff67 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/528337 > Reviewed-by: Kevin Lubick <kjlubick@google.com> > Commit-Queue: Brian Osman <brianosman@google.com> Change-Id: I785909f3c7d51d6fd1c90114d01f114be76981f2 Cq-Include-Trybots: luci.skia.skia.primary:Build-Mac-Clang-arm64-Debug-iOS Reviewed-on: https://skia-review.googlesource.com/c/skia/+/528441 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> | 4 年前 | |
[python3] Reland recipes -> python3 changes This was enabled by moving the iPhones off of the old RPI2 hosts. This reverts commit 04cd6fba97bee0f9ebdaac5314e0f8898e7b13f2. This reverts commit a726978ae771ae7e93b03fb207712c703c9010c0. Bug: chromium:1256037 Change-Id: I35069089aa39baf62a18235c8d0514923f327c53 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/477987 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com> | 4 年前 | |
[sksl][compute] Enable SPIR-V compute test outputs * The compute tests now compile to SPIR-V, with a .asm.comp extension. Most of the output contains compilation errors which will be addressed in follow up CLs. * Support the GLCompute and LocalSize entry point decorations. The LocalSize is hardcoded to (16, 16, 1) until SkSL supports local size declarations. Bug: b/262428625 Change-Id: I08795badd75788e0724f03d77e1da537d07ceb51 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/729097 Reviewed-by: John Stiles <johnstiles@google.com> Commit-Queue: Arman Uguray <armansito@google.com> | 2 年前 | |
[python3] Reland recipes -> python3 changes This was enabled by moving the iPhones off of the old RPI2 hosts. This reverts commit 04cd6fba97bee0f9ebdaac5314e0f8898e7b13f2. This reverts commit a726978ae771ae7e93b03fb207712c703c9010c0. Bug: chromium:1256037 Change-Id: I35069089aa39baf62a18235c8d0514923f327c53 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/477987 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com> | 4 年前 | |
[rust png] Extract ReadAndSeek traits from rust/png to rust/common Extract the StreamAdapter (Read+Seek) classes from PNG-specific code to rust/common for reuse across multiple codec implementations (PNG, BMP, JPEG, etc.). Key changes: - Created rust/common/SkStreamAdapter.h with StreamAdapter classes that wrap SkStream for Rust FFI - Created rust/common/io_traits_ffi.rs as canonical CXX bridge declaration for type unification across codec modules - Updated rust/png/FFI.rs to use unified types from rust::stream namespace Bug: 452666425 Change-Id: I8e2ec205d485603816690c0e15600f0c9a4eb89f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1093916 Reviewed-by: Łukasz Anforowicz <lukasza@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Łukasz Anforowicz <lukasza@google.com> | 7 个月前 | |
Work around python 2 shutil.copy2 date bug ninja 1.9.0+ exposes a bug in python shutil.copy2 where a copied file's date is the src's date but rounded. This can cause ninja to think the copied file is slightly older than the src and trigger unnecessary work when building. https://github.com/ninja-build/ninja/issues/1554 Change-Id: I53247453f38a1c98c5d832bdda0f68effbe786c2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/320261 Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com> | 5 年前 | |
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 年前 | |
[graphite] Make MatrixConvolutionImageFilter stably keyed (take 2) Bug: b/238759147 Change-Id: Ie5d8047a292d05311acae31cea4221c9532100e0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/853096 Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> | 2 年前 | |
Give a human-intelligible message if gn generates a warning If gn generates an otherwise harmless warning, find_headers.py will fail to parse the gn-generated JSON because gn prints both the warning and the JSON to stdout. Though find_headers.py prints the output of gn if JSON parsing fails, a human investigating this failure is likely to assume that the warning they see is from the build system, not from find_headers.py, and is not related to the failure. For example, this warning about an argument which has no effect breaks the build: WARNING at build arg file (use "gn args <out_dir>" to edit):36:36: Build argument has no effect. skia_skqp_global_error_tolerance = 8 ^ [...] Change-Id: Iafa7252ba161e4def1438f5d9480b64fdaa887d2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/510536 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> | 4 年前 | |
[gn] Remove msvc env setting SetEnv.cmd has long been removed but kept alive by Skia packaging of MSVC. Older MSVC compilers and Windows SDKs relied on a large number of environment variables to be set when cross compiling, but this should no longer be necessary. In addition, the MSVC ARM toolchains are now complete and no longer require the host toolchain to be on the path. These improvements mean that it is no longer necessary to use cmd.exe /c as a "compiler wrapper" to set up the environment. Because it is not generally possible to properly escape such command lines this has caused issues with any arguments which would require proper escaping such as paths with spaces and special characters. This change drops support for building Skia for x86 or ARM64 with MSVC from before VS2022, with the benefit of building for x86 or ARM64 without extemely odd, undocumented, and limiting restrictions. Also add support for detecting MSVC 2022 before looking for 2019 or 2017. Bug: 420271794, 376381591 Change-Id: I8591636aec31095b13db6dc1a376abfa23199ebd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/999356 Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 1 年前 | |
[python3] Reland recipes -> python3 changes This was enabled by moving the iPhones off of the old RPI2 hosts. This reverts commit 04cd6fba97bee0f9ebdaac5314e0f8898e7b13f2. This reverts commit a726978ae771ae7e93b03fb207712c703c9010c0. Bug: chromium:1256037 Change-Id: I35069089aa39baf62a18235c8d0514923f327c53 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/477987 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com> | 4 年前 | |
Add gni filegroups for chromium fuzzers The plan is to update the hard-coded file lists in [1] with these Skia-defined lists so we can update them easier. [1] https://source.chromium.org/chromium/chromium/src/+/main:skia/tools/fuzzers/BUILD.gn;l=42;drc=6a01934281fe731260cfdbca2de520f8b2273ea4 Bug: b/305780908 Change-Id: Ic70eb039f6f89210248d24e103cc9b84197fc109 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/778716 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> | 2 年前 | |
More tools/ gms/ changes in anticipation of Graphite-only viewer build. Change-Id: Iaf2284250ea5e0caf4c431c393073172ac788f29 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1056216 Commit-Queue: Brian Salomon <briansalomon@gmail.com> Reviewed-by: Thomas Smith <thomsmit@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> | 9 个月前 | |
Update docs to reference python3 Also updates a script with 2to3 to be python3-compatible. Change-Id: I45981349664aef4491e7ab20e49889d461f8bf3f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/538417 Commit-Queue: James Godfrey-Kittle <jamesgk@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> | 4 年前 | |
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 个月前 | |
Build the NDK compliant shared library to be used by JNI in SkQP. This CL creates a new build target with a customized set of GN flags that enable the native components of SkQP to be built using NDK APIs. Bug: skia:12777 Change-Id: I8341eafa7fc794cfb759045b7c7238e69c29b0a3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/491447 Reviewed-by: Leon Scroggins <scroggo@google.com> Commit-Queue: Derek Sollenberger <djsollen@google.com> | 4 年前 | |
[cmake] Enable OBJC and OBJCXX CMake does not enable these languages automatically. If they are not enabled then generator expressions which would match those languages will be ignored. Bug: 348660666 Change-Id: Ia30a742871788eb6beeb66fbb036f460e3ed4a0b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/870976 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> | 1 年前 | |
Revert "DM builds with graphite enabled and ganesh disabled" This reverts commit 54f6a0b55a312ce77e90ff9e7e474398bf4272be. Reason for revert: Blocking Chrome roller (unknown symbol skia_gpu_vk_chromium_public) Original change's description: > DM builds with graphite enabled and ganesh disabled > > Change-Id: I98f4f67150d79a96f3de6851c288f8176a218692 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1075538 > Reviewed-by: Greg Daniel <egdaniel@google.com> > Reviewed-by: Michael Ludwig <michaelludwig@google.com> > Commit-Queue: Brian Salomon <briansalomon@gmail.com> Change-Id: Iafd4115700d8d86eee517827cbf5443c27e8fd38 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1084017 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Jorge Betancourt <jmbetancourt@google.com> | 8 个月前 | |
[graphite] Make Key Serialization no longer be Precompile only With the upcoming label tracking callback CL we will need to be able to create serialized keys even when Precompilation isn't being used. Bug: b/455580777 Change-Id: I85c7a1bb09a6eb8088ba6fa09fcfaf4c15fc00c1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1085357 Reviewed-by: Thomas Smith <thomsmit@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com> | 8 个月前 | |
fix print functions for python3 gn scripts Change-Id: I478b10c23aae0c363b0d7342f25663ca8fc0d0fa Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274637 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Klein <mtklein@google.com> | 6 年前 | |
[infra] Fixes to support iOS devices in new lab - Use "device" Swarming dimension. This is now provided by machines in our local lab. - Launch the app via xcode rather than idevicedebug for devices in the new lab. - Use idevicefs instead of ifuse for machines in the new lab. Bug: b/396105360 Change-Id: I5f245f9995c34e2f5041d647ac8e25b49cc709ad Reviewed-on: https://skia-review.googlesource.com/c/skia/+/951914 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 1 年前 | |
Fix is_clang python3 encoding errors python 2.7 subprocess.check_output: "Run command with arguments and return its output as a byte string." python 3 ubprocess.check_output: "By default, this function will return the data as encoded bytes." Using a 'b' prefix seems to cover both cases. Change-Id: Ie2d90bc93d0e98a723a262fae63fe538137812c5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299382 Commit-Queue: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@google.com> Reviewed-by: Mike Klein <mtklein@google.com> | 5 年前 | |
GMs: re-sort gni, script to regenerate gni, file name convention Change-Id: I4b61c2a66f8d6b2d8c4a5ffd8c373b643415c70c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/211598 Commit-Queue: Hal Canary <halcanary@google.com> Reviewed-by: Ben Wagner <bungeman@google.com> | 7 年前 | |
[graphite] Remove placeholder ES2 shader modules Change-Id: I2ec9b27450e1c42054d38c9167a4ab81e5f3c62b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/996117 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Commit-Queue: James Godfrey-Kittle <jamesgk@google.com> | 1 年前 | |
Add support for golden tests of private Runtime Shaders. Since private Runtime Shaders have different rules than public ones, we should have tests which verify that those rules are upheld. At present, the only significant difference is the availability of sk_FragCoord, but this may change if we support new features like passing child-effects to a function. Bug: b/328051548 Change-Id: I07fa24d8229b1876161b7ddf1c15f7748fe00342 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/857540 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 2 年前 | |
[Loong64] Added binary compatibility support. Currently, the lasx optimization branch is used by default on the loong64 platform. Some loongarch processors(such as 2k2000) that do not support lasx instruction sets will crashed. Change-Id: I6a86e1916f1093818f117de39ada5c3be57c4d4a Reviewed-on: https://skia-review.googlesource.com/c/skia/+/873776 Reviewed-by: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@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> | 11 个月前 | |
Reland "Decouple PDF backend and JPEG encoding/decoding" This reverts commit aad4ee35f544182797ae6c75f78d1f5d180e5402. Landed upstream fixes - http://cl/728626566 - http://cl/728631058 - https://crrev.com/c/6282389 Original change's description: > Revert "Decouple PDF backend and JPEG encoding/decoding" > > This reverts commit 23f0bf51c6adcd1073ac4acfe3112a7952689d0d. > > Reason for revert: Breaking Chromium roll because _DECODES_JPEG and _ENCODES_JPEG are not set > > Original change's description: > > Decouple PDF backend and JPEG encoding/decoding > > > > Some clients may want to supply their own JPEG encoder/decoder or, > > if they have very specific use cases, use no JPEG encoding whatsoever. > > > > This CL allows that to happen, by having clients be able to provide > > callbacks in SkPDF::Metadata. For convenience, we provide > > SkPDFJpegHelpers.h to wrap Skia's encoder and decoder to be passed in. > > > > For now, if those callbacks are not supplied, we can provide them, > > but eventually that will go away. > > > > Change-Id: I5adeffda8e12b4db085b20814fc92436689ccdbb > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/929456 > > Commit-Queue: Kaylee Lubick <kjlubick@google.com> > > Reviewed-by: Ben Wagner <bungeman@google.com> > > Change-Id: I77cf0bd510e37271fa0cfe1c0c30254c3ca31a19 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/953376 > Owners-Override: Kaylee Lubick <kjlubick@google.com> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Change-Id: I105c8e00eb5a82d19113891bf80f79d6fd6efe5b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/953578 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <bungeman@google.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> | 1 年前 | |
Make SkFontMgr_AndroidNDK find names in fonts.xml Change-Id: Ic956029821d2abbb9c6c74b46682380e54e9320f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1028699 Reviewed-by: Julia Lavrova <jlavrova@google.com> Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> | 10 个月前 | |
skia_android_serial = "auto" Handy if you only have one device plugged in at a time. Note that when used, this makes GN treat all devices' serial numbers as "auto", so it won't re-push if you swap devices until you change the binary that's being pushed. Change-Id: I1f72133d6d8c4b5569734f191d414088e0b41f06 Reviewed-on: https://skia-review.googlesource.com/6948 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Derek Sollenberger <djsollen@google.com> | 9 年前 | |
Delete .a file before writing static library. The ar utility only adds symbols. As a result removed symbols are sometimes still present in later builds, leading to great confusion. Delete the .a file before recreating it with ar. This is similar to what Chromium currently does in gcc_toolchain.gni. However, we cannot always just use 'rm' because of the build for Android on Windows, so this introduces 'rm.py' which is just like 'cp.py' but without the copy part. Bug: skia:10363 Change-Id: Icc0c3d18dab1e48ccfec47386662c7b4d2dc8811 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/295569 Commit-Queue: Ben Wagner <bungeman@google.com> Reviewed-by: Mike Klein <mtklein@google.com> | 5 年前 | |
Reland "Fetch clang-format automatically when compiling .fp files." This is a reland of a25422faa462e8cfb00af1a3818547abbbb9c7f9 Original change's description: > Fetch clang-format automatically when compiling .fp files. > > On a freshly fetched repo, setting skia_compile_processors = true will > fail to compile because clang-format is missing from the bin directory. > This CL automatically runs fetch-clang-format for you when clang-format > is absent. > > Change-Id: Ieeb359176072e92ca235316c820310333732f608 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/295780 > Reviewed-by: Mike Klein <mtklein@google.com> > Commit-Queue: John Stiles <johnstiles@google.com> Change-Id: Ib397716771f742f192ebd6ed7ec0a9915b2400c5 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/295956 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> | 5 年前 | |
[rust png] Extract ReadAndSeek traits from rust/png to rust/common Extract the StreamAdapter (Read+Seek) classes from PNG-specific code to rust/common for reuse across multiple codec implementations (PNG, BMP, JPEG, etc.). Key changes: - Created rust/common/SkStreamAdapter.h with StreamAdapter classes that wrap SkStream for Rust FFI - Created rust/common/io_traits_ffi.rs as canonical CXX bridge declaration for type unification across codec modules - Updated rust/png/FFI.rs to use unified types from rust::stream namespace Bug: 452666425 Change-Id: I8e2ec205d485603816690c0e15600f0c9a4eb89f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1093916 Reviewed-by: Łukasz Anforowicz <lukasza@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Łukasz Anforowicz <lukasza@google.com> | 7 个月前 | |
[rust png] Create skeleton for rust common gn integration in Chromium This CL is creating defining the required gni vars for the integration in Chromium. This CL is intended to be landed in conjunction to this CL on Chromium repository: https://chromium-review.googlesource.com/c/chromium/src/+/7130444 Both are a pre-requisite for landing the CL refactoring rust png to extract ReadAndSeek traits to rust/common: https://skia-review.googlesource.com/c/skia/+/1093916?tab=comments Bug: 452666425 Change-Id: If0321e0f510b0717025e1044bf5616def018aaf1 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1095596 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Łukasz Anforowicz <lukasza@google.com> | 7 个月前 | |
Add ability to override fuzzing engine OSS-fuzz compiles its fuzz engine from source and then links it in manually, so does *not* want us to set -fsanitize=fuzzer or we'll have conflicting symbols when they build and link centipede. We want the default to be true because Skia engineers want an "out of the box" solution. Step #21 - "compile-centipede-none-x86_64": clang++ -fsanitize=fuzzer -rdynamic -Wl,-rpath,\$ORIGIN -Wl,--gc-sections /usr/lib/libFuzzingEngine.a -O1 -fno-omit-frame-pointer -gline-tables-only -Wno-error=incompatible-function-pointer-types -Wno-error=int-conversion -Wno-error=deprecated-declarations -Wno-error=implicit-function-declaration -Wno-error=implicit-int -Wno-error=unknown-warning-option -Wno-error=vla-cxx-extension -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fno-builtin -fsanitize-coverage=trace-pc-guard,pc-table,trace-cmp -O2 -gline-tables-only -Wno-unused-command-line-argument -Wl,-ldl -Wl,-lrt -Wl,-lpthread -Wl,/src/fuzztest/centipede/weak.o -stdlib=libc++ -Wno-zero-as-null-pointer-constant -Wno-unused-template -Wno-cast-qual -I/src/skia/third_party/externals/swiftshader/include -fno-sanitize=vptr -DSK_BUILD_FOR_LIBFUZZER -D SK_BUILD_FOR_FUZZER -L/workspace/out/centipede-none-x86_64 -fuse-ld=lld -Wl,--start-group @./animated_image_decode.rsp -Wl,--end-group -lpthread -ldl -o ./animated_image_decode Step #21 - "compile-centipede-none-x86_64": ld.lld: error: duplicate symbol: memcmp Step #21 - "compile-centipede-none-x86_64": >>> defined at FuzzerInterceptors.cpp:160 (/src/llvm-project/compiler-rt/lib/fuzzer/FuzzerInterceptors.cpp:160) Step #21 - "compile-centipede-none-x86_64": >>> fuzzer_interceptors.o:(memcmp) in archive /usr/local/lib/clang/22/lib/x86_64-unknown-linux-gnu/libclang_rt.fuzzer_interceptors.a Step #21 - "compile-centipede-none-x86_64": >>> defined at runner_interceptors.cc Step #21 - "compile-centipede-none-x86_64": >>> runner_interceptors.pic.o:(.text.memcmp+0x0) in archive /usr/lib/libFuzzingEngine.a Step #21 - "compile-centipede-none-x86_64": Change-Id: Iec3c978ca61facdac576f3ac6bc11518676d040f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1092737 Reviewed-by: Eric Boren <borenet@google.com> Auto-Submit: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Eric Boren <borenet@google.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com> | 7 个月前 | |
Remove SkHeifCodec and related defines Android framework was the only client which made use if this and it no longer uses this for HEIC decoding (it goes via crabbyavif now). So remove the SkHeifCodec wrapper and related defines. Bug: 442820580 Change-Id: Ifa5cd4bd21d0d5de59815172fb54231a537741bf Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1050039 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Owners-Override: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Vignesh Venkat <vigneshv@google.com> | 9 个月前 | |
[vulkan] Use std::vector<uint32_t> for SPIR-V Instead of std::string. SPIR-V is defined in 32-bit words and includes lots of zeros. Use of std::string (due to other backends) is awkward. This change makes the "native shader" be a type that can either contain text or binary. As a bonus, it makes it less error-prone to mix up shader input and output std::strings. As a small drive-by, the skia_print_native_shaders feature now works with SPIR-V as well. Bug: b/390458117 Change-Id: I65326b043e9634651071b66403906a2295fbaacc Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1006716 Commit-Queue: Shahbaz Youssefi <syoussefi@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> | 1 年前 | |
Fix the syntax error of mixed semicolons and commas in interfaceBlock. Bug: b/0 Change-Id: I0d999371d965d25ae231734fcb9cd2a8b5ee9ae0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/996096 Commit-Queue: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 1 年前 | |
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 个月前 | |
2026.03.30 arkweb_144代码蓝黄同步 Signed-off-by: jiujiaoxiaogula <sujiahao10@huawei.com> Change-Id: Id6b8f8e08796118f0ec7d7f82b8f2430ef3448e1 | 3 个月前 | |
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> | 11 个月前 | |
Add SkShapers::Factory as way to configure shaper behavior in svg/skottie We don't want our modules to use #ifdefs to figure out what shaping logic to use because that would require skottie to depend directly on the skshaper/skunicode implementation that it needs and we really want those things to be independent. This adds a mechanism for a client to depend on skottie and whatever version of skshaper (and optionally skunicode) that they want (or even inject their own SkShaper implementation if they are feeling ambitious) and then tell skottie to use said code. The same idea holds for SVG. SkParagraph was not touched here because it always needs skshaper and some skunicode (although follow-up CLs will need to allow configuration of skunicode). This adds some backwards compatible behavior (can be disabled by defining SK_DISABLE_LEGACY_SHAPER_FACTORY for skottie. None of our internal clients seem to be using SVGs for text stuff, so it does not appear necessary to make SVG code backwards compatible. Interested parties can look at earlier patchsets to see other implementations/approaches taken. We decided *not* to use a global SkShapers::Factory variable because although that would have been easier it was deemed a bad idea if client A depends on client B, both of whom depend on skottie and want to use different shaping behavior. While this may seem unlikely, it can happen in very large build graphs. See also: http://cl/621201820 Change-Id: I323a4fca7f08e2f867e059e3a4c8baef4790a756 Bug: b/40045064 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/826067 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com> | 2 年前 | |
Reland "[rust png] Stop using+remove SkXPS...PngHelpers.h from Skia." This CL replaces usage of the helpers with a lambda provided by the callers (e.g. by dm/DMSrcSink.cpp). This CL also removes the helpers altogether, because Chromium and PDFium have already stopped using the helpers in https://crrev.com/c/7090470 and https://pdfium-review.googlesource.com/137410 (the latter rolled into Chromium in https://crrev.com/c/7097284 ). This removal seems desirable based on the following: 1. No need for separate headers. There is no need today for SVG- or XPS-specific encoding options. If such a need arises in the future, then for backcompatibility we would still need to keep the current helpers. And since current helpers are SVG- and XPS-agnostic, then it seems okay to just rely directly on the API in include/encode/...PngEncoder.h headers. 2. No need for separate build components. The separate build components (e.g. skia_xps_libpng_public in gn/xps.gni, xps_libpng_helpers in BUILD.gn, xps_libpng_hdrs in include/docs/BUILD.bazel) were tightly coupled with the PNG encoder implementations. 3. Simplification. After this CL there is less build components to manage: - In theory skia_xps_libpng_public from gn/xps.gni would have to be used in skia/BUILD.gn in Chromium and PDFium. - In theory, in the future we would also have to have even more components - e.g. skia_svg_libpng_public that would provide dependency injection for SkSVGCanvas::Make. 4. Additional concerns (e.g. reducing the risk of breaking Skia clients in the future) have been discussed in more detail during the code review - see https://skia-review.googlesource.com/c/skia/+/1082338/5#message-e592667c4a466ab0b2f52551f07760e9d2f9d960 Bug: chromium:381900683 Change-Id: Idddfeeef20699685a7fe2df6c888c6e306a885d2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1088236 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Ben Wagner <bungeman@google.com> Auto-Submit: Łukasz Anforowicz <lukasza@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 8 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 5 年前 | ||
| 8 个月前 | ||
| 11 个月前 | ||
| 1 年前 | ||
| 11 个月前 | ||
| 9 年前 | ||
| 2 年前 | ||
| 1 年前 | ||
| 6 年前 | ||
| 6 年前 | ||
| 3 个月前 | ||
| 4 年前 | ||
| 4 年前 | ||
| 2 年前 | ||
| 4 年前 | ||
| 7 个月前 | ||
| 5 年前 | ||
| 1 年前 | ||
| 2 年前 | ||
| 4 年前 | ||
| 1 年前 | ||
| 4 年前 | ||
| 2 年前 | ||
| 9 个月前 | ||
| 4 年前 | ||
| 7 个月前 | ||
| 4 年前 | ||
| 1 年前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 6 年前 | ||
| 1 年前 | ||
| 5 年前 | ||
| 7 年前 | ||
| 1 年前 | ||
| 2 年前 | ||
| 1 年前 | ||
| 11 个月前 | ||
| 1 年前 | ||
| 10 个月前 | ||
| 9 年前 | ||
| 5 年前 | ||
| 5 年前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 9 个月前 | ||
| 1 年前 | ||
| 1 年前 | ||
| 7 个月前 | ||
| 3 个月前 | ||
| 11 个月前 | ||
| 2 年前 | ||
| 8 个月前 |