Closed
Bug 1160200
Opened 9 years ago
Closed 8 years ago
Add image/apng MIME type so it can be used with `picture` type switching.
Categories
(Core :: General, defect)
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: david, Assigned: marcosc)
References
(Depends on 1 open bug, )
Details
(Keywords: dev-doc-needed, Whiteboard: good first bug)
Attachments
(1 file, 9 obsolete files)
10.71 KB,
patch
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.135 Safari/537.36 Steps to reproduce: ```` <picture> <source type="image/x-apng" srcset="animated.apng" /> <img src="static.png" /> </picture> ```` (…or `video/png`, `image/apng`, etc.) Actual results: `static.png` was displayed Expected results: `animated.apng` should have been displayed.
Reporter | ||
Comment 1•9 years ago
|
||
Firefox doesn’t recognize any MIME types for APNG that are distinct from those for regular PNG. Since it doesn’t have a unique MIME type, APNG can’t be used reliably as an alternate format in `picture` file type switching. This becomes extra problematic if an animation is required. Given: <picture> <source type="image/png" srcset="animated.apng" /> <img src="animated.gif" /> </picture> …a non-APNG supporting browser will show the APNG as a static image, rather than showing the fallback GIF.
Assignee | ||
Comment 2•9 years ago
|
||
is "type="image/x-apng" a valid type? I thought they were both image/png? I think the spec should define it's MIME type first. It doesn't seem to do that.
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(pavlov)
Reporter | ||
Comment 3•9 years ago
|
||
Nah, `image/x-apng` is not valid, I just using it as an example. Sorry that wasn’t clear. My understanding is that `x-` MIME types are specifically for non-standard things. Really, any MIME type other than `image/png` would be great. And I agree, ideally this would be defined in the spec.
Comment 4•9 years ago
|
||
FWIW this would just-work with <picture> if there were an APNG specific MIME type registered.
Assignee | ||
Comment 5•9 years ago
|
||
@johns, where do we have the registrations? Could easily add that.
Updated•9 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(pavlov)
Reporter | ||
Comment 6•9 years ago
|
||
Attachment #8600216 -
Flags: review?(john)
Comment 7•9 years ago
|
||
Comment on attachment 8600216 [details] [diff] [review] Add image/apng MIME type Review of attachment 8600216 [details] [diff] [review]: ----------------------------------------------------------------- This looks fine with nits below, but since we're adding a new MIME type to imagelib, I think :seth will need to do the review here ::: image/src/imgLoader.cpp @@ +1354,4 @@ > mAcceptHeader = accept; > } else { > mAcceptHeader = > + IMAGE_PNG "," IMAGE_APNG "," IMAGE_WILDCARD ";q=0.8," ANY_WILDCARD ";q=0.5"; I'm not sure if we want to include apng here, but I'm not sure what contexts this header is used in. ::: uriloader/exthandler/nsExternalHelperAppService.cpp @@ +512,4 @@ > { IMAGE_ICO, "ico,cur", "ICO Image" }, > { IMAGE_JPEG, "jpeg,jpg,jfif,pjpeg,pjp", "JPEG Image" }, > { IMAGE_PNG, "png", "PNG Image" }, > + { IMAGE_APNG, "apng", "APNG Image" }, Is the ".apng" extension something from the spec?
Attachment #8600216 -
Flags: review?(seth)
Attachment #8600216 -
Flags: review?(john)
Attachment #8600216 -
Flags: feedback+
Updated•9 years ago
|
Assignee: nobody → david
Component: Untriaged → General
Product: Firefox → Core
Reporter | ||
Comment 8•9 years ago
|
||
(In reply to John Schoenick [:johns] from comment #7) > > ::: image/src/imgLoader.cpp > @@ +1354,4 @@ > > mAcceptHeader = accept; > > } else { > > mAcceptHeader = > > + IMAGE_PNG "," IMAGE_APNG "," IMAGE_WILDCARD ";q=0.8," ANY_WILDCARD ";q=0.5"; > > I'm not sure if we want to include apng here, but I'm not sure what contexts > this header is used in. I wasn’t sure about that bit either. Happy to remove if necessary. > > ::: uriloader/exthandler/nsExternalHelperAppService.cpp > @@ +512,4 @@ > > { IMAGE_ICO, "ico,cur", "ICO Image" }, > > { IMAGE_JPEG, "jpeg,jpg,jfif,pjpeg,pjp", "JPEG Image" }, > > { IMAGE_PNG, "png", "PNG Image" }, > > + { IMAGE_APNG, "apng", "APNG Image" }, > > Is the ".apng" extension something from the spec? Nope, the spec [1] doesn’t mention a file extension. The PNG spec [2] that is referenced by APNG states that “On systems where file names customarily include an extension signifying file type, the extension ‘.png’ is recommended for PNG files,” but this isn’t a requirement. The ‘.apng’ extension is mentioned (without sourcing) on Wikipedia [3] and the File Formats Wiki [4], though. If people use either of those as references, they might create files with a ‘.apng’ extension, so I figured it might be good to register it here. [1] https://wiki.mozilla.org/APNG_Specification [2] http://www.w3.org/TR/PNG/ [3] http://en.wikipedia.org/wiki/APNG [4] http://fileformats.wikia.com/wiki/Animated_Portable_Network_Graphics
Reporter | ||
Comment 9•9 years ago
|
||
Contains relevant discussion: https://bugzilla.mozilla.org/show_bug.cgi?id=257263
Comment 10•9 years ago
|
||
(In reply to Dave Newton from comment #9) > Contains relevant discussion: > https://bugzilla.mozilla.org/show_bug.cgi?id=257263 Yeah, that bug is excellent reading. Kinda sad that this issue could've been avoided years ago. =\ I strongly support fixing the problem in this bug, but some of the details here need a bit of thought. In particular, we can't just use "image/apng" without following the appropriate processes - that kind of MIME type needs to be registered, and AFAIK nobody has done that. As I understand it, we _could_ use "image/x-apng", or perhaps "video/x-png", right away. Those don't require registration. We can then look into registering the non-"x-" variants. I'm not experienced with the MIME type registration process, though, so I may not have the details correct here. Glenn, is that correct? What do you think we should do here?
Flags: needinfo?(glennrp+bmo)
Comment 11•9 years ago
|
||
I have not looked at IANA registration requirements since about 15 years ago. Unless things have changed, video/x-apng would work ....checking..... The rules are in RFC 6838 (Jan 2013) now, http://tools.ietf.org/html/rfc6838 which strongly discourages the use of "x-" MIME types. I think MIME type "vnd.mozilla.video.apng" would work; both the vendor name "mozilla" (if not already registered) and the type vnd.mozilla.video.apng (and/or vnd.mozilla.video.png, if you still insist on hijacking the PNG suffix) would have to be registered.
Flags: needinfo?(glennrp+bmo)
Comment 12•9 years ago
|
||
Browsing IANA stuff (www.iana.org/assignments/media-types/media-types.xhtml) a little more, I found that "mozilla" already exists as a "vendor" ID, so vnd.mozilla is OK to use. Also, I had the MIME type facet order slightly wrong; should be video/vnd.mozilla.apng instead of vnd.mozilla.video.apng.
Reporter | ||
Comment 13•9 years ago
|
||
I’d be strongly in favour of registering this as "video/vnd.mozilla.apng" (the registration form is at http://www.iana.org/form/media-types). My gut says that this should be accompanied by a change to the spec, though; Stuart, is that something that would have support from the APNG spec authors? If not, is accepting "video/x-apng" in `picture` the only way to resolve this?
Reporter | ||
Comment 14•9 years ago
|
||
updated patch to use video/vnd.mozilla.apng MIME type
Attachment #8600216 -
Attachment is obsolete: true
Attachment #8600216 -
Flags: review?(seth)
Assignee | ||
Updated•9 years ago
|
Attachment #8601215 -
Flags: review?(seth)
Comment 15•9 years ago
|
||
Stuart, I'd love to hear your thoughts on comment 13.
Flags: needinfo?(pavlov)
Assignee | ||
Comment 16•9 years ago
|
||
Comment on attachment 8601215 [details] [diff] [review] updated to video/vnd.mozilla.apng Review of attachment 8601215 [details] [diff] [review]: ----------------------------------------------------------------- Few nits (tho fine if other tests already use xhtml and similar spacing), but lgtm. ::: layout/reftests/apng-mime/expected.html @@ +1,3 @@ > +<!doctype html> > +<html> > + <title>apng expected</title> nit: why so much whitespace? you can probably also drop the html tags. @@ +1,4 @@ > +<!doctype html> > +<html> > + <title>apng expected</title> > + <img src="animated.apng" /> nit: " />" (not needed) ::: layout/reftests/apng-mime/test.html @@ +1,5 @@ > +<!doctype html> > +<html> > + <title>apng test</title> > + <picture> > + <source type="video/vnd.mozilla.apng" srcset="animated.apng" /> nit: more xhtml syntax.
Attachment #8601215 -
Flags: review?(seth) → review+
Reporter | ||
Comment 17•9 years ago
|
||
Attachment #8601215 -
Attachment is obsolete: true
Attachment #8612875 -
Flags: review+
Assignee | ||
Comment 18•9 years ago
|
||
Comment on attachment 8612875 [details] [diff] [review] Bug 1160200 - Add image/apng MIME type. Updates re: review. Review of attachment 8612875 [details] [diff] [review]: ----------------------------------------------------------------- Nit: please reintroduce whitespace in nsMimeTypes.h.
Reporter | ||
Comment 19•9 years ago
|
||
Attachment #8612875 -
Attachment is obsolete: true
Attachment #8612888 -
Flags: review?(mcaceres)
Assignee | ||
Updated•9 years ago
|
Attachment #8612888 -
Flags: review?(mcaceres) → review+
Assignee | ||
Comment 20•9 years ago
|
||
Registered: On May 29, 2015 at 2:10:09 PM, IANA MIME Requests via RT (iana-mime@iana.org) wrote: > To whom it may concern: > > This is an automatically generated message to notify you that we have > received your request, and it has been recorded in our ticketing > system with a reference number of 826172. To check the status > of your request, please see: > > https://tools.iana.org/ticket-status/app > > If you have any problems accessing this page, please contact > iana@iana.org. > > There is no need to reply to this message right now. IANA staff will > review your message shortly. > > If this message is in reply to a previously submitted ticket, it is > possible that the previous ticket has been marked as closed. As we > review this ticket, we will also review previous correspondence and > take appropriate action. > > To expedite processing, and ensure our staff can view the full history > of this request, please make sure you include the follow exact text in > the subject line of all future correspondence on this issue: > > [IANA #826172] > > You can also simply reply to this message, as this tag is already in > the subject line. > > Thank you, > > The Internet Assigned Numbers Authority > iana-mime@iana.org > > ------------------------------------------------------------------------- > > Name : Marcos Caceres > > Email : mcaceres@mozilla.com > > MIME media type name : Application > > MIME subtype name : Vendor Tree - vnd.mozilla.apng > > Required parameters : None > > Optional parameters : > None > > Encoding considerations : 7bit > > > Security considerations : > Same as PNG. > > Interoperability considerations : > An APNG stream is a normal PNG stream as defined in the PNG Specification, with three additional > chunk types describing the animation and providing additional frame data. > > To be recognized as an APNG, an `acTL` chunk must appear in the stream before any `IDAT` > chunks. The `acTL` structure is described below. > > Published specification : > https://wiki.mozilla.org/APNG_Specification > > Applications which use this media : > Web browsers, image processors > > Fragment identifier considerations : > none > > Restrictions on usage : > none > > Provisional registration? (standards tree only) : > Yes. > > > Additional information : > > 1. Deprecated alias names for this type : None > 2. Magic number(s) : 137 80 78 71 13 10 26 10 > 3. File extension(s) : apng, png > 4. Macintosh file type code : None > 5. Object Identifiers: None > > See: https://bugzilla.mozilla.org/show_bug.cgi?id=1160200 > > Person to contact for further information : > > 1. Name : Stuart Parmenter > 2. Email : mcaceres@mozilla.com > > Intended usage : Common > It is intended to be a replacement for simple animated images that have traditionally > used the GIF format, while adding support for 24-bit images and 8-bit transparency. > APNG is a simpler alternative to MNG, providing a spec suitable for the most common usage > of animated images on the Internet. > > Author/Change controller : Stuart Parmenter > Vladimir Vukicevic > Andrew Smith > > > > >
Assignee | ||
Comment 21•9 years ago
|
||
Ooops, wrong email in contact info. Will get that fixed.
Assignee | ||
Comment 22•9 years ago
|
||
Ok, contact is getting fixed by IANA. Also, the Encoding considerations is a bug in their form. Also fixed. What's next step here? Do we need a super review or can I check this in?
Assignee | ||
Comment 23•9 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3fd7d6a116ed
Assignee | ||
Updated•9 years ago
|
Summary: APNG can't be used with `picture` type switching → Add image/apng MIME type so it can be used with `picture` type switching.
Assignee | ||
Comment 24•9 years ago
|
||
Comment on attachment 8612888 [details] [diff] [review] Bug 1160200 - Add image/apng MIME type. Whitespace Talked to Jeff; he will super review.
Flags: needinfo?(pavlov)
Attachment #8612888 -
Flags: superreview?(jmuizelaar)
Comment 25•9 years ago
|
||
Comment on attachment 8612888 [details] [diff] [review] Bug 1160200 - Add image/apng MIME type. Whitespace Review of attachment 8612888 [details] [diff] [review]: ----------------------------------------------------------------- ::: image/imgLoader.cpp @@ +1315,1 @@ > } If we use image/vnd.mozilla.apng we don't need to increase the size of our accept header. Using video for apng but gif seems very weird to me.
Attachment #8612888 -
Flags: superreview?(jmuizelaar) → superreview-
Comment 26•9 years ago
|
||
Things got registered. http://www.iana.org/assignments/media-types/image/vnd.mozilla.apng
Assignee | ||
Comment 27•9 years ago
|
||
"so i guess you could say things are getting pretty serious" :) Dave, let's get this done. See Jeff's comments above.
Flags: needinfo?(david)
Comment 28•9 years ago
|
||
Why did we not register "image/apng"? Given that other browsers are interested in implementing this using a vendor proprietary MIME type seems weird.
Comment 29•9 years ago
|
||
+1 for a vendor neutral MIME type. Also, are there plans to expose said MIME type to `Accept` headers? Ideally, we would want developers to be able to choose between a client-side, <picture> based MIME fallback and a server-side based content negotiation.
Comment 30•9 years ago
|
||
I attempted to register "video/png" in April 2007 but was shot down by IANA. Earlier I had attempted to get "image/apng" in the APNG spec but was shot down by Vlad and Stuart (with that change there's a non-zero chance that the APNG chunks would have been approved by the PNG group). IANA won't approve "image/apng" if it's not in the APNG spec or in the PNG spec, but there's zero chance of either of those happening now.
There wasn't a strong reason to have a separate MIME type back then, since it was just PNG... I have no problem with "image/apng" given <picture> and fallbacks. We can add "image/apng" to the APNG spec if that would let the IANA officially assign it?
Comment 32•9 years ago
|
||
The assignment by IANA wouldn't be automatic. But if we put it in the spec they might at least be willing to review it.
Assignee | ||
Comment 33•9 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=352a6c9cbdb6
Assignee: david → mcaceres
Attachment #8612888 -
Attachment is obsolete: true
Flags: needinfo?(david)
Attachment #8666130 -
Flags: superreview?(jmuizelaar)
Updated•9 years ago
|
Attachment #8666130 -
Flags: superreview?(jmuizelaar) → superreview+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 34•9 years ago
|
||
Jeff, you don't think we should change the MIME type to be vendor neutral? I updated the APNG specification to add the vendor neutral type, FWIW.
Flags: needinfo?(jmuizelaar)
Comment 35•9 years ago
|
||
(In reply to Anne (:annevk) from comment #34) > Jeff, you don't think we should change the MIME type to be vendor neutral? I > updated the APNG specification to add the vendor neutral type, FWIW. The vendor neutral MIME type has not been accepted by the IANA so I'm not sure how much of a choice we have.
Flags: needinfo?(jmuizelaar)
Comment 36•9 years ago
|
||
Jeff, according to Glenn the only thing blocking that was a lack of mention of image/apng in the specification, which has been rectified. Also, I don't think we should let IANA stop us from doing the right thing here, if it comes to that.
Comment 37•9 years ago
|
||
You added to the APNG spec, "APNG can be identified using the image/apng MIME type." Anne, That's jumping the gun. It should say, similar to what's in the original PNG spec RFC-2083, "This specification defines a proposed Internet Media Type image/apng." (but actually according to RFC-6838 it should be proposed to be "video/apng"; see paragraphs 4.2.2 and 4.2.4). And it should say that it defines Internet Media Type image/vnd.mozilla.apng as well, and should mention that the file can be identified with file extensions png or apng, as was stated in the IANA registration of image/vnd.mozilla.apng. In the past I would have suggested "video/x-apng", similar to what we did with image/x-png; however, that's deprecated now by RFC-6648. What's left is "image/vnd.mozilla.apng" that has been registered.
Comment 38•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b4f64c940524
Keywords: checkin-needed
Comment 39•9 years ago
|
||
Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/7031e812cdfc
Comment 40•9 years ago
|
||
I see IANA says image/vnd.mozilla.apng But the patch says video/vnd.mozilla.apng Please check.
Assignee | ||
Comment 41•9 years ago
|
||
(In reply to Max Stepin from comment #40) > I see IANA says image/vnd.mozilla.apng > But the patch says video/vnd.mozilla.apng > Please check. D'oh. Fixing.
Assignee | ||
Comment 42•9 years ago
|
||
Attachment #8666130 -
Attachment is obsolete: true
Assignee | ||
Comment 43•9 years ago
|
||
Try again... https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e1575b774b2
Assignee | ||
Comment 44•8 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1fd2bfeb830d
Attachment #8668052 -
Attachment is obsolete: true
Assignee | ||
Comment 46•8 years ago
|
||
Try again, but with all tests this time: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2b6113a8493c
Comment 47•8 years ago
|
||
(In reply to Glenn Randers-Pehrson from comment #37) > What's left is "image/vnd.mozilla.apng" that has been registered. FWIW I strongly disagree with this and the attached patch. We know that other vendors have implemented and shipped APNG and more are planning on doing so. It's a part of the web. It shouldn't matter what registries and standards say as they can be updated. We should ship what we believe is right, and that is that APNG is a standard and not a proprietary format belonging to Mozilla.
Assignee | ||
Comment 48•8 years ago
|
||
(In reply to Anne (:annevk) from comment #47) > (In reply to Glenn Randers-Pehrson from comment #37) > > What's left is "image/vnd.mozilla.apng" that has been registered. > > FWIW I strongly disagree with this and the attached patch. We know that > other vendors have implemented and shipped APNG and more are planning on > doing so. It's a part of the web. It shouldn't matter what registries and > standards say as they can be updated. We should ship what we believe is > right, and that is that APNG is a standard and not a proprietary format > belonging to Mozilla. Happy to change it to -webkit/apng ... I mean image/apng.
Assignee | ||
Comment 49•8 years ago
|
||
Attachment #8742655 -
Attachment is obsolete: true
Comment 50•8 years ago
|
||
Better make image/x-apng.
Comment 51•8 years ago
|
||
No, x- is no longer to be used. We don't want another application/x-www-form-urlencoded or X-Frame-Options. It doesn't stand the test of time.
Assignee | ||
Comment 52•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=22a66fff00b3
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 53•8 years ago
|
||
(In reply to Anne (:annevk) from comment #51) > No, x- is no longer to be used. We don't want another > application/x-www-form-urlencoded or X-Frame-Options. It doesn't stand the > test of time. Agree. Checking in as "image/apng"... we can deal with IANA later.
Comment 54•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/68442febc36c
Keywords: checkin-needed
Comment 55•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/68442febc36c
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Blocks: 1266817
Updated•8 years ago
|
Comment 56•8 years ago
|
||
Backed this out for almost permafailing added test apng-mime/test.html Android 4.3 API15+ opt Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/1f1bf431eb48
Flags: needinfo?(mcaceres)
Assignee | ||
Comment 57•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=08423e6174bf
Assignee | ||
Comment 58•8 years ago
|
||
Seeing if I can reproduce on Android: https://treeherder.mozilla.org/#/jobs?repo=try&revision=490054d3582a
Flags: needinfo?(mcaceres)
Assignee | ||
Comment 59•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ec022d5e6f98
Assignee | ||
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 60•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7c9e90367007
Assignee | ||
Comment 61•8 years ago
|
||
So, this doesn't work anymore. I don't know what's changed by `<source>` is not being picked up by <img> for "image/apng". it does "work" (incorrectly) when the test is set discriminate on jpeg: ``` <picture> <source type="image/jpg" srcset="animated.apng"> <img> </picture> ``` But when it's switched back to type="image/apng", there is only white displayed. The same with: ``` <picture> <source type="image/jpg" srcset="animated.apng"> <img src="static.png"> </picture> ``` Discrimination on type does seem to be working because: ``` <picture> <source type="invalid/garbage" srcset="animated.apng"> <img src="static.png"> </picture> ``` displays "static.png". @johns, any suggestions?
Flags: needinfo?(john)
Assignee | ||
Comment 62•8 years ago
|
||
Argh... nm... recompiled and it started working again
Flags: needinfo?(john)
Assignee | ||
Comment 63•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e701f1477dd
Assignee | ||
Comment 64•8 years ago
|
||
Yeah, it's borked on Android. No idea how to proceed. Happy for someone else to take it over from here.
Assignee | ||
Updated•8 years ago
|
Assignee: mcaceres → nobody
Assignee | ||
Updated•8 years ago
|
Whiteboard: good first bug
Comment 65•8 years ago
|
||
(In reply to Marcos Caceres [:marcosc] from comment #64) > Yeah, it's borked on Android. No idea how to proceed. Happy for someone else > to take it over from here. Looking at this test: > <source type="image/apng" srcset="animated.apng"> Wouldn't the reftest fail depending on the number of frames animated.apng manages to render before the harness considers the document complete? I'm not sure reftests will work as expected for animations at all. Converting this to a mochitest that just asserts that img.currentSrc == animated.apng might be all that is necessary
Assignee | ||
Comment 66•8 years ago
|
||
(In reply to John Schoenick [:johns] from comment #65) > (In reply to Marcos Caceres [:marcosc] from comment #64) > > Yeah, it's borked on Android. No idea how to proceed. Happy for someone else > > to take it over from here. > > Looking at this test: > > <source type="image/apng" srcset="animated.apng"> > > Wouldn't the reftest fail depending on the number of frames animated.apng > manages to render before the harness considers the document complete? I'm > not sure reftests will work as expected for animations at all. Yeah, that's probably what's happening. > Converting this to a mochitest that just asserts that img.currentSrc == > animated.apng might be all that is necessary Yeah, that should be good enough. I'll convert it.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mcaceres
Assignee | ||
Comment 67•8 years ago
|
||
So, I'm making redoing the apng as a 2 frame animation that only shows green. Right now, it was showing green and then red, which would be subject to random fails. I'll also add the mochitest.
Assignee | ||
Comment 68•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e709e3440030
Assignee | ||
Comment 69•8 years ago
|
||
Added a couple of simple (mochi) tests. Made the apng animate over two frames that are the same (green).
Attachment #8743680 -
Attachment is obsolete: true
Attachment #8777685 -
Flags: review?(john)
Comment 70•8 years ago
|
||
Comment on attachment 8777685 [details] [diff] [review] Adds mochitest Review of attachment 8777685 [details] [diff] [review]: ----------------------------------------------------------------- r=me on mochitest addition (Assuming this is carrying over jmuizelaar's sr+ since I'm not a DOM peer)
Attachment #8777685 -
Flags: review?(john) → review+
Assignee | ||
Comment 71•8 years ago
|
||
Carrying over r+ from Jeff for the c++ bits.
Attachment #8777685 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 72•8 years ago
|
||
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f29c9642fbca APNG can't be used with type switching. r=mcaceres, r=jrmuizel, r=johns
Keywords: checkin-needed
Comment 73•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f29c9642fbca
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Comment 74•8 years ago
|
||
Typo in test_picture_apng.html <srouce>
Updated•8 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 75•8 years ago
|
||
(In reply to Max Stepin from comment #74) > Typo in test_picture_apng.html > > <srouce> Patch: https://bugzilla.mozilla.org/show_bug.cgi?id=1295434
Updated•2 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•