1680387 - Images should be able to declare their own intrinsic resolution/size (EXIF density not honored)

Status: RESOLVED FIXED

(Core :: Layout: Images, Video, and HTML Frames, enhancement)

Description

[Eric Portis]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15

Steps to reproduce:

Run any of these tests:
https://wpt.fyi/results/density-size-correction?label=master&label=experimental&aligned&q=density

Actual results:

Firefox does not respect the EXIF information in the JPEG, and fails the tests.

Expected results:

Firefox should set the image's intrinsic size based on EXIF information, and pass the tests.

See the spec change, discussion, and links to the commits that landed this in Chromium and WebKit, here: https://github.com/whatwg/html/pull/5574

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Andrew / Tim, this seems more of an ImageLib thing than a layout thing right?

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1680387 - Read and expose EXIF image resolution data. r=tnikkel,aosmond

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1680387 - Apply intrinsic image resolution as appropriate in layout/style/dom, and update test expectations. r=tnikkel

This should be mostly straight-forward, since we have code for this
anyways for image-set() and srcset.

The only thing is that we were using floats for resolution, but since
EXIF allows you to scale each axis separately, we now need to pass an
image::Resolution instead.

The main outstanding issue is the spec comment mentioned in the previous
patch, about what happens if you have srcset/image-set and the image
density specified together. For now I've implemented what the
image-set() spec says, but this is subject to change before shipping of
course.

Comment 5

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1680387 - Fix interaction with src-set() / image-set(), and enable the feature by default. r=tnikkel

As discussed here: https://github.com/whatwg/html/pull/5574#issuecomment-826347560

This matches other browsers.

Depends on D113265

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0c1c5e1f901b
Read and expose EXIF image resolution data. r=tnikkel,aosmond
https://hg.mozilla.org/integration/autoland/rev/51531850a9a2
Apply intrinsic image resolution as appropriate in layout/style/dom, and update test expectations. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/65616921e520
Fix interaction with src-set() / image-set(), and enable the feature by default. r=tnikkel

Comment 7

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/28811 for changes under testing/web-platform/tests

Comment 8

[Iulian Moraru]

Backed out for causing browser chrome failures on browser_bug592641.js.

Push with failures

Failure log

Backout link

Comment 9

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 10

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1680387 - Sanity-check exif resolution with other exif metadata. r=tnikkel,aosmond

Also matches other browsers, and fixes the test that got me backed out,
since it has a huge EXIF resolution value.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9317cd872430
Read and expose EXIF image resolution data. r=tnikkel,aosmond
https://hg.mozilla.org/integration/autoland/rev/4f74cdfb4fdc
Apply intrinsic image resolution as appropriate in layout/style/dom, and update test expectations. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/2ac7c9cd4629
Fix interaction with src-set() / image-set(), and enable the feature by default. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/82461e5bf8ae
Sanity-check exif resolution with other exif metadata. r=tnikkel

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/09481b273d7b
For some reason exif_resolution.jpg landed empty?

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/021e5ade2475
Restore some test annotations that are still failing.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f573ec57b561
Indent correctly one fuzzy annotation.

Comment 15

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9317cd872430
https://hg.mozilla.org/mozilla-central/rev/4f74cdfb4fdc
https://hg.mozilla.org/mozilla-central/rev/2ac7c9cd4629
https://hg.mozilla.org/mozilla-central/rev/82461e5bf8ae
https://hg.mozilla.org/mozilla-central/rev/09481b273d7b
https://hg.mozilla.org/mozilla-central/rev/021e5ade2475
https://hg.mozilla.org/mozilla-central/rev/f573ec57b561

Comment 16

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot