Closed
Bug 1501554
Opened 6 years ago
Closed 5 years ago
<img> should fire ProgressEvent for CORS-same-origin images
Categories
(Core :: DOM: Core & HTML, defect, P3)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: lcz970, Unassigned)
Details
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:51.0) Gecko/20160105164030 Firefox/62.0 Steps to reproduce: <img src="example.png"> <script> img = document.querySelector('img'); img.addEventListener('progress', function (event) { console.log(event); }); </script> Actual results: No event is triggered / logged Expected results: According to HTML spec[1], ProgressEvent should be fired for CORS-same-origin[2] images. I'm not sure if it is because this spec is new or something, none vendor has implemented the ProgressEvent on <img> element. Though loadstart / loadend works on Firefox 62, and only loadend works on Chrome 69 [1]: https://html.spec.whatwg.org/#event-progress [2]: https://html.spec.whatwg.org/#cors-same-origin
Also, MDN lists <img> under ProgressEvent, which is kind of misleading for now. https://developer.mozilla.org/en-US/docs/Web/API/ProgressEvent
Updated•6 years ago
|
Component: Untriaged → Networking: HTTP
Product: Firefox → Core
Updated•6 years ago
|
Component: Networking: HTTP → General
Updated•6 years ago
|
Component: General → DOM
Comment 2•6 years ago
|
||
Christoph, do you know about this?
Component: DOM → DOM: Security
Flags: needinfo?(ckerschb)
Priority: -- → P2
Comment 3•6 years ago
|
||
(In reply to esterTion from comment #0) > According to HTML spec[1], ProgressEvent should be fired for > CORS-same-origin[2] images. > I'm not sure if it is because this spec is new or something, none vendor has > implemented the ProgressEvent on <img> element. > Though loadstart / loadend works on Firefox 62, and only loadend works on > Chrome 69 I haven't investigated, but presumably this is not CORS related, but potentially (as the author indicates) Firefox does not support the progressEvent at all as of now. I don't know who might know the answer for sure, probably someone in the DOM team is better suited to answer. I'll ask Anne for now since he might be able to help out.
Flags: needinfo?(ckerschb) → needinfo?(annevk)
Comment 4•6 years ago
|
||
Yeah, pretty sure that's correct (progress events would only be supported for CORS images, but that information is already available in DOM as it's also needed for canvas and such). There's also https://github.com/whatwg/html/issues/1915 about this text not being as specific as it needs to be.
Component: DOM: Security → DOM
Flags: needinfo?(annevk)
Priority: P2 → P3
Comment 5•6 years ago
|
||
This seems like a case of the HTML spec making up something without talking to any browsers, then not telling them about it. Such things were done back in the day. It's not clear to me whether this is desirable to implement, or even what it would mean given async decoding.
Assignee | ||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
Comment 6•5 years ago
|
||
We have some of this, but I think we should remove it as per bug 1574487.
Status: UNCONFIRMED → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•