msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2015 Google Inc. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license that can be |
| 5 | * found in the LICENSE file. |
| 6 | */ |
| 7 | |
| 8 | #include "SkCodec.h" |
mtklein | e721a8e | 2016-02-06 19:12:23 -0800 | [diff] [blame] | 9 | #include "SkMSAN.h" |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 10 | #include "SkJpegCodec.h" |
| 11 | #include "SkJpegDecoderMgr.h" |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 12 | #include "SkCodecPriv.h" |
| 13 | #include "SkColorPriv.h" |
| 14 | #include "SkStream.h" |
| 15 | #include "SkTemplates.h" |
| 16 | #include "SkTypes.h" |
| 17 | |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 18 | // stdio is needed for libjpeg-turbo |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 19 | #include <stdio.h> |
msarett | c1d0312 | 2016-03-25 08:58:55 -0700 | [diff] [blame] | 20 | #include "SkJpegUtility.h" |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 21 | |
mtklein | dc90b53 | 2016-07-28 14:45:28 -0700 | [diff] [blame] | 22 | // This warning triggers false postives way too often in here. |
| 23 | #if defined(__GNUC__) && !defined(__clang__) |
| 24 | #pragma GCC diagnostic ignored "-Wclobbered" |
| 25 | #endif |
| 26 | |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 27 | extern "C" { |
| 28 | #include "jerror.h" |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 29 | #include "jpeglib.h" |
| 30 | } |
| 31 | |
scroggo | db30be2 | 2015-12-08 18:54:13 -0800 | [diff] [blame] | 32 | bool SkJpegCodec::IsJpeg(const void* buffer, size_t bytesRead) { |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 33 | static const uint8_t jpegSig[] = { 0xFF, 0xD8, 0xFF }; |
scroggo | db30be2 | 2015-12-08 18:54:13 -0800 | [diff] [blame] | 34 | return bytesRead >= 3 && !memcmp(buffer, jpegSig, sizeof(jpegSig)); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 35 | } |
| 36 | |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 37 | static uint32_t get_endian_int(const uint8_t* data, bool littleEndian) { |
| 38 | if (littleEndian) { |
| 39 | return (data[3] << 24) | (data[2] << 16) | (data[1] << 8) | (data[0]); |
| 40 | } |
| 41 | |
| 42 | return (data[0] << 24) | (data[1] << 16) | (data[2] << 8) | (data[3]); |
| 43 | } |
| 44 | |
| 45 | const uint32_t kExifHeaderSize = 14; |
| 46 | const uint32_t kICCHeaderSize = 14; |
| 47 | const uint32_t kExifMarker = JPEG_APP0 + 1; |
| 48 | const uint32_t kICCMarker = JPEG_APP0 + 2; |
| 49 | |
| 50 | static bool is_orientation_marker(jpeg_marker_struct* marker, SkCodec::Origin* orientation) { |
| 51 | if (kExifMarker != marker->marker || marker->data_length < kExifHeaderSize) { |
| 52 | return false; |
| 53 | } |
| 54 | |
| 55 | const uint8_t* data = marker->data; |
| 56 | static const uint8_t kExifSig[] { 'E', 'x', 'i', 'f', '\0' }; |
| 57 | if (memcmp(data, kExifSig, sizeof(kExifSig))) { |
| 58 | return false; |
| 59 | } |
| 60 | |
| 61 | bool littleEndian; |
| 62 | if (!is_valid_endian_marker(data + 6, &littleEndian)) { |
| 63 | return false; |
| 64 | } |
| 65 | |
| 66 | // Get the offset from the start of the marker. |
| 67 | // Account for 'E', 'x', 'i', 'f', '\0', '<fill byte>'. |
| 68 | uint32_t offset = get_endian_int(data + 10, littleEndian); |
| 69 | offset += sizeof(kExifSig) + 1; |
| 70 | |
| 71 | // Require that the marker is at least large enough to contain the number of entries. |
| 72 | if (marker->data_length < offset + 2) { |
| 73 | return false; |
| 74 | } |
| 75 | uint32_t numEntries = get_endian_short(data + offset, littleEndian); |
| 76 | |
| 77 | // Tag (2 bytes), Datatype (2 bytes), Number of elements (4 bytes), Data (4 bytes) |
| 78 | const uint32_t kEntrySize = 12; |
| 79 | numEntries = SkTMin(numEntries, (marker->data_length - offset - 2) / kEntrySize); |
| 80 | |
| 81 | // Advance the data to the start of the entries. |
| 82 | data += offset + 2; |
| 83 | |
| 84 | const uint16_t kOriginTag = 0x112; |
| 85 | const uint16_t kOriginType = 3; |
| 86 | for (uint32_t i = 0; i < numEntries; i++, data += kEntrySize) { |
| 87 | uint16_t tag = get_endian_short(data, littleEndian); |
| 88 | uint16_t type = get_endian_short(data + 2, littleEndian); |
| 89 | uint32_t count = get_endian_int(data + 4, littleEndian); |
| 90 | if (kOriginTag == tag && kOriginType == type && 1 == count) { |
| 91 | uint16_t val = get_endian_short(data + 8, littleEndian); |
| 92 | if (0 < val && val <= SkCodec::kLast_Origin) { |
| 93 | *orientation = (SkCodec::Origin) val; |
| 94 | return true; |
| 95 | } |
| 96 | } |
| 97 | } |
| 98 | |
| 99 | return false; |
| 100 | } |
| 101 | |
| 102 | static SkCodec::Origin get_exif_orientation(jpeg_decompress_struct* dinfo) { |
| 103 | SkCodec::Origin orientation; |
| 104 | for (jpeg_marker_struct* marker = dinfo->marker_list; marker; marker = marker->next) { |
| 105 | if (is_orientation_marker(marker, &orientation)) { |
| 106 | return orientation; |
| 107 | } |
| 108 | } |
| 109 | |
| 110 | return SkCodec::kDefault_Origin; |
| 111 | } |
| 112 | |
| 113 | static bool is_icc_marker(jpeg_marker_struct* marker) { |
| 114 | if (kICCMarker != marker->marker || marker->data_length < kICCHeaderSize) { |
| 115 | return false; |
| 116 | } |
| 117 | |
| 118 | static const uint8_t kICCSig[] { 'I', 'C', 'C', '_', 'P', 'R', 'O', 'F', 'I', 'L', 'E', '\0' }; |
| 119 | return !memcmp(marker->data, kICCSig, sizeof(kICCSig)); |
| 120 | } |
| 121 | |
| 122 | /* |
| 123 | * ICC profiles may be stored using a sequence of multiple markers. We obtain the ICC profile |
| 124 | * in two steps: |
| 125 | * (1) Discover all ICC profile markers and verify that they are numbered properly. |
| 126 | * (2) Copy the data from each marker into a contiguous ICC profile. |
| 127 | */ |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 128 | static sk_sp<SkData> get_icc_profile(jpeg_decompress_struct* dinfo) { |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 129 | // Note that 256 will be enough storage space since each markerIndex is stored in 8-bits. |
| 130 | jpeg_marker_struct* markerSequence[256]; |
| 131 | memset(markerSequence, 0, sizeof(markerSequence)); |
| 132 | uint8_t numMarkers = 0; |
| 133 | size_t totalBytes = 0; |
| 134 | |
| 135 | // Discover any ICC markers and verify that they are numbered properly. |
| 136 | for (jpeg_marker_struct* marker = dinfo->marker_list; marker; marker = marker->next) { |
| 137 | if (is_icc_marker(marker)) { |
| 138 | // Verify that numMarkers is valid and consistent. |
| 139 | if (0 == numMarkers) { |
| 140 | numMarkers = marker->data[13]; |
| 141 | if (0 == numMarkers) { |
| 142 | SkCodecPrintf("ICC Profile Error: numMarkers must be greater than zero.\n"); |
| 143 | return nullptr; |
| 144 | } |
| 145 | } else if (numMarkers != marker->data[13]) { |
| 146 | SkCodecPrintf("ICC Profile Error: numMarkers must be consistent.\n"); |
| 147 | return nullptr; |
| 148 | } |
| 149 | |
| 150 | // Verify that the markerIndex is valid and unique. Note that zero is not |
| 151 | // a valid index. |
| 152 | uint8_t markerIndex = marker->data[12]; |
| 153 | if (markerIndex == 0 || markerIndex > numMarkers) { |
| 154 | SkCodecPrintf("ICC Profile Error: markerIndex is invalid.\n"); |
| 155 | return nullptr; |
| 156 | } |
| 157 | if (markerSequence[markerIndex]) { |
| 158 | SkCodecPrintf("ICC Profile Error: Duplicate value of markerIndex.\n"); |
| 159 | return nullptr; |
| 160 | } |
| 161 | markerSequence[markerIndex] = marker; |
| 162 | SkASSERT(marker->data_length >= kICCHeaderSize); |
| 163 | totalBytes += marker->data_length - kICCHeaderSize; |
| 164 | } |
| 165 | } |
| 166 | |
| 167 | if (0 == totalBytes) { |
| 168 | // No non-empty ICC profile markers were found. |
| 169 | return nullptr; |
| 170 | } |
| 171 | |
| 172 | // Combine the ICC marker data into a contiguous profile. |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 173 | sk_sp<SkData> iccData = SkData::MakeUninitialized(totalBytes); |
| 174 | void* dst = iccData->writable_data(); |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 175 | for (uint32_t i = 1; i <= numMarkers; i++) { |
| 176 | jpeg_marker_struct* marker = markerSequence[i]; |
| 177 | if (!marker) { |
| 178 | SkCodecPrintf("ICC Profile Error: Missing marker %d of %d.\n", i, numMarkers); |
| 179 | return nullptr; |
| 180 | } |
| 181 | |
| 182 | void* src = SkTAddOffset<void>(marker->data, kICCHeaderSize); |
| 183 | size_t bytes = marker->data_length - kICCHeaderSize; |
| 184 | memcpy(dst, src, bytes); |
| 185 | dst = SkTAddOffset<void>(dst, bytes); |
| 186 | } |
| 187 | |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 188 | return iccData; |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 189 | } |
| 190 | |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 191 | bool SkJpegCodec::ReadHeader(SkStream* stream, SkCodec** codecOut, |
| 192 | JpegDecoderMgr** decoderMgrOut) { |
| 193 | |
| 194 | // Create a JpegDecoderMgr to own all of the decompress information |
halcanary | 385fe4d | 2015-08-26 13:07:48 -0700 | [diff] [blame] | 195 | SkAutoTDelete<JpegDecoderMgr> decoderMgr(new JpegDecoderMgr(stream)); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 196 | |
| 197 | // libjpeg errors will be caught and reported here |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 198 | if (setjmp(decoderMgr->getJmpBuf())) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 199 | return decoderMgr->returnFalse("ReadHeader"); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 200 | } |
| 201 | |
| 202 | // Initialize the decompress info and the source manager |
| 203 | decoderMgr->init(); |
| 204 | |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 205 | // Instruct jpeg library to save the markers that we care about. Since |
| 206 | // the orientation and color profile will not change, we can skip this |
| 207 | // step on rewinds. |
| 208 | if (codecOut) { |
| 209 | jpeg_save_markers(decoderMgr->dinfo(), kExifMarker, 0xFFFF); |
| 210 | jpeg_save_markers(decoderMgr->dinfo(), kICCMarker, 0xFFFF); |
| 211 | } |
| 212 | |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 213 | // Read the jpeg header |
msarett | fbccb59 | 2015-09-01 06:43:41 -0700 | [diff] [blame] | 214 | if (JPEG_HEADER_OK != jpeg_read_header(decoderMgr->dinfo(), true)) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 215 | return decoderMgr->returnFalse("ReadHeader"); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 216 | } |
| 217 | |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 218 | if (codecOut) { |
msarett | c30c418 | 2016-04-20 11:53:35 -0700 | [diff] [blame] | 219 | // Get the encoded color type |
msarett | ac6c750 | 2016-04-25 09:30:24 -0700 | [diff] [blame] | 220 | SkEncodedInfo::Color color; |
| 221 | if (!decoderMgr->getEncodedColor(&color)) { |
msarett | c30c418 | 2016-04-20 11:53:35 -0700 | [diff] [blame] | 222 | return false; |
| 223 | } |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 224 | |
| 225 | // Create image info object and the codec |
msarett | c30c418 | 2016-04-20 11:53:35 -0700 | [diff] [blame] | 226 | SkEncodedInfo info = SkEncodedInfo::Make(color, SkEncodedInfo::kOpaque_Alpha, 8); |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 227 | |
| 228 | Origin orientation = get_exif_orientation(decoderMgr->dinfo()); |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 229 | sk_sp<SkData> iccData = get_icc_profile(decoderMgr->dinfo()); |
| 230 | sk_sp<SkColorSpace> colorSpace = nullptr; |
| 231 | if (iccData) { |
| 232 | colorSpace = SkColorSpace::NewICC(iccData->data(), iccData->size()); |
| 233 | if (!colorSpace) { |
| 234 | SkCodecPrintf("Could not create SkColorSpace from ICC data.\n"); |
| 235 | } |
| 236 | } |
msarett | f34cd63 | 2016-05-25 10:13:53 -0700 | [diff] [blame] | 237 | if (!colorSpace) { |
| 238 | // Treat unmarked jpegs as sRGB. |
| 239 | colorSpace = SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named); |
| 240 | } |
msarett | 0e6274f | 2016-03-21 08:04:40 -0700 | [diff] [blame] | 241 | |
msarett | c30c418 | 2016-04-20 11:53:35 -0700 | [diff] [blame] | 242 | const int width = decoderMgr->dinfo()->image_width; |
| 243 | const int height = decoderMgr->dinfo()->image_height; |
msarett | f34cd63 | 2016-05-25 10:13:53 -0700 | [diff] [blame] | 244 | *codecOut = new SkJpegCodec(width, height, info, stream, decoderMgr.release(), |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 245 | std::move(colorSpace), orientation, std::move(iccData)); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 246 | } else { |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 247 | SkASSERT(nullptr != decoderMgrOut); |
mtklein | 18300a3 | 2016-03-16 13:53:35 -0700 | [diff] [blame] | 248 | *decoderMgrOut = decoderMgr.release(); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 249 | } |
| 250 | return true; |
| 251 | } |
| 252 | |
| 253 | SkCodec* SkJpegCodec::NewFromStream(SkStream* stream) { |
| 254 | SkAutoTDelete<SkStream> streamDeleter(stream); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 255 | SkCodec* codec = nullptr; |
| 256 | if (ReadHeader(stream, &codec, nullptr)) { |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 257 | // Codec has taken ownership of the stream, we do not need to delete it |
| 258 | SkASSERT(codec); |
mtklein | 18300a3 | 2016-03-16 13:53:35 -0700 | [diff] [blame] | 259 | streamDeleter.release(); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 260 | return codec; |
| 261 | } |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 262 | return nullptr; |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 263 | } |
| 264 | |
msarett | c30c418 | 2016-04-20 11:53:35 -0700 | [diff] [blame] | 265 | SkJpegCodec::SkJpegCodec(int width, int height, const SkEncodedInfo& info, SkStream* stream, |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 266 | JpegDecoderMgr* decoderMgr, sk_sp<SkColorSpace> colorSpace, Origin origin, |
| 267 | sk_sp<SkData> iccData) |
msarett | f34cd63 | 2016-05-25 10:13:53 -0700 | [diff] [blame] | 268 | : INHERITED(width, height, info, stream, std::move(colorSpace), origin) |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 269 | , fDecoderMgr(decoderMgr) |
msarett | fbccb59 | 2015-09-01 06:43:41 -0700 | [diff] [blame] | 270 | , fReadyState(decoderMgr->dinfo()->global_state) |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 271 | , fSwizzleSrcRow(nullptr) |
| 272 | , fColorXformSrcRow(nullptr) |
msarett | 91c22b2 | 2016-02-22 12:27:46 -0800 | [diff] [blame] | 273 | , fSwizzlerSubset(SkIRect::MakeEmpty()) |
msarett | 9876ac5 | 2016-06-01 14:47:18 -0700 | [diff] [blame] | 274 | , fICCData(std::move(iccData)) |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 275 | {} |
| 276 | |
| 277 | /* |
emmaleer | 8f4ba76 | 2015-08-14 07:44:46 -0700 | [diff] [blame] | 278 | * Return the row bytes of a particular image type and width |
| 279 | */ |
msarett | 23e78d3 | 2016-02-06 15:58:50 -0800 | [diff] [blame] | 280 | static size_t get_row_bytes(const j_decompress_ptr dinfo) { |
msarett | 70e418b | 2016-02-12 12:35:48 -0800 | [diff] [blame] | 281 | const size_t colorBytes = (dinfo->out_color_space == JCS_RGB565) ? 2 : |
| 282 | dinfo->out_color_components; |
emmaleer | 8f4ba76 | 2015-08-14 07:44:46 -0700 | [diff] [blame] | 283 | return dinfo->output_width * colorBytes; |
| 284 | |
| 285 | } |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 286 | |
| 287 | /* |
| 288 | * Calculate output dimensions based on the provided factors. |
| 289 | * |
| 290 | * Not to be used on the actual jpeg_decompress_struct used for decoding, since it will |
| 291 | * incorrectly modify num_components. |
| 292 | */ |
| 293 | void calc_output_dimensions(jpeg_decompress_struct* dinfo, unsigned int num, unsigned int denom) { |
| 294 | dinfo->num_components = 0; |
| 295 | dinfo->scale_num = num; |
| 296 | dinfo->scale_denom = denom; |
| 297 | jpeg_calc_output_dimensions(dinfo); |
| 298 | } |
| 299 | |
emmaleer | 8f4ba76 | 2015-08-14 07:44:46 -0700 | [diff] [blame] | 300 | /* |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 301 | * Return a valid set of output dimensions for this decoder, given an input scale |
| 302 | */ |
| 303 | SkISize SkJpegCodec::onGetScaledDimensions(float desiredScale) const { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 304 | // libjpeg-turbo supports scaling by 1/8, 1/4, 3/8, 1/2, 5/8, 3/4, 7/8, and 1/1, so we will |
| 305 | // support these as well |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 306 | unsigned int num; |
| 307 | unsigned int denom = 8; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 308 | if (desiredScale >= 0.9375) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 309 | num = 8; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 310 | } else if (desiredScale >= 0.8125) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 311 | num = 7; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 312 | } else if (desiredScale >= 0.6875f) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 313 | num = 6; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 314 | } else if (desiredScale >= 0.5625f) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 315 | num = 5; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 316 | } else if (desiredScale >= 0.4375f) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 317 | num = 4; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 318 | } else if (desiredScale >= 0.3125f) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 319 | num = 3; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 320 | } else if (desiredScale >= 0.1875f) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 321 | num = 2; |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 322 | } else { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 323 | num = 1; |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 324 | } |
| 325 | |
| 326 | // Set up a fake decompress struct in order to use libjpeg to calculate output dimensions |
| 327 | jpeg_decompress_struct dinfo; |
mtklein | f7aaadb | 2015-04-16 06:09:27 -0700 | [diff] [blame] | 328 | sk_bzero(&dinfo, sizeof(dinfo)); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 329 | dinfo.image_width = this->getInfo().width(); |
| 330 | dinfo.image_height = this->getInfo().height(); |
msarett | fbccb59 | 2015-09-01 06:43:41 -0700 | [diff] [blame] | 331 | dinfo.global_state = fReadyState; |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 332 | calc_output_dimensions(&dinfo, num, denom); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 333 | |
| 334 | // Return the calculated output dimensions for the given scale |
| 335 | return SkISize::Make(dinfo.output_width, dinfo.output_height); |
| 336 | } |
| 337 | |
scroggo | b427db1 | 2015-08-12 07:24:13 -0700 | [diff] [blame] | 338 | bool SkJpegCodec::onRewind() { |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 339 | JpegDecoderMgr* decoderMgr = nullptr; |
| 340 | if (!ReadHeader(this->stream(), nullptr, &decoderMgr)) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 341 | return fDecoderMgr->returnFalse("onRewind"); |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 342 | } |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 343 | SkASSERT(nullptr != decoderMgr); |
scroggo | b427db1 | 2015-08-12 07:24:13 -0700 | [diff] [blame] | 344 | fDecoderMgr.reset(decoderMgr); |
msarett | 2812f03 | 2016-07-18 15:56:08 -0700 | [diff] [blame] | 345 | |
| 346 | fSwizzler.reset(nullptr); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 347 | fSwizzleSrcRow = nullptr; |
| 348 | fColorXformSrcRow = nullptr; |
msarett | 2812f03 | 2016-07-18 15:56:08 -0700 | [diff] [blame] | 349 | fStorage.reset(); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 350 | fColorXform.reset(nullptr); |
msarett | 2812f03 | 2016-07-18 15:56:08 -0700 | [diff] [blame] | 351 | |
scroggo | b427db1 | 2015-08-12 07:24:13 -0700 | [diff] [blame] | 352 | return true; |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 353 | } |
| 354 | |
| 355 | /* |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 356 | * Checks if the conversion between the input image and the requested output |
| 357 | * image has been implemented |
| 358 | * Sets the output color space |
| 359 | */ |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 360 | bool SkJpegCodec::setOutputColorSpace(const SkImageInfo& dstInfo, bool needsColorXform) { |
| 361 | if (kUnknown_SkAlphaType == dstInfo.alphaType()) { |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 362 | return false; |
| 363 | } |
| 364 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 365 | if (kOpaque_SkAlphaType != dstInfo.alphaType()) { |
scroggo | c5560be | 2016-02-03 09:42:42 -0800 | [diff] [blame] | 366 | SkCodecPrintf("Warning: an opaque image should be decoded as opaque " |
| 367 | "- it is being decoded as non-opaque, which will draw slower\n"); |
| 368 | } |
| 369 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 370 | // Check if we will decode to CMYK. libjpeg-turbo does not convert CMYK to RGBA, so |
| 371 | // we must do it ourselves. |
| 372 | J_COLOR_SPACE encodedColorType = fDecoderMgr->dinfo()->jpeg_color_space; |
| 373 | bool isCMYK = (JCS_CMYK == encodedColorType || JCS_YCCK == encodedColorType); |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 374 | |
| 375 | // Check for valid color types and set the output color space |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 376 | switch (dstInfo.colorType()) { |
msarett | 34e0ec4 | 2016-04-22 16:27:24 -0700 | [diff] [blame] | 377 | case kRGBA_8888_SkColorType: |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 378 | if (isCMYK) { |
| 379 | fDecoderMgr->dinfo()->out_color_space = JCS_CMYK; |
| 380 | } else { |
msarett | f25bff9 | 2016-07-21 12:00:24 -0700 | [diff] [blame] | 381 | fDecoderMgr->dinfo()->out_color_space = JCS_EXT_RGBA; |
msarett | 34e0ec4 | 2016-04-22 16:27:24 -0700 | [diff] [blame] | 382 | } |
| 383 | return true; |
| 384 | case kBGRA_8888_SkColorType: |
| 385 | if (isCMYK) { |
| 386 | fDecoderMgr->dinfo()->out_color_space = JCS_CMYK; |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 387 | } else if (needsColorXform) { |
| 388 | // Our color transformation code requires RGBA order inputs, but it'll swizzle |
| 389 | // to BGRA for us. |
| 390 | fDecoderMgr->dinfo()->out_color_space = JCS_EXT_RGBA; |
msarett | 34e0ec4 | 2016-04-22 16:27:24 -0700 | [diff] [blame] | 391 | } else { |
msarett | f25bff9 | 2016-07-21 12:00:24 -0700 | [diff] [blame] | 392 | fDecoderMgr->dinfo()->out_color_space = JCS_EXT_BGRA; |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 393 | } |
| 394 | return true; |
| 395 | case kRGB_565_SkColorType: |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 396 | if (needsColorXform) { |
| 397 | return false; |
| 398 | } |
| 399 | |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 400 | if (isCMYK) { |
scroggo | ef27d89 | 2015-10-23 09:29:22 -0700 | [diff] [blame] | 401 | fDecoderMgr->dinfo()->out_color_space = JCS_CMYK; |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 402 | } else { |
msarett | 8ff6ca6 | 2015-09-18 12:06:04 -0700 | [diff] [blame] | 403 | fDecoderMgr->dinfo()->dither_mode = JDITHER_NONE; |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 404 | fDecoderMgr->dinfo()->out_color_space = JCS_RGB565; |
| 405 | } |
| 406 | return true; |
| 407 | case kGray_8_SkColorType: |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 408 | if (needsColorXform || JCS_GRAYSCALE != encodedColorType) { |
msarett | 39979d8 | 2016-07-28 17:11:18 -0700 | [diff] [blame] | 409 | return false; |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 410 | } |
| 411 | |
| 412 | fDecoderMgr->dinfo()->out_color_space = JCS_GRAYSCALE; |
| 413 | return true; |
| 414 | case kRGBA_F16_SkColorType: |
| 415 | SkASSERT(needsColorXform); |
| 416 | |
| 417 | if (isCMYK) { |
| 418 | fDecoderMgr->dinfo()->out_color_space = JCS_CMYK; |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 419 | } else { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 420 | fDecoderMgr->dinfo()->out_color_space = JCS_EXT_RGBA; |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 421 | } |
| 422 | return true; |
| 423 | default: |
| 424 | return false; |
| 425 | } |
| 426 | } |
| 427 | |
| 428 | /* |
mtklein | e721a8e | 2016-02-06 19:12:23 -0800 | [diff] [blame] | 429 | * Checks if we can natively scale to the requested dimensions and natively scales the |
emmaleer | 8f4ba76 | 2015-08-14 07:44:46 -0700 | [diff] [blame] | 430 | * dimensions if possible |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 431 | */ |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 432 | bool SkJpegCodec::onDimensionsSupported(const SkISize& size) { |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 433 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 434 | return fDecoderMgr->returnFalse("onDimensionsSupported"); |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 435 | } |
| 436 | |
| 437 | const unsigned int dstWidth = size.width(); |
| 438 | const unsigned int dstHeight = size.height(); |
| 439 | |
| 440 | // Set up a fake decompress struct in order to use libjpeg to calculate output dimensions |
| 441 | // FIXME: Why is this necessary? |
| 442 | jpeg_decompress_struct dinfo; |
| 443 | sk_bzero(&dinfo, sizeof(dinfo)); |
| 444 | dinfo.image_width = this->getInfo().width(); |
| 445 | dinfo.image_height = this->getInfo().height(); |
| 446 | dinfo.global_state = fReadyState; |
| 447 | |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 448 | // libjpeg-turbo can scale to 1/8, 1/4, 3/8, 1/2, 5/8, 3/4, 7/8, and 1/1 |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 449 | unsigned int num = 8; |
| 450 | const unsigned int denom = 8; |
| 451 | calc_output_dimensions(&dinfo, num, denom); |
| 452 | while (dinfo.output_width != dstWidth || dinfo.output_height != dstHeight) { |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 453 | |
| 454 | // Return a failure if we have tried all of the possible scales |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 455 | if (1 == num || dstWidth > dinfo.output_width || dstHeight > dinfo.output_height) { |
emmaleer | 8f4ba76 | 2015-08-14 07:44:46 -0700 | [diff] [blame] | 456 | return false; |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 457 | } |
| 458 | |
| 459 | // Try the next scale |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 460 | num -= 1; |
| 461 | calc_output_dimensions(&dinfo, num, denom); |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 462 | } |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 463 | |
| 464 | fDecoderMgr->dinfo()->scale_num = num; |
| 465 | fDecoderMgr->dinfo()->scale_denom = denom; |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 466 | return true; |
| 467 | } |
| 468 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 469 | int SkJpegCodec::readRows(const SkImageInfo& dstInfo, void* dst, size_t rowBytes, int count) { |
| 470 | // Set the jump location for libjpeg-turbo errors |
| 471 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
| 472 | return 0; |
| 473 | } |
| 474 | |
| 475 | // When fSwizzleSrcRow is non-null, it means that we need to swizzle. In this case, |
| 476 | // we will always decode into fSwizzlerSrcRow before swizzling into the next buffer. |
| 477 | // We can never swizzle "in place" because the swizzler may perform sampling and/or |
| 478 | // subsetting. |
| 479 | // When fColorXformSrcRow is non-null, it means that we need to color xform and that |
| 480 | // we cannot color xform "in place" (many times we can, but not when the dst is F16). |
| 481 | // In this case, we will color xform from fColorXformSrc into the dst. |
| 482 | JSAMPLE* decodeDst = (JSAMPLE*) dst; |
| 483 | uint32_t* swizzleDst = (uint32_t*) dst; |
| 484 | size_t decodeDstRowBytes = rowBytes; |
| 485 | size_t swizzleDstRowBytes = rowBytes; |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 486 | int dstWidth = dstInfo.width(); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 487 | if (fSwizzleSrcRow && fColorXformSrcRow) { |
| 488 | decodeDst = (JSAMPLE*) fSwizzleSrcRow; |
| 489 | swizzleDst = fColorXformSrcRow; |
| 490 | decodeDstRowBytes = 0; |
| 491 | swizzleDstRowBytes = 0; |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 492 | dstWidth = fSwizzler->swizzleWidth(); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 493 | } else if (fColorXformSrcRow) { |
| 494 | decodeDst = (JSAMPLE*) fColorXformSrcRow; |
| 495 | swizzleDst = fColorXformSrcRow; |
| 496 | decodeDstRowBytes = 0; |
| 497 | swizzleDstRowBytes = 0; |
| 498 | } else if (fSwizzleSrcRow) { |
| 499 | decodeDst = (JSAMPLE*) fSwizzleSrcRow; |
| 500 | decodeDstRowBytes = 0; |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 501 | dstWidth = fSwizzler->swizzleWidth(); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 502 | } |
| 503 | |
| 504 | for (int y = 0; y < count; y++) { |
| 505 | uint32_t lines = jpeg_read_scanlines(fDecoderMgr->dinfo(), &decodeDst, 1); |
| 506 | size_t srcRowBytes = get_row_bytes(fDecoderMgr->dinfo()); |
| 507 | sk_msan_mark_initialized(decodeDst, decodeDst + srcRowBytes, "skbug.com/4550"); |
| 508 | if (0 == lines) { |
| 509 | return y; |
| 510 | } |
| 511 | |
| 512 | if (fSwizzler) { |
| 513 | fSwizzler->swizzle(swizzleDst, decodeDst); |
| 514 | } |
| 515 | |
| 516 | if (fColorXform) { |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 517 | fColorXform->apply(dst, swizzleDst, dstWidth, dstInfo.colorType(), kOpaque_SkAlphaType); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 518 | dst = SkTAddOffset<void>(dst, rowBytes); |
| 519 | } |
| 520 | |
| 521 | decodeDst = SkTAddOffset<JSAMPLE>(decodeDst, decodeDstRowBytes); |
| 522 | swizzleDst = SkTAddOffset<uint32_t>(swizzleDst, swizzleDstRowBytes); |
| 523 | } |
| 524 | |
| 525 | return count; |
| 526 | } |
| 527 | |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 528 | /* |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 529 | * Performs the jpeg decode |
| 530 | */ |
| 531 | SkCodec::Result SkJpegCodec::onGetPixels(const SkImageInfo& dstInfo, |
| 532 | void* dst, size_t dstRowBytes, |
msarett | e6dd004 | 2015-10-09 11:07:34 -0700 | [diff] [blame] | 533 | const Options& options, SkPMColor*, int*, |
| 534 | int* rowsDecoded) { |
scroggo | b636b45 | 2015-07-22 07:16:20 -0700 | [diff] [blame] | 535 | if (options.fSubset) { |
| 536 | // Subsets are not supported. |
| 537 | return kUnimplemented; |
| 538 | } |
| 539 | |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 540 | // Get a pointer to the decompress info since we will use it quite frequently |
| 541 | jpeg_decompress_struct* dinfo = fDecoderMgr->dinfo(); |
| 542 | |
| 543 | // Set the jump location for libjpeg errors |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 544 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 545 | return fDecoderMgr->returnFailure("setjmp", kInvalidInput); |
| 546 | } |
| 547 | |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 548 | // Check if we can decode to the requested destination and set the output color space |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 549 | bool needsColorXform = needs_color_xform(dstInfo, this->getInfo()); |
| 550 | if (!this->setOutputColorSpace(dstInfo, needsColorXform)) { |
| 551 | return fDecoderMgr->returnFailure("setOutputColorSpace", kInvalidConversion); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 552 | } |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 553 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 554 | if (!this->initializeColorXform(dstInfo, needsColorXform)) { |
| 555 | return fDecoderMgr->returnFailure("initializeColorXform", kInvalidParameters); |
| 556 | } |
| 557 | |
msarett | fbccb59 | 2015-09-01 06:43:41 -0700 | [diff] [blame] | 558 | if (!jpeg_start_decompress(dinfo)) { |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 559 | return fDecoderMgr->returnFailure("startDecompress", kInvalidInput); |
| 560 | } |
| 561 | |
msarett | 1c8a587 | 2015-07-07 08:50:01 -0700 | [diff] [blame] | 562 | // The recommended output buffer height should always be 1 in high quality modes. |
| 563 | // If it's not, we want to know because it means our strategy is not optimal. |
| 564 | SkASSERT(1 == dinfo->rec_outbuf_height); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 565 | |
msarett | 70e418b | 2016-02-12 12:35:48 -0800 | [diff] [blame] | 566 | J_COLOR_SPACE colorSpace = dinfo->out_color_space; |
mtklein | da19f6f | 2016-08-23 11:49:29 -0700 | [diff] [blame^] | 567 | if (JCS_CMYK == colorSpace) { |
scroggo | ef27d89 | 2015-10-23 09:29:22 -0700 | [diff] [blame] | 568 | this->initializeSwizzler(dstInfo, options); |
| 569 | } |
| 570 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 571 | this->allocateStorage(dstInfo); |
scroggo | ef27d89 | 2015-10-23 09:29:22 -0700 | [diff] [blame] | 572 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 573 | int rows = this->readRows(dstInfo, dst, dstRowBytes, dstInfo.height()); |
| 574 | if (rows < dstInfo.height()) { |
| 575 | *rowsDecoded = rows; |
| 576 | return fDecoderMgr->returnFailure("Incomplete image data", kIncompleteInput); |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 577 | } |
msarett | e16b04a | 2015-04-15 07:32:19 -0700 | [diff] [blame] | 578 | |
| 579 | return kSuccess; |
| 580 | } |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 581 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 582 | void SkJpegCodec::allocateStorage(const SkImageInfo& dstInfo) { |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 583 | int dstWidth = dstInfo.width(); |
| 584 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 585 | size_t swizzleBytes = 0; |
| 586 | if (fSwizzler) { |
| 587 | swizzleBytes = get_row_bytes(fDecoderMgr->dinfo()); |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 588 | dstWidth = fSwizzler->swizzleWidth(); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 589 | SkASSERT(!fColorXform || SkIsAlign4(swizzleBytes)); |
| 590 | } |
| 591 | |
| 592 | size_t xformBytes = 0; |
| 593 | if (kRGBA_F16_SkColorType == dstInfo.colorType()) { |
| 594 | SkASSERT(fColorXform); |
msarett | 35bb74b | 2016-08-22 07:41:28 -0700 | [diff] [blame] | 595 | xformBytes = dstWidth * sizeof(uint32_t); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 596 | } |
| 597 | |
| 598 | size_t totalBytes = swizzleBytes + xformBytes; |
| 599 | if (totalBytes > 0) { |
| 600 | fStorage.reset(totalBytes); |
| 601 | fSwizzleSrcRow = (swizzleBytes > 0) ? fStorage.get() : nullptr; |
| 602 | fColorXformSrcRow = (xformBytes > 0) ? |
| 603 | SkTAddOffset<uint32_t>(fStorage.get(), swizzleBytes) : nullptr; |
| 604 | } |
| 605 | } |
| 606 | |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 607 | void SkJpegCodec::initializeSwizzler(const SkImageInfo& dstInfo, const Options& options) { |
msarett | a45a668 | 2016-04-22 13:18:37 -0700 | [diff] [blame] | 608 | // libjpeg-turbo may have already performed color conversion. We must indicate the |
| 609 | // appropriate format to the swizzler. |
| 610 | SkEncodedInfo swizzlerInfo = this->getEncodedInfo(); |
msarett | 68758ae | 2016-04-25 11:41:15 -0700 | [diff] [blame] | 611 | bool preSwizzled = true; |
mtklein | da19f6f | 2016-08-23 11:49:29 -0700 | [diff] [blame^] | 612 | if (JCS_CMYK == fDecoderMgr->dinfo()->out_color_space) { |
| 613 | preSwizzled = false; |
| 614 | swizzlerInfo = SkEncodedInfo::Make(SkEncodedInfo::kInvertedCMYK_Color, |
| 615 | swizzlerInfo.alpha(), |
| 616 | swizzlerInfo.bitsPerComponent()); |
msarett | 70e418b | 2016-02-12 12:35:48 -0800 | [diff] [blame] | 617 | } |
| 618 | |
msarett | 91c22b2 | 2016-02-22 12:27:46 -0800 | [diff] [blame] | 619 | Options swizzlerOptions = options; |
| 620 | if (options.fSubset) { |
| 621 | // Use fSwizzlerSubset if this is a subset decode. This is necessary in the case |
| 622 | // where libjpeg-turbo provides a subset and then we need to subset it further. |
| 623 | // Also, verify that fSwizzlerSubset is initialized and valid. |
| 624 | SkASSERT(!fSwizzlerSubset.isEmpty() && fSwizzlerSubset.x() <= options.fSubset->x() && |
| 625 | fSwizzlerSubset.width() == options.fSubset->width()); |
| 626 | swizzlerOptions.fSubset = &fSwizzlerSubset; |
| 627 | } |
msarett | 68758ae | 2016-04-25 11:41:15 -0700 | [diff] [blame] | 628 | fSwizzler.reset(SkSwizzler::CreateSwizzler(swizzlerInfo, nullptr, dstInfo, swizzlerOptions, |
| 629 | nullptr, preSwizzled)); |
msarett | b30d698 | 2016-02-15 10:18:45 -0800 | [diff] [blame] | 630 | SkASSERT(fSwizzler); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 631 | } |
| 632 | |
| 633 | bool SkJpegCodec::initializeColorXform(const SkImageInfo& dstInfo, bool needsColorXform) { |
| 634 | if (needsColorXform) { |
| 635 | fColorXform = SkColorSpaceXform::New(sk_ref_sp(this->getInfo().colorSpace()), |
| 636 | sk_ref_sp(dstInfo.colorSpace())); |
| 637 | if (!fColorXform && kRGBA_F16_SkColorType == dstInfo.colorType()) { |
| 638 | return false; |
| 639 | } |
| 640 | } |
| 641 | |
| 642 | return true; |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 643 | } |
| 644 | |
| 645 | SkSampler* SkJpegCodec::getSampler(bool createIfNecessary) { |
| 646 | if (!createIfNecessary || fSwizzler) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 647 | SkASSERT(!fSwizzler || (fSwizzleSrcRow && fStorage.get() == fSwizzleSrcRow)); |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 648 | return fSwizzler; |
| 649 | } |
| 650 | |
| 651 | this->initializeSwizzler(this->dstInfo(), this->options()); |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 652 | this->allocateStorage(this->dstInfo()); |
scroggo | e7fc14b | 2015-10-02 13:14:46 -0700 | [diff] [blame] | 653 | return fSwizzler; |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 654 | } |
scroggo | 1c005e4 | 2015-08-04 09:24:45 -0700 | [diff] [blame] | 655 | |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 656 | SkCodec::Result SkJpegCodec::onStartScanlineDecode(const SkImageInfo& dstInfo, |
| 657 | const Options& options, SkPMColor ctable[], int* ctableCount) { |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 658 | // Set the jump location for libjpeg errors |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 659 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 660 | SkCodecPrintf("setjmp: Error from libjpeg\n"); |
| 661 | return kInvalidInput; |
| 662 | } |
| 663 | |
| 664 | // Check if we can decode to the requested destination and set the output color space |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 665 | bool needsColorXform = needs_color_xform(dstInfo, this->getInfo()); |
| 666 | if (!this->setOutputColorSpace(dstInfo, needsColorXform)) { |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 667 | return kInvalidConversion; |
| 668 | } |
| 669 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 670 | if (!this->initializeColorXform(dstInfo, needsColorXform)) { |
| 671 | return fDecoderMgr->returnFailure("initializeColorXform", kInvalidParameters); |
| 672 | } |
| 673 | |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 674 | if (!jpeg_start_decompress(fDecoderMgr->dinfo())) { |
| 675 | SkCodecPrintf("start decompress failed\n"); |
| 676 | return kInvalidInput; |
| 677 | } |
| 678 | |
msarett | 91c22b2 | 2016-02-22 12:27:46 -0800 | [diff] [blame] | 679 | if (options.fSubset) { |
| 680 | uint32_t startX = options.fSubset->x(); |
| 681 | uint32_t width = options.fSubset->width(); |
| 682 | |
| 683 | // libjpeg-turbo may need to align startX to a multiple of the IDCT |
| 684 | // block size. If this is the case, it will decrease the value of |
| 685 | // startX to the appropriate alignment and also increase the value |
| 686 | // of width so that the right edge of the requested subset remains |
| 687 | // the same. |
| 688 | jpeg_crop_scanline(fDecoderMgr->dinfo(), &startX, &width); |
| 689 | |
| 690 | SkASSERT(startX <= (uint32_t) options.fSubset->x()); |
| 691 | SkASSERT(width >= (uint32_t) options.fSubset->width()); |
| 692 | SkASSERT(startX + width >= (uint32_t) options.fSubset->right()); |
| 693 | |
| 694 | // Instruct the swizzler (if it is necessary) to further subset the |
| 695 | // output provided by libjpeg-turbo. |
| 696 | // |
| 697 | // We set this here (rather than in the if statement below), so that |
| 698 | // if (1) we don't need a swizzler for the subset, and (2) we need a |
| 699 | // swizzler for CMYK, the swizzler will still use the proper subset |
| 700 | // dimensions. |
| 701 | // |
| 702 | // Note that the swizzler will ignore the y and height parameters of |
| 703 | // the subset. Since the scanline decoder (and the swizzler) handle |
| 704 | // one row at a time, only the subsetting in the x-dimension matters. |
| 705 | fSwizzlerSubset.setXYWH(options.fSubset->x() - startX, 0, |
| 706 | options.fSubset->width(), options.fSubset->height()); |
| 707 | |
| 708 | // We will need a swizzler if libjpeg-turbo cannot provide the exact |
| 709 | // subset that we request. |
| 710 | if (startX != (uint32_t) options.fSubset->x() || |
| 711 | width != (uint32_t) options.fSubset->width()) { |
| 712 | this->initializeSwizzler(dstInfo, options); |
| 713 | } |
| 714 | } |
| 715 | |
| 716 | // Make sure we have a swizzler if we are converting from CMYK. |
| 717 | if (!fSwizzler && JCS_CMYK == fDecoderMgr->dinfo()->out_color_space) { |
| 718 | this->initializeSwizzler(dstInfo, options); |
| 719 | } |
msarett | fdb4757 | 2015-10-13 12:50:14 -0700 | [diff] [blame] | 720 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 721 | this->allocateStorage(dstInfo); |
| 722 | |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 723 | return kSuccess; |
| 724 | } |
| 725 | |
mtklein | e721a8e | 2016-02-06 19:12:23 -0800 | [diff] [blame] | 726 | int SkJpegCodec::onGetScanlines(void* dst, int count, size_t dstRowBytes) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 727 | int rows = this->readRows(this->dstInfo(), dst, dstRowBytes, count); |
| 728 | if (rows < count) { |
| 729 | // This allows us to skip calling jpeg_finish_decompress(). |
| 730 | fDecoderMgr->dinfo()->output_scanline = this->dstInfo().height(); |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 731 | } |
| 732 | |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 733 | return rows; |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 734 | } |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 735 | |
msarett | e6dd004 | 2015-10-09 11:07:34 -0700 | [diff] [blame] | 736 | bool SkJpegCodec::onSkipScanlines(int count) { |
scroggo | 46c5747 | 2015-09-30 08:57:13 -0700 | [diff] [blame] | 737 | // Set the jump location for libjpeg errors |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 738 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
msarett | 50ce1f2 | 2016-07-29 06:23:33 -0700 | [diff] [blame] | 739 | return fDecoderMgr->returnFalse("onSkipScanlines"); |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 740 | } |
| 741 | |
msarett | f724b99 | 2015-10-15 06:41:06 -0700 | [diff] [blame] | 742 | return (uint32_t) count == jpeg_skip_scanlines(fDecoderMgr->dinfo(), count); |
msarett | 97fdea6 | 2015-04-29 08:17:15 -0700 | [diff] [blame] | 743 | } |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 744 | |
| 745 | static bool is_yuv_supported(jpeg_decompress_struct* dinfo) { |
| 746 | // Scaling is not supported in raw data mode. |
| 747 | SkASSERT(dinfo->scale_num == dinfo->scale_denom); |
| 748 | |
| 749 | // I can't imagine that this would ever change, but we do depend on it. |
| 750 | static_assert(8 == DCTSIZE, "DCTSIZE (defined in jpeg library) should always be 8."); |
| 751 | |
| 752 | if (JCS_YCbCr != dinfo->jpeg_color_space) { |
| 753 | return false; |
| 754 | } |
| 755 | |
| 756 | SkASSERT(3 == dinfo->num_components); |
| 757 | SkASSERT(dinfo->comp_info); |
| 758 | |
| 759 | // It is possible to perform a YUV decode for any combination of |
| 760 | // horizontal and vertical sampling that is supported by |
| 761 | // libjpeg/libjpeg-turbo. However, we will start by supporting only the |
| 762 | // common cases (where U and V have samp_factors of one). |
| 763 | // |
| 764 | // The definition of samp_factor is kind of the opposite of what SkCodec |
| 765 | // thinks of as a sampling factor. samp_factor is essentially a |
| 766 | // multiplier, and the larger the samp_factor is, the more samples that |
| 767 | // there will be. Ex: |
| 768 | // U_plane_width = image_width * (U_h_samp_factor / max_h_samp_factor) |
| 769 | // |
| 770 | // Supporting cases where the samp_factors for U or V were larger than |
| 771 | // that of Y would be an extremely difficult change, given that clients |
| 772 | // allocate memory as if the size of the Y plane is always the size of the |
| 773 | // image. However, this case is very, very rare. |
msarett | 7c87cf4 | 2016-03-04 06:23:20 -0800 | [diff] [blame] | 774 | if ((1 != dinfo->comp_info[1].h_samp_factor) || |
| 775 | (1 != dinfo->comp_info[1].v_samp_factor) || |
| 776 | (1 != dinfo->comp_info[2].h_samp_factor) || |
| 777 | (1 != dinfo->comp_info[2].v_samp_factor)) |
| 778 | { |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 779 | return false; |
| 780 | } |
| 781 | |
| 782 | // Support all common cases of Y samp_factors. |
| 783 | // TODO (msarett): As mentioned above, it would be possible to support |
| 784 | // more combinations of samp_factors. The issues are: |
| 785 | // (1) Are there actually any images that are not covered |
| 786 | // by these cases? |
| 787 | // (2) How much complexity would be added to the |
| 788 | // implementation in order to support these rare |
| 789 | // cases? |
| 790 | int hSampY = dinfo->comp_info[0].h_samp_factor; |
| 791 | int vSampY = dinfo->comp_info[0].v_samp_factor; |
| 792 | return (1 == hSampY && 1 == vSampY) || |
| 793 | (2 == hSampY && 1 == vSampY) || |
| 794 | (2 == hSampY && 2 == vSampY) || |
| 795 | (1 == hSampY && 2 == vSampY) || |
| 796 | (4 == hSampY && 1 == vSampY) || |
| 797 | (4 == hSampY && 2 == vSampY); |
| 798 | } |
| 799 | |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 800 | bool SkJpegCodec::onQueryYUV8(SkYUVSizeInfo* sizeInfo, SkYUVColorSpace* colorSpace) const { |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 801 | jpeg_decompress_struct* dinfo = fDecoderMgr->dinfo(); |
| 802 | if (!is_yuv_supported(dinfo)) { |
| 803 | return false; |
| 804 | } |
| 805 | |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 806 | sizeInfo->fSizes[SkYUVSizeInfo::kY].set(dinfo->comp_info[0].downsampled_width, |
| 807 | dinfo->comp_info[0].downsampled_height); |
| 808 | sizeInfo->fSizes[SkYUVSizeInfo::kU].set(dinfo->comp_info[1].downsampled_width, |
| 809 | dinfo->comp_info[1].downsampled_height); |
| 810 | sizeInfo->fSizes[SkYUVSizeInfo::kV].set(dinfo->comp_info[2].downsampled_width, |
| 811 | dinfo->comp_info[2].downsampled_height); |
| 812 | sizeInfo->fWidthBytes[SkYUVSizeInfo::kY] = dinfo->comp_info[0].width_in_blocks * DCTSIZE; |
| 813 | sizeInfo->fWidthBytes[SkYUVSizeInfo::kU] = dinfo->comp_info[1].width_in_blocks * DCTSIZE; |
| 814 | sizeInfo->fWidthBytes[SkYUVSizeInfo::kV] = dinfo->comp_info[2].width_in_blocks * DCTSIZE; |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 815 | |
| 816 | if (colorSpace) { |
| 817 | *colorSpace = kJPEG_SkYUVColorSpace; |
| 818 | } |
| 819 | |
| 820 | return true; |
| 821 | } |
| 822 | |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 823 | SkCodec::Result SkJpegCodec::onGetYUV8Planes(const SkYUVSizeInfo& sizeInfo, void* planes[3]) { |
| 824 | SkYUVSizeInfo defaultInfo; |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 825 | |
| 826 | // This will check is_yuv_supported(), so we don't need to here. |
| 827 | bool supportsYUV = this->onQueryYUV8(&defaultInfo, nullptr); |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 828 | if (!supportsYUV || |
| 829 | sizeInfo.fSizes[SkYUVSizeInfo::kY] != defaultInfo.fSizes[SkYUVSizeInfo::kY] || |
| 830 | sizeInfo.fSizes[SkYUVSizeInfo::kU] != defaultInfo.fSizes[SkYUVSizeInfo::kU] || |
| 831 | sizeInfo.fSizes[SkYUVSizeInfo::kV] != defaultInfo.fSizes[SkYUVSizeInfo::kV] || |
| 832 | sizeInfo.fWidthBytes[SkYUVSizeInfo::kY] < defaultInfo.fWidthBytes[SkYUVSizeInfo::kY] || |
| 833 | sizeInfo.fWidthBytes[SkYUVSizeInfo::kU] < defaultInfo.fWidthBytes[SkYUVSizeInfo::kU] || |
| 834 | sizeInfo.fWidthBytes[SkYUVSizeInfo::kV] < defaultInfo.fWidthBytes[SkYUVSizeInfo::kV]) { |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 835 | return fDecoderMgr->returnFailure("onGetYUV8Planes", kInvalidInput); |
| 836 | } |
| 837 | |
| 838 | // Set the jump location for libjpeg errors |
mtklein | 2f42896 | 2016-07-28 13:59:59 -0700 | [diff] [blame] | 839 | if (setjmp(fDecoderMgr->getJmpBuf())) { |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 840 | return fDecoderMgr->returnFailure("setjmp", kInvalidInput); |
| 841 | } |
| 842 | |
| 843 | // Get a pointer to the decompress info since we will use it quite frequently |
| 844 | jpeg_decompress_struct* dinfo = fDecoderMgr->dinfo(); |
| 845 | |
| 846 | dinfo->raw_data_out = TRUE; |
| 847 | if (!jpeg_start_decompress(dinfo)) { |
| 848 | return fDecoderMgr->returnFailure("startDecompress", kInvalidInput); |
| 849 | } |
| 850 | |
| 851 | // A previous implementation claims that the return value of is_yuv_supported() |
| 852 | // may change after calling jpeg_start_decompress(). It looks to me like this |
| 853 | // was caused by a bug in the old code, but we'll be safe and check here. |
| 854 | SkASSERT(is_yuv_supported(dinfo)); |
| 855 | |
| 856 | // Currently, we require that the Y plane dimensions match the image dimensions |
| 857 | // and that the U and V planes are the same dimensions. |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 858 | SkASSERT(sizeInfo.fSizes[SkYUVSizeInfo::kU] == sizeInfo.fSizes[SkYUVSizeInfo::kV]); |
| 859 | SkASSERT((uint32_t) sizeInfo.fSizes[SkYUVSizeInfo::kY].width() == dinfo->output_width && |
| 860 | (uint32_t) sizeInfo.fSizes[SkYUVSizeInfo::kY].height() == dinfo->output_height); |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 861 | |
| 862 | // Build a JSAMPIMAGE to handle output from libjpeg-turbo. A JSAMPIMAGE has |
| 863 | // a 2-D array of pixels for each of the components (Y, U, V) in the image. |
| 864 | // Cheat Sheet: |
| 865 | // JSAMPIMAGE == JSAMPLEARRAY* == JSAMPROW** == JSAMPLE*** |
| 866 | JSAMPARRAY yuv[3]; |
| 867 | |
| 868 | // Set aside enough space for pointers to rows of Y, U, and V. |
| 869 | JSAMPROW rowptrs[2 * DCTSIZE + DCTSIZE + DCTSIZE]; |
| 870 | yuv[0] = &rowptrs[0]; // Y rows (DCTSIZE or 2 * DCTSIZE) |
| 871 | yuv[1] = &rowptrs[2 * DCTSIZE]; // U rows (DCTSIZE) |
| 872 | yuv[2] = &rowptrs[3 * DCTSIZE]; // V rows (DCTSIZE) |
| 873 | |
| 874 | // Initialize rowptrs. |
| 875 | int numYRowsPerBlock = DCTSIZE * dinfo->comp_info[0].v_samp_factor; |
| 876 | for (int i = 0; i < numYRowsPerBlock; i++) { |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 877 | rowptrs[i] = SkTAddOffset<JSAMPLE>(planes[SkYUVSizeInfo::kY], |
| 878 | i * sizeInfo.fWidthBytes[SkYUVSizeInfo::kY]); |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 879 | } |
| 880 | for (int i = 0; i < DCTSIZE; i++) { |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 881 | rowptrs[i + 2 * DCTSIZE] = SkTAddOffset<JSAMPLE>(planes[SkYUVSizeInfo::kU], |
| 882 | i * sizeInfo.fWidthBytes[SkYUVSizeInfo::kU]); |
| 883 | rowptrs[i + 3 * DCTSIZE] = SkTAddOffset<JSAMPLE>(planes[SkYUVSizeInfo::kV], |
| 884 | i * sizeInfo.fWidthBytes[SkYUVSizeInfo::kV]); |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 885 | } |
| 886 | |
| 887 | // After each loop iteration, we will increment pointers to Y, U, and V. |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 888 | size_t blockIncrementY = numYRowsPerBlock * sizeInfo.fWidthBytes[SkYUVSizeInfo::kY]; |
| 889 | size_t blockIncrementU = DCTSIZE * sizeInfo.fWidthBytes[SkYUVSizeInfo::kU]; |
| 890 | size_t blockIncrementV = DCTSIZE * sizeInfo.fWidthBytes[SkYUVSizeInfo::kV]; |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 891 | |
| 892 | uint32_t numRowsPerBlock = numYRowsPerBlock; |
| 893 | |
| 894 | // We intentionally round down here, as this first loop will only handle |
| 895 | // full block rows. As a special case at the end, we will handle any |
| 896 | // remaining rows that do not make up a full block. |
| 897 | const int numIters = dinfo->output_height / numRowsPerBlock; |
| 898 | for (int i = 0; i < numIters; i++) { |
| 899 | JDIMENSION linesRead = jpeg_read_raw_data(dinfo, yuv, numRowsPerBlock); |
| 900 | if (linesRead < numRowsPerBlock) { |
| 901 | // FIXME: Handle incomplete YUV decodes without signalling an error. |
| 902 | return kInvalidInput; |
| 903 | } |
| 904 | |
| 905 | // Update rowptrs. |
| 906 | for (int i = 0; i < numYRowsPerBlock; i++) { |
| 907 | rowptrs[i] += blockIncrementY; |
| 908 | } |
| 909 | for (int i = 0; i < DCTSIZE; i++) { |
| 910 | rowptrs[i + 2 * DCTSIZE] += blockIncrementU; |
| 911 | rowptrs[i + 3 * DCTSIZE] += blockIncrementV; |
| 912 | } |
| 913 | } |
| 914 | |
| 915 | uint32_t remainingRows = dinfo->output_height - dinfo->output_scanline; |
| 916 | SkASSERT(remainingRows == dinfo->output_height % numRowsPerBlock); |
| 917 | SkASSERT(dinfo->output_scanline == numIters * numRowsPerBlock); |
| 918 | if (remainingRows > 0) { |
| 919 | // libjpeg-turbo needs memory to be padded by the block sizes. We will fulfill |
| 920 | // this requirement using a dummy row buffer. |
| 921 | // FIXME: Should SkCodec have an extra memory buffer that can be shared among |
| 922 | // all of the implementations that use temporary/garbage memory? |
msarett | 4984c3c | 2016-03-10 05:44:43 -0800 | [diff] [blame] | 923 | SkAutoTMalloc<JSAMPLE> dummyRow(sizeInfo.fWidthBytes[SkYUVSizeInfo::kY]); |
msarett | b714fb0 | 2016-01-22 14:46:42 -0800 | [diff] [blame] | 924 | for (int i = remainingRows; i < numYRowsPerBlock; i++) { |
| 925 | rowptrs[i] = dummyRow.get(); |
| 926 | } |
| 927 | int remainingUVRows = dinfo->comp_info[1].downsampled_height - DCTSIZE * numIters; |
| 928 | for (int i = remainingUVRows; i < DCTSIZE; i++) { |
| 929 | rowptrs[i + 2 * DCTSIZE] = dummyRow.get(); |
| 930 | rowptrs[i + 3 * DCTSIZE] = dummyRow.get(); |
| 931 | } |
| 932 | |
| 933 | JDIMENSION linesRead = jpeg_read_raw_data(dinfo, yuv, numRowsPerBlock); |
| 934 | if (linesRead < remainingRows) { |
| 935 | // FIXME: Handle incomplete YUV decodes without signalling an error. |
| 936 | return kInvalidInput; |
| 937 | } |
| 938 | } |
| 939 | |
| 940 | return kSuccess; |
| 941 | } |