Index: third_party/libwebp/dec/webp.c |
diff --git a/third_party/libwebp/dec/webp.c b/third_party/libwebp/dec/webp.c |
index e4fe73dc0a8bd8f9b8f29069977a8937ddcfc389..302220fecd2bc29d36f750f651ffda4d736931e7 100644 |
--- a/third_party/libwebp/dec/webp.c |
+++ b/third_party/libwebp/dec/webp.c |
@@ -18,10 +18,6 @@ |
#include "./webpi.h" |
#include "../webp/mux_types.h" // ALPHA_FLAG |
-#if defined(__cplusplus) || defined(c_plusplus) |
-extern "C" { |
-#endif |
- |
//------------------------------------------------------------------------------ |
// RIFF layout is: |
// Offset tag |
@@ -285,6 +281,7 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
int* const height, |
int* const has_alpha, |
int* const has_animation, |
+ int* const format, |
WebPHeaderStructure* const headers) { |
int canvas_width = 0; |
int canvas_height = 0; |
@@ -292,6 +289,9 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
int image_height = 0; |
int found_riff = 0; |
int found_vp8x = 0; |
+ int animation_present = 0; |
+ int fragments_present = 0; |
+ |
VP8StatusCode status; |
WebPHeaderStructure hdrs; |
@@ -312,13 +312,13 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
// Skip over VP8X. |
{ |
uint32_t flags = 0; |
- int animation_present; |
status = ParseVP8X(&data, &data_size, &found_vp8x, |
&canvas_width, &canvas_height, &flags); |
if (status != VP8_STATUS_OK) { |
return status; // Wrong VP8X / insufficient data. |
} |
animation_present = !!(flags & ANIMATION_FLAG); |
+ fragments_present = !!(flags & FRAGMENTS_FLAG); |
if (!found_riff && found_vp8x) { |
// Note: This restriction may be removed in the future, if it becomes |
// necessary to send VP8X chunk to the decoder. |
@@ -326,8 +326,10 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
} |
if (has_alpha != NULL) *has_alpha = !!(flags & ALPHA_FLAG); |
if (has_animation != NULL) *has_animation = animation_present; |
+ if (format != NULL) *format = 0; // default = undefined |
- if (found_vp8x && animation_present && headers == NULL) { |
+ if (found_vp8x && (animation_present || fragments_present) && |
+ headers == NULL) { |
if (width != NULL) *width = canvas_width; |
if (height != NULL) *height = canvas_height; |
return VP8_STATUS_OK; // Just return features from VP8X header. |
@@ -356,6 +358,10 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
return VP8_STATUS_BITSTREAM_ERROR; |
} |
+ if (format != NULL && !(animation_present || fragments_present)) { |
+ *format = hdrs.is_lossless ? 2 : 1; |
+ } |
+ |
if (!hdrs.is_lossless) { |
if (data_size < VP8_FRAME_HEADER_SIZE) { |
return VP8_STATUS_NOT_ENOUGH_DATA; |
@@ -374,7 +380,7 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data, |
return VP8_STATUS_BITSTREAM_ERROR; |
} |
} |
- // Validates image size coherency. TODO(urvang): what about FRGM? |
+ // Validates image size coherency. |
if (found_vp8x) { |
if (canvas_width != image_width || canvas_height != image_height) { |
return VP8_STATUS_BITSTREAM_ERROR; |
@@ -402,7 +408,8 @@ VP8StatusCode WebPParseHeaders(WebPHeaderStructure* const headers) { |
assert(headers != NULL); |
// fill out headers, ignore width/height/has_alpha. |
status = ParseHeadersInternal(headers->data, headers->data_size, |
- NULL, NULL, NULL, &has_animation, headers); |
+ NULL, NULL, NULL, &has_animation, |
+ NULL, headers); |
if (status == VP8_STATUS_OK || status == VP8_STATUS_NOT_ENOUGH_DATA) { |
// TODO(jzern): full support of animation frames will require API additions. |
if (has_animation) { |
@@ -416,7 +423,7 @@ VP8StatusCode WebPParseHeaders(WebPHeaderStructure* const headers) { |
// WebPDecParams |
void WebPResetDecParams(WebPDecParams* const params) { |
- if (params) { |
+ if (params != NULL) { |
memset(params, 0, sizeof(*params)); |
} |
} |
@@ -449,11 +456,6 @@ static VP8StatusCode DecodeInto(const uint8_t* const data, size_t data_size, |
if (dec == NULL) { |
return VP8_STATUS_OUT_OF_MEMORY; |
} |
-#ifdef WEBP_USE_THREAD |
- dec->use_threads_ = params->options && (params->options->use_threads > 0); |
-#else |
- dec->use_threads_ = 0; |
-#endif |
dec->alpha_data_ = headers.alpha_data; |
dec->alpha_data_size_ = headers.alpha_data_size; |
@@ -465,6 +467,10 @@ static VP8StatusCode DecodeInto(const uint8_t* const data, size_t data_size, |
status = WebPAllocateDecBuffer(io.width, io.height, params->options, |
params->output); |
if (status == VP8_STATUS_OK) { // Decode |
+ // This change must be done before calling VP8Decode() |
+ dec->mt_method_ = VP8GetThreadMethod(params->options, &headers, |
+ io.width, io.height); |
+ VP8InitDithering(params->options, dec); |
if (!VP8Decode(dec, &io)) { |
status = dec->status_; |
} |
@@ -651,7 +657,6 @@ uint8_t* WebPDecodeYUV(const uint8_t* data, size_t data_size, |
static void DefaultFeatures(WebPBitstreamFeatures* const features) { |
assert(features != NULL); |
memset(features, 0, sizeof(*features)); |
- features->bitstream_version = 0; |
} |
static VP8StatusCode GetFeatures(const uint8_t* const data, size_t data_size, |
@@ -665,7 +670,7 @@ static VP8StatusCode GetFeatures(const uint8_t* const data, size_t data_size, |
return ParseHeadersInternal(data, data_size, |
&features->width, &features->height, |
&features->has_alpha, &features->has_animation, |
- NULL); |
+ &features->format, NULL); |
} |
//------------------------------------------------------------------------------ |
@@ -803,6 +808,3 @@ int WebPIoInitFromOptions(const WebPDecoderOptions* const options, |
//------------------------------------------------------------------------------ |
-#if defined(__cplusplus) || defined(c_plusplus) |
-} // extern "C" |
-#endif |