| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
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 个月前 | |
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 个月前 | |
Add kR16_unorm_SkColorType * Adding support for this color type was requested by ccameron@ while looking into accelerated RGB to YUV conversion. * This CL adds kR16_unorm_SkColorType and subsequently updates all the relevant areas (Caps format tables, testing, raster pipeline ops, etc). Bug: b/452079352 Change-Id: I2629a89c498365a1d41a52e583c0043ac74ba80d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1075756 Reviewed-by: Florin Malita <fmalita@google.com> Commit-Queue: Nicolette Prevost <nicolettep@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> | 7 个月前 | |
skhdr::Metadata: Add CLLI and MDCV metadata and parsing Add the skhdr namespace and add the skhdr::Metadata structure, which includes all HDR metadata. Add the structures skhdr::ContentLightLevelInformation and skhdr::MasteringDisplayColorVolume, with code to parse and print, and tests for this. Of note is that this mirrors several gfx::HDRMetadata structures. The goal is to replace those structures with SkHdrMetadata structures. Bug: 376550658 Change-Id: I9ced6cbdd8d20e7370420f303cd023fc1e050341 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1055616 Reviewed-by: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Christopher Cameron <ccameron@google.com> Reviewed-by: Christopher Cameron <ccameron@google.com> | 9 个月前 | |
Reland "Add OWNERS to include (and include/private) to enforce API review" This is a reland of be0e255c3e0f62249a1ddd767136d40fa7bf5370 Original change's description: > Add OWNERS to include (and include/private) to enforce API review > > Change-Id: I3ae83fbdcb4b3b13150df8b6a6461c36cbeaa55e > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/445641 > Reviewed-by: Ravi Mistry <rmistry@google.com> > Reviewed-by: Kevin Lubick <kjlubick@google.com> > Commit-Queue: Brian Osman <brianosman@google.com> Change-Id: I090456a05128865bc134b9c4df4e5ffcb5d0e1dc Reviewed-on: https://skia-review.googlesource.com/c/skia/+/450479 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> | 4 年前 | |
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 个月前 | |
Add SubIFD to exif data when using WriteExif. Previously, WriteExif implementation stored all of the entries under the IFD0 tag. However, some entries should be stored under the ExifIFD (SubIFD) tag (the read implementation doesn't differentiate IFD0 and ExifIFD). This CL is a correction to a previous CL where I added the WriteExif method (http://review.skia.org/901166). The functionality of this CL isn't currently load bearing but it is a more complete/correct implementation of the function added earlier. This is necessary if someone wants to use the function to write an exif file from Metadata, and also offers a way to extend the implementation if the Metadata struct is extended or SkExif get more usage/support. The WriteExif function now writes the subIfd data to the stream if SubIFD entries exist. It is added before the data stream for values that exceed the 12 byte entry size (the size of a TIFF entry). Bug: 310510975 Change-Id: Ic606ccd75f8077cda1dccc9d18b87fe7d15a964e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/906557 Commit-Queue: Daniel Dilan <danieldilan@google.com> Reviewed-by: Florin Malita <fmalita@google.com> | 1 年前 | |
Reland: Decode and encode ISO 21496-1 JPEG images Decode and encode ISO 21496-1 JPEG images Add SkJpegMetadataDecoder::getISOGainmapMetadata, to extract the ISO 21496-1 metadata blob from a JPEG file. Update SkJpegMetadataDecoderImpl::findGainmapImage to search for ISO 21496-1 metadata in the base JPEG image. Update the extract_gainmap function to also search for ISO 21496-1 metadata. If that metadata is found, and if it indicates not to use the base image color space for applying the gainmap, then parse the gainmap image's ICC profile. Update lots of tests. This reverts commit 62f369c759947272dfdd2d8f060afadbcc361e79. Reason for revert: Did not break build Original change's description: > Revert "Decode and encode ISO 21496-1 JPEG images" > This reverts commit 977a43773f7ce1c6e69cf966deb534311b3a18b4. > Reason for revert: Forgot to add SkGainmapInfo.cpp to Chrome build. > Original change's description: > > Decode and encode ISO 21496-1 JPEG images Bug: b/338342146 Change-Id: I9637ec62a3ab7f01f26637ddf3bc0139e752431c Reviewed-on: https://skia-review.googlesource.com/c/skia/+/855896 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Christopher Cameron <ccameron@google.com> Commit-Queue: Christopher Cameron <ccameron@google.com> | 2 年前 | |
Move SkGainmapShader::Make overload definition out of header Follow up to I6044c27f0c5f7c794e47a5ce89510b4845aa9b87, attempting to fix error on Android roll: error: member access into incomplete type 'SkShader' Change-Id: I3b725cf5b9f39734f9803f919214d24c04a5356f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/986436 Auto-Submit: Noelle Scobie <nscobie@google.com> Reviewed-by: Christopher Cameron <ccameron@google.com> Commit-Queue: Noelle Scobie <nscobie@google.com> Commit-Queue: Christopher Cameron <ccameron@google.com> | 1 年前 | |
SkCodec/SkPngCodec/SkPngEncoder: Add HDR metadata support To SkEncodedInfo, add an skhdr::Metadata item. To SkCodec, add an accessor for the skhdr::Metadata in its SkEncodedInfo. In SkPngCompositeChunkReader, which currently reads the gainmap chunks, add support to read the MDCV and CLLI chunks. Store an skhdr::Metadata member in SkPngCompositeChunkReader and propagate it to the SkEncodedInfo of the SkPngCodec. In SkPngEncoderMgr, rename setV0Gainmap to setHdrMetadata, which will write the MDCV and CLLI chunks (in addition to the gainmap chunks). Slightly clean up this area. Do not add tests yet. They come with the Rust implementation. Bug: 376550658 Change-Id: Ibf90e237202b00d589e06fe05d25d59b99256ad0 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1055197 Reviewed-by: Christopher Cameron <ccameron@google.com> Commit-Queue: ccameron chromium <ccameron@chromium.org> Owners-Override: Kaylee Lubick <kjlubick@google.com> Commit-Queue: Christopher Cameron <ccameron@google.com> Reviewed-by: Kaylee Lubick <kjlubick@google.com> | 9 个月前 | |
Use STArray in skia/include Bug: skia:14148 Change-Id: I0f03fc93e2361d9c454aa0a90cd31a7a7a6c3066 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/667556 Commit-Queue: Herb Derby <herb@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> | 3 年前 | |
SkGainmapInfo: Post-launch cleanup Remove unused parameters from SkGainmapInfo. Leave the fType member present because it is used in the Android repository. Remove SkJpegGainmapEncoder::EncodeJpegR. Bug: None Change-Id: I5d08859dc1ce12cd11ac573be579427f6b357282 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/760336 Commit-Queue: ccameron chromium <ccameron@chromium.org> Reviewed-by: Brian Osman <brianosman@google.com> | 2 年前 | |
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 个月前 | |
Update (some) callers to follow more strict span rules - allows us to migrate to std::span, keeping SkSpan (for now) as an alias. - idea is to land call-site-cleanups separately, so that when we do try to actually throw the switch, that CL only contains the switch (build-flag) itself. Change-Id: I5f9aaafeb297b7adc1d953019fdac6ffda95fe03 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1097896 Commit-Queue: Mike Reed <mike@reedtribe.org> Reviewed-by: Ben Wagner <bungeman@google.com> Reviewed-by: Florin Malita <fmalita@google.com> | 7 个月前 | |
Eliminate SampleUsage::constructor. This seems to be a vestige from our .fp code generator. Change-Id: Iab7f4b11a467f11d6c0f6596995cc8d0c9a003c2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/636820 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: John Stiles <johnstiles@google.com> | 3 年前 | |
Replace SK_WARN_UNUSED_RESULT with C++17 [[nodiscard]]. We no longer need a custom macro for [[nodiscard]], since it was added as a standard attribute in C++17. The official attribute does have slightly different placement rules than the compiler-specific attribute; it is required to be leftmost. Change-Id: I6ff5eb77514b5d96f0272a2c6287f378cf8a97ad Reviewed-on: https://skia-review.googlesource.com/c/skia/+/724396 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: John Stiles <johnstiles@google.com> | 2 年前 | |
SkJpegMetadataDecoderImpl: Clean up Apple gainmap rendering Change SkJpegMetadataDecoderImpl::findGainmapImage to parse the Exif metadata of the base image in search of the Apple MakerNote that indicates the HDR headroom to be used for rendering the image. Only accept gainmap images if the Apple MakerNote was present in the base image Exif. There exists a class of Apple gainmap images that have a rotation baked into the base image but not the gainmap image (and therefore have the gainmap oriented incorrectly). These images appear not to have the Apple MakerNote Exif data. This will happen to fix the bug where these would be rendered with an inappropriately rotated gainmap (by not checking for the gainmap). Resurrect the deprecated SkGainmapInfo::Type enum, and use it to indicate if an image has an Apple gainmap. Add a new path in SkGainmapShader to support transcoding Apple gainmaps to Adobe gainmaps on the fly. Add tests for extracting the Apple MakerNote HDR headroom parameter. Add tests for newer iPhone images. Add tests for the new shader path. Bug: b/338342146 Bug: b/338299886 Change-Id: Ifbdcd4adabd470a30ddbf7cd740d75e97174bf6f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/849738 Commit-Queue: Christopher Cameron <ccameron@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: ccameron chromium <ccameron@chromium.org> Reviewed-by: Christopher Cameron <ccameron@google.com> | 2 年前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 7 个月前 | ||
| 8 个月前 | ||
| 7 个月前 | ||
| 9 个月前 | ||
| 4 年前 | ||
| 7 个月前 | ||
| 1 年前 | ||
| 2 年前 | ||
| 1 年前 | ||
| 9 个月前 | ||
| 3 年前 | ||
| 2 年前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 3 年前 | ||
| 2 年前 | ||
| 2 年前 |