Bug 210021

Summary: EXIF orientation info in PNGs isn't used for image-orientation: from-image
Product: WebKit Reporter: Eric Portis <e>
Component: ImagesAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: bfulgham, jonlee, karlcow, sabouhallawa, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: BrowserCompat, InRadar
Version: Safari Technology Preview   
Hardware: Mac   
OS: macOS 10.15   

Description Eric Portis 2020-04-05 02:37:45 PDT
`image-orientation: from-image` correctly reads and applies EXIF orientation from JPEGs; however it does not read or apply EXIF orientation info from PNGs.

See a test page/resources here: https://ericportis.com/etc/PNG-EXIF-orientation/
Comment 1 Said Abou-Hallawa 2020-04-05 17:27:33 PDT
<rdar://problem/61323993>
Comment 2 Said Abou-Hallawa 2020-04-05 17:28:30 PDT
Either there is a bug in exiftool or in one of the macOS underlying framework.
Comment 3 Simon Fraser (smfr) 2020-04-06 10:26:19 PDT
This seems to be a quirk of the image.

The image has these chunks:
	IHDR
	PLTE
	IDAT
	eXIf
	IEND
libpng does not recognize the ‘eXIf’ chunk if it’s after the image data (IDAT).
The spec states: “The eXIf chunk may appear anywhere between the IHDR and IEND chunks except between IDAT chunks.”
So in theory it’s allowed, but libpng does not support this.
If the PNG is modified (moved eXIf before IDAT) it displays as expected (90º rotated).
FireFox, Chrome, GraphicConverter don’t handle this either.
Comment 4 Eric Portis 2020-04-06 14:31:51 PDT
@smfr Interesting!

Here's a new test page: https://ericportis.com/etc/PNG-EXIF-orientation/shuffling-chunks/

Looks like (on MacOS 10.15):

Finder, Preview, and Safari respect eXIf-before-IDAT.

Chrome and Firefox do not respect eXIf-before-IDAT.

I'm not sure how any browser can support both eXIF-*after*-IDAT and also progressively render partially-downloaded, progressively-encoded PNGs. In some initial testing both Firefox and Chrome will paint partially-downloaded PNGs, but Safari does not.

Does a note about PNG chunk order and/or progressive rendering belong in the spec?
Comment 5 Karl Dubost 2022-06-20 00:14:34 PDT
A similar case has been reported on webcompat.com
https://github.com/webcompat/web-bugs/issues/94467