Skip to content

Commit

Permalink
Update wpt test to match the new spec where loadend isn't anymore
Browse files Browse the repository at this point in the history
  • Loading branch information
Eijebong committed Sep 1, 2019
1 parent eb9c9fb commit 84898bc
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
2 changes: 1 addition & 1 deletion tests/wpt/metadata/MANIFEST.json
Expand Up @@ -640719,7 +640719,7 @@
"support"
],
"html/semantics/embedded-content/the-img-element/invalid-src.html": [
"c3c57ee3c0a05019be4db6650eeefdd53648401e",
"37ea8ce754a3feaa3e305f10899632d231c92363",
"testharness"
],
"html/semantics/embedded-content/the-img-element/ismap/img-ismap-coordinates-iframe-after.html": [
Expand Down
Expand Up @@ -9,41 +9,23 @@
async_test(function(t) {
var img = document.getElementById("brokenurl");
img.src = "http://[";
var errorevent = false;
var loadendevent = false;

// The errors should be queued in the event loop, so they should only trigger
// after this block of code finishes, not during the img.src setter itself
img.addEventListener('error', t.step_func(function() {
assert_false(loadendevent, "loadend should fire after error");
errorevent = true;
t.step_timeout(t.step_func_done(function() {
assert_true(loadendevent, "loadend event fired");
}), 0);
}));
img.addEventListener('loadend', t.step_func(function() {
assert_true(errorevent, "error event fired");
loadendevent = true;
t.step_timeout(t.step_func_done(), 0);
}));
}, 'src="http://["');

async_test(function(t) {
var img = document.getElementById("emptysrc");
img.src = "";
var loadendevent = false;

// Setting src to empty string triggers only error event.
// The errors should be queued in the event loop, so they should only trigger
// after this block of code finishes, not during the img.src setter itself
img.addEventListener('error', t.step_func(function() {
// Queue this check in the event loop to check there is no loadend event
// fired.
t.step_timeout(t.step_func_done(function() {
assert_false(loadendevent, "loadend event should not be fired");
}), 0);
}));
img.addEventListener('loadend', t.step_func(function() {
loadendevent = true;
t.step_timeout(t.step_func_done(), 0);
}));
}, 'src=""');

Expand Down

0 comments on commit 84898bc

Please sign in to comment.