New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Compat



Sign in to add a comment

WebP can't play animation only once

Reported by gn00742...@gmail.com, Sep 22 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36

Example URL:

Steps to reproduce the problem:
1.  use webpmux make a webp animation image and set loop to 1
2. webp loop twice
3. use webpmux make a webp animation image and set loop to 0
4. webp loop infinite

What is the expected behavior?

What went wrong?
It's no way to play webp once on current version

Does it occur on multiple sites: N/A

Is it a problem with a plugin? N/A 

Did this work before? N/A 

Does this work in other browsers? N/A 

Chrome version: 53.0.2785.116  Channel: stable
OS Version: OS X 10.11.6
Flash Version: Shockwave Flash 23.0 r0
 
Components: Blink>Image
Components: -Blink>Image Internals>Media>Codecs
Labels: Needs-Feedback
I'm inferring that the looping setting is part of the content, in which case this is a codec issue, I think.

Send back to Blink>Image if it turns out we've decoding the right info but then ignoring it.

gn00742754@, can you provide example videos? Does it matter how the video is loaded or embedded in the page?

Comment 3 Deleted

Hi, schenny,

you can see this link for more detail:

https://bugs.chromium.org/p/chromium/issues/detail?id=592735

webp was play once when loop set 1 before this commit.

That would cause the behavior diff with GIF, so this commit is to match existing behavior for GIF.

But according this document:

https://developers.google.com/speed/webp/docs/webpmux

webp animation default will set to 0 which will cause loop infinite

and set 1 will loop twice.

There is no way to make a webp animation loop only once.
Cc: dalecur...@chromium.org
gn00742754@gmail.com, can you please provide a repro image?
Dale, as questioned in #2, do you think this could be a codec issue? if not, we need give back to blink>image.
Attachment is a example image,

This image loop twice in current chrome version(my version is 53.0.2785.143),

But the same image open in old version chrome(I use 48.0.2564.109) loop only once!

You can use any older version before this commit

https://chromium.googlesource.com/chromium/src.git/+/7b7ed0e2c9ce2de3f23102d1eb4c9625a9d73f30

animated.webp
5.6 MB View Download
Project Member

Comment 7 by sheriffbot@chromium.org, Oct 13 2016

Labels: -Needs-Feedback Needs-Review
Owner: schenney@chromium.org
Thank you for providing more feedback. Adding requester "schenney@chromium.org" for another review and adding "Needs-Review" label for tracking.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: schenney@chromium.org
Labels: -Needs-Review
Owner: ----
Status: Available (was: Unconfirmed)
We'll leave this bug open, but most of the discussion has happened on https://bugs.chromium.org/p/chromium/issues/detail?id=592735.

Progress is currently blocked on Webp bug https://bugs.chromium.org/p/webp/issues/detail?id=309
Cc: pkasting@chromium.org noel@chromium.org

Comment 10 by noel@chromium.org, Sep 6 2017

Cc: jzern@chromium.org
Project Member

Comment 11 by bugdroid1@chromium.org, Sep 20 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ed8464fa995759af68440c044c9605aabb1e1b2c

commit ed8464fa995759af68440c044c9605aabb1e1b2c
Author: James Zern <jzern@google.com>
Date: Wed Sep 20 06:45:23 2017

WebP animation: allow images to loop once

In  crbug.com/592735  WebP was made to match GIF behavior, but as the loop
count is mandatory left it without a way to signal 1 repetition. This
restores the previous behavior.

BUG= 649264 

Change-Id: I4bc00c439af986c37fa44c65d67c9802d332e610
Reviewed-on: https://chromium-review.googlesource.com/668169
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: James Zern <jzern@google.com>
Cr-Commit-Position: refs/heads/master@{#503072}
[modify] https://crrev.com/ed8464fa995759af68440c044c9605aabb1e1b2c/third_party/WebKit/Source/platform/image-decoders/webp/WEBPImageDecoder.cpp
[modify] https://crrev.com/ed8464fa995759af68440c044c9605aabb1e1b2c/third_party/WebKit/Source/platform/image-decoders/webp/WEBPImageDecoderTest.cpp

Comment 12 by jzern@chromium.org, Sep 21 2017

Owner: jzern@chromium.org
Status: Fixed (was: Available)

Sign in to add a comment