|WEBPImageDecoder.cpp uses unreleased libwebp features|
|Project Member Reported by email@example.com, Sep 9 2013||Back to list|
Here's what I get when trying to compile against libwebp-0.3.1: ../../third_party/WebKit/Source/core/platform/image-decoders/webp/WEBPImageDecoder.cpp: In member function ‘bool WebCore::WEBPImageDecoder::updateDemuxer()’: ../../third_party/WebKit/Source/core/platform/image-decoders/webp/WEBPImageDecoder.cpp:248:69: error: ‘WebPIterator’ has no member named ‘blend_method’ ../../third_party/WebKit/Source/core/platform/image-decoders/webp/WEBPImageDecoder.cpp:248:85: error: ‘WEBP_MUX_BLEND’ was not declared in this scope ../../third_party/WebKit/Source/core/platform/image-decoders/webp/WEBPImageDecoder.cpp:256:109: error: ‘WebPIterator’ has no member named ‘has_alpha’ I think this was introduced in https://chromiumcodereview.appspot.com/22802020 as a cherry-pick from upstream git. It's obviously fine to let this bake for a while in our repo, but ideally libwebp should make a release with this code before M31 hits stable. It seems I can't assign it to Urvang, so trying Frank first - could you help route this to the right people?
Sep 10 2013,
Yes, this is known, and due to the cherry-picked changes as Pawel mentioned. We will release the next libwebp version before M31 hits stable (which seems to be 12th Nov) as suggested. Feel free to mark this as M31 releaseblock-stable.
Sep 13 2013,
Oct 14 2013,
Oct 14 2013,
Oct 22 2013,
We'll make an attempt to make a release within the same time frame of the stable release, but this isn't a release blocker.
Nov 8 2013,
Moving all non essential bugs to the next Milestone.
Dec 3 2013,
Sure would be nice if this got made into a release. :/
Dec 17 2013,
This issue has already been moved once and is lower than Priority 1,therefore removing mstone.
Jan 2 2014,
Looks like this is fixed with libwebp-0.4.0. I've tested it and building with system libwebp seems to work with that version. Thanks!
Jan 7 2014,
As noted the availability of libwebp-0.4.0 addresses this issue.
|► Sign in to add a comment|