Closed Bug 1728359 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 30259 - Images: Add test for list of available image coalescing

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
93 Branch
Tracking Flags:
Tracking Status
firefox93 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 30259 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/30259
Details from upstream follow.

b'Dominic Farolino <dom@chromium.org>' wrote:

Images: Add test for list of available image coalescing

This CL adds a test ensuring that the list of available images is not
used to coalesce an image request into a current in-flight request

The HTML Standard's list of available images 1 is only added to once
all of of the bytes of an image have been fetched, and the load event
is about to be fired 2. This means that if an image request A is
currently in-flight, and an image request B is made for the exact same
image, both requests should hit the network and not be coalesced. This
is also explicitly the intention of the spec:
"It is not used to avoid re-downloading the same image while the
previous image is still loading."

Currently all browsers fail this test likely due to the memory cache
(unspecified) or some non-spec-conforming form of the "list of
available images". We should consider changing the spec to allow
in-flight request coalescing, however until that is done we should at
least have a test codifying the current state of the specification.

R=domenic@chromium.org

Bug: 994329
Change-Id: I9479ef286435704108e80e78096d5418ab014767
Reviewed-on: https://chromium-review.googlesource.com/3133524
WPT-Export-Revision: 0060e9d4f157c7668cfb3a91f1bdc4d95b52f9b8

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
FAIL: 1

Chrome

OK : 1
FAIL: 1

Safari

OK : 1
FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/semantics/embedded-content/the-img-element/list-of-available-images-does-not-coalesce-in-flight-requests.sub.tentative.html
list of available images does not coalesce in-flight requests: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/077894161a31
[wpt PR 30259] - Images: Add test for list of available image coalescing, a=testonly
https://hg.mozilla.org/integration/autoland/rev/cbe995ae7587
[wpt PR 30259] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/077894161a31
https://hg.mozilla.org/mozilla-central/rev/cbe995ae7587

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox93: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
You need to log in before you can comment on or make changes to this bug.