diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/codec/SkJpegCodec.cpp | 13 | ||||
-rw-r--r-- | src/codec/SkJpegPriv.h | 36 | ||||
-rw-r--r-- | src/codec/SkJpegUtility.h | 8 | ||||
-rw-r--r-- | src/images/SkJPEGImageEncoder.cpp | 19 | ||||
-rw-r--r-- | src/images/SkJPEGWriteUtility.h | 9 | ||||
-rw-r--r-- | src/images/SkPNGImageEncoder.cpp | 20 | ||||
-rw-r--r-- | src/images/SkWEBPImageEncoder.cpp | 2 | ||||
-rw-r--r-- | src/images/transform_scanline.h (renamed from src/images/SkImageEncoderFns.h) | 20 |
8 files changed, 41 insertions, 86 deletions
diff --git a/src/codec/SkJpegCodec.cpp b/src/codec/SkJpegCodec.cpp index 624bc25845..55ee3b5dbc 100644 --- a/src/codec/SkJpegCodec.cpp +++ b/src/codec/SkJpegCodec.cpp @@ -44,7 +44,9 @@ static uint32_t get_endian_int(const uint8_t* data, bool littleEndian) { } const uint32_t kExifHeaderSize = 14; +const uint32_t kICCHeaderSize = 14; const uint32_t kExifMarker = JPEG_APP0 + 1; +const uint32_t kICCMarker = JPEG_APP0 + 2; static bool is_orientation_marker(jpeg_marker_struct* marker, SkCodec::Origin* orientation) { if (kExifMarker != marker->marker || marker->data_length < kExifHeaderSize) { @@ -110,10 +112,11 @@ static SkCodec::Origin get_exif_orientation(jpeg_decompress_struct* dinfo) { } static bool is_icc_marker(jpeg_marker_struct* marker) { - if (kICCMarker != marker->marker || marker->data_length < kICCMarkerHeaderSize) { + if (kICCMarker != marker->marker || marker->data_length < kICCHeaderSize) { return false; } + static const uint8_t kICCSig[] { 'I', 'C', 'C', '_', 'P', 'R', 'O', 'F', 'I', 'L', 'E', '\0' }; return !memcmp(marker->data, kICCSig, sizeof(kICCSig)); } @@ -157,8 +160,8 @@ static sk_sp<SkData> get_icc_profile(jpeg_decompress_struct* dinfo) { return nullptr; } markerSequence[markerIndex] = marker; - SkASSERT(marker->data_length >= kICCMarkerHeaderSize); - totalBytes += marker->data_length - kICCMarkerHeaderSize; + SkASSERT(marker->data_length >= kICCHeaderSize); + totalBytes += marker->data_length - kICCHeaderSize; } } @@ -177,8 +180,8 @@ static sk_sp<SkData> get_icc_profile(jpeg_decompress_struct* dinfo) { return nullptr; } - void* src = SkTAddOffset<void>(marker->data, kICCMarkerHeaderSize); - size_t bytes = marker->data_length - kICCMarkerHeaderSize; + void* src = SkTAddOffset<void>(marker->data, kICCHeaderSize); + size_t bytes = marker->data_length - kICCHeaderSize; memcpy(dst, src, bytes); dst = SkTAddOffset<void>(dst, bytes); } diff --git a/src/codec/SkJpegPriv.h b/src/codec/SkJpegPriv.h deleted file mode 100644 index e4e5b12763..0000000000 --- a/src/codec/SkJpegPriv.h +++ /dev/null @@ -1,36 +0,0 @@ -/* - * Copyright 2017 Google Inc. - * - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - - -#ifndef SkJpegPriv_DEFINED -#define SkJpegPriv_DEFINED - -#include "SkStream.h" - -#include <setjmp.h> -// stdio is needed for jpeglib -#include <stdio.h> - -extern "C" { - #include "jpeglib.h" - #include "jerror.h" -} - -static constexpr uint32_t kICCMarker = JPEG_APP0 + 2; -static constexpr uint32_t kICCMarkerHeaderSize = 14; -static constexpr uint8_t kICCSig[] = { - 'I', 'C', 'C', '_', 'P', 'R', 'O', 'F', 'I', 'L', 'E', '\0', -}; - -/* - * Error handling struct - */ -struct skjpeg_error_mgr : jpeg_error_mgr { - jmp_buf fJmpBuf; -}; - -#endif diff --git a/src/codec/SkJpegUtility.h b/src/codec/SkJpegUtility.h index 33f4fbda8b..43391017b5 100644 --- a/src/codec/SkJpegUtility.h +++ b/src/codec/SkJpegUtility.h @@ -9,7 +9,6 @@ #ifndef SkJpegUtility_codec_DEFINED #define SkJpegUtility_codec_DEFINED -#include "SkJpegPriv.h" #include "SkStream.h" #include <setjmp.h> @@ -22,6 +21,13 @@ extern "C" { } /* + * Error handling struct + */ +struct skjpeg_error_mgr : jpeg_error_mgr { + jmp_buf fJmpBuf; +}; + +/* * Error handling function */ void skjpeg_err_exit(j_common_ptr cinfo); diff --git a/src/images/SkJPEGImageEncoder.cpp b/src/images/SkJPEGImageEncoder.cpp index 014a0ae160..1f8f0d6394 100644 --- a/src/images/SkJPEGImageEncoder.cpp +++ b/src/images/SkJPEGImageEncoder.cpp @@ -10,10 +10,10 @@ #ifdef SK_HAS_JPEG_LIBRARY #include "SkColorPriv.h" -#include "SkImageEncoderFns.h" #include "SkJPEGWriteUtility.h" #include "SkStream.h" #include "SkTemplates.h" +#include "transform_scanline.h" #include <stdio.h> @@ -141,23 +141,6 @@ bool SkEncodeImageAsJPEG(SkWStream* stream, const SkPixmap& pixmap, int quality) jpeg_start_compress(&cinfo, TRUE); - if (pixmap.colorSpace()) { - sk_sp<SkData> icc = icc_from_color_space(*pixmap.colorSpace()); - if (icc) { - // Create a contiguous block of memory with the icc signature followed by the profile. - sk_sp<SkData> markerData = - SkData::MakeUninitialized(kICCMarkerHeaderSize + icc->size()); - uint8_t* ptr = (uint8_t*) markerData->writable_data(); - memcpy(ptr, kICCSig, sizeof(kICCSig)); - ptr += sizeof(kICCSig); - *ptr++ = 1; // This is the first marker. - *ptr++ = 1; // Out of one total markers. - memcpy(ptr, icc->data(), icc->size()); - - jpeg_write_marker(&cinfo, kICCMarker, markerData->bytes(), markerData->size()); - } - } - if (proc) { storage.reset(numComponents * pixmap.width()); } diff --git a/src/images/SkJPEGWriteUtility.h b/src/images/SkJPEGWriteUtility.h index 3765e7e674..91d07a3616 100644 --- a/src/images/SkJPEGWriteUtility.h +++ b/src/images/SkJPEGWriteUtility.h @@ -9,7 +9,6 @@ #ifndef SkJpegUtility_DEFINED #define SkJpegUtility_DEFINED -#include "SkJpegPriv.h" #include "SkStream.h" extern "C" { @@ -19,6 +18,14 @@ extern "C" { #include <setjmp.h> +/* Our error-handling struct. + * +*/ +struct skjpeg_error_mgr : jpeg_error_mgr { + jmp_buf fJmpBuf; +}; + + void skjpeg_error_exit(j_common_ptr cinfo); ///////////////////////////////////////////////////////////////////////////// diff --git a/src/images/SkPNGImageEncoder.cpp b/src/images/SkPNGImageEncoder.cpp index e28ae12dc0..2eac91d0ca 100644 --- a/src/images/SkPNGImageEncoder.cpp +++ b/src/images/SkPNGImageEncoder.cpp @@ -12,13 +12,14 @@ #include "SkColor.h" #include "SkColorPriv.h" #include "SkDither.h" -#include "SkImageEncoderFns.h" +#include "SkICC.h" #include "SkMath.h" #include "SkStream.h" #include "SkString.h" #include "SkTemplates.h" #include "SkUnPreMultiply.h" #include "SkUtils.h" +#include "transform_scanline.h" #include "png.h" @@ -39,7 +40,9 @@ static void sk_write_fn(png_structp png_ptr, png_bytep data, png_size_t len) { } } -static void set_icc(png_structp png_ptr, png_infop info_ptr, sk_sp<SkData> icc) { +static void set_icc(png_structp png_ptr, png_infop info_ptr, const SkColorSpaceTransferFn& fn, + const SkMatrix44& toXYZD50) { + sk_sp<SkData> icc = SkICC::WriteToICC(fn, toXYZD50); #if PNG_LIBPNG_VER_MAJOR > 1 || (PNG_LIBPNG_VER_MAJOR == 1 && PNG_LIBPNG_VER_MINOR >= 5) const char* name = "Skia"; png_const_bytep iccPtr = icc->bytes(); @@ -348,14 +351,17 @@ static bool do_encode(SkWStream* stream, const SkPixmap& pixmap, } if (pixmap.colorSpace()) { + SkColorSpaceTransferFn fn; + SkMatrix44 toXYZD50(SkMatrix44::kUninitialized_Constructor); if (pixmap.colorSpace()->isSRGB()) { png_set_sRGB(png_ptr, info_ptr, PNG_sRGB_INTENT_PERCEPTUAL); - } else { - sk_sp<SkData> icc = icc_from_color_space(*pixmap.colorSpace()); - if (icc) { - set_icc(png_ptr, info_ptr, std::move(icc)); - } + } else if (pixmap.colorSpace()->isNumericalTransferFn(&fn) && + pixmap.colorSpace()->toXYZD50(&toXYZD50)) + { + set_icc(png_ptr, info_ptr, fn, toXYZD50); } + + // TODO: Should we support writing ICC profiles for additional color spaces? } png_set_sBIT(png_ptr, info_ptr, &sig_bit); diff --git a/src/images/SkWEBPImageEncoder.cpp b/src/images/SkWEBPImageEncoder.cpp index a9fcc31265..8797ff5925 100644 --- a/src/images/SkWEBPImageEncoder.cpp +++ b/src/images/SkWEBPImageEncoder.cpp @@ -20,11 +20,11 @@ #include "SkBitmap.h" #include "SkColorPriv.h" -#include "SkImageEncoderFns.h" #include "SkStream.h" #include "SkTemplates.h" #include "SkUnPreMultiply.h" #include "SkUtils.h" +#include "transform_scanline.h" // A WebP decoder only, on top of (subset of) libwebp // For more information on WebP image format, and libwebp library, see: diff --git a/src/images/SkImageEncoderFns.h b/src/images/transform_scanline.h index 5120570c48..3c754275e0 100644 --- a/src/images/SkImageEncoderFns.h +++ b/src/images/transform_scanline.h @@ -4,9 +4,8 @@ * Use of this source code is governed by a BSD-style license that can be * found in the LICENSE file. */ - -#ifndef SkImageEncoderFns_DEFINED -#define SkImageEncoderFns_DEFINED +#ifndef transform_scanline_DEFINED +#define transform_scanline_DEFINED /** * Functions to transform scanlines between packed-pixel formats. @@ -15,7 +14,6 @@ #include "SkBitmap.h" #include "SkColor.h" #include "SkColorPriv.h" -#include "SkICC.h" #include "SkPreConfig.h" #include "SkRasterPipeline.h" #include "SkUnPreMultiply.h" @@ -275,16 +273,4 @@ static inline void transform_scanline_F16_premul_to_8888(char* SK_RESTRICT dst, p.append(SkRasterPipeline::store_8888, (void**) &dst); p.run(0, width); } - -static inline sk_sp<SkData> icc_from_color_space(const SkColorSpace& cs) { - SkColorSpaceTransferFn fn; - SkMatrix44 toXYZD50(SkMatrix44::kUninitialized_Constructor); - if (cs.isNumericalTransferFn(&fn) && cs.toXYZD50(&toXYZD50)) { - return SkICC::WriteToICC(fn, toXYZD50); - } - - // TODO: Should we support writing ICC profiles for additional color spaces? - return nullptr; -} - -#endif // SkImageEncoderFns_DEFINED +#endif // transform_scanline_DEFINED |