1308126 - Inconsistent behavior of image loading between Firefox and Chrome/Edge

Status: RESOLVED DUPLICATE of bug 1308069

(Core :: DOM: Core & HTML, defect, P2)

Description

[Alice0775 White]

Attachment: testcase

Something timing problem.

img.src="";
img.src="valid image url";

An alert box pops up on Nigitly52.0a1, Aurora51.0a2 and IE11.
No error on Chrome and Edge.


Steps to reproduce:
1. Open attached testcase

Actual Results:
 an alert pops up

Expected Results:
 I'm not sure either

Comment 1

[Mike Taylor [:miketaylr]]

Seems like Chrome clears the pending error event if the src changed before it fired:

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/loader/ImageLoader.cpp?sq=package:chromium&rcl=1475738528&l=189

(see ImageLoader::setImageWithoutConsideringPendingLoadEvent)

Comment 2

[Edgar Chen [:edgar]]

Currently we load image synchronously if image is in non-responsive mode [1].
Besides the comment 0, If we set img.src to "" multiple times in succession, error event will be fired multiple times. 
This is not in line with spec [2]. We need to fix this.

[1] See https://dxr.mozilla.org/mozilla-central/source/dom/html/HTMLImageElement.cpp#525 and bug 1076583.
[2] See step 7 of https://html.spec.whatwg.org/multipage/embedded-content.html#update-the-image-data.

Comment 3

[Alice0775 White]

FYI, Bug 1308069 fixed the problem in case the testcase(comment#0).

Comment 4

[homm86]

More detailed explanation and working test case in this bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1321300

Comment 5

[Julien Cristau [:jcristau]]

Mass wontfix for bugs affecting firefox 52.

Comment 6

[Jonathan Watt [:jwatt]]

Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)

Comment 7

[Edgar Chen [:edgar]]

Per comment 3.