Re: [csswg-drafts] [css-images-3] Allow impls to not respect exif data if it's after the image data (#4929)

The CSS Working Group just discussed `Allow impls to not respect exif data if it's after the image data`, and agreed to the following:

* `RESOLVED: UAs should ignore exif data that comes after image data`

<details><summary>The full IRC log of that discussion</summary>
&lt;gregwhitworth> Topic: Allow impls to not respect exif data if it's after the image data<br>
&lt;astearns> github: https://github.com/w3c/csswg-drafts/issues/4929<br>
&lt;gregwhitworth> TabAtkins: so the image orientation from image value allows you to use the exif data and rotate accordingly<br>
&lt;gregwhitworth> TabAtkins: some people noted that the location is flexible. If the exif data can live at the end and you can not initially do the orientation if you're progressively rendering<br>
&lt;gregwhitworth> TabAtkins: recommending to not support exif data if the exif data comes after the image data<br>
&lt;gregwhitworth> leaverou: any numbers on how common this is?<br>
&lt;gregwhitworth> TabAtkins: no<br>
&lt;gregwhitworth> TabAtkins: someone mentioned it was less likely, but no strict numbers<br>
&lt;smfr> q+<br>
&lt;gregwhitworth> chris: cameras have that first and the only cases where this will get messed with is the tools that will shove it at the end<br>
&lt;gregwhitworth> TabAtkins: once the image data starts the browser will ignore any other exif data<br>
&lt;smfr> i will type<br>
&lt;smfr> i find it really weird that css would say anything about how images are encoded<br>
&lt;gregwhitworth> astearns: anyone else with a comment on this while we wait for smfr<br>
&lt;smfr> i think we treat this as a “bad image” and just let the bad stuff happen<br>
&lt;gregwhitworth> TabAtkins: while weird, all browsers are doing this<br>
&lt;smfr> also might be hard for implementors: not sure if we can ask our image framework HOW the metadata is ordered<br>
&lt;gregwhitworth> astearns: we're doing bad things now because we haven't found exif prior to rendering<br>
&lt;smfr> what about a small image which is loaded in 1 go and the EXIF data is still after the image data<br>
&lt;gregwhitworth> florian: what about cache scenarios?<br>
&lt;gregwhitworth> iank_: I wouldn't bet on that, it's often sometimes slower<br>
&lt;smfr> would be OK with a MAY<br>
&lt;gregwhitworth> astearns: what if we have UAs may choose to ignore exif data if it isn't before the image data<br>
&lt;smfr> don’t want it to be a MUST because of reasons above<br>
&lt;gregwhitworth> chris: advise authors to avoid putting it at the end since there isn't interop<br>
&lt;gregwhitworth> TabAtkins: then we could go with a should then<br>
&lt;gregwhitworth> hober: smfr says he's ok with may<br>
&lt;gregwhitworth> florian: not having interop is bad too<br>
&lt;TabAtkins> smfr, you okay with SHOULD?<br>
&lt;TabAtkins> (probably with an explicit callout to violations being due to image-decoders not reflecting the ordering)<br>
&lt;smfr> ok with SHOULD<br>
&lt;gregwhitworth> astearns: Should we be "UAs should ignore exif data that comes after image data"<br>
&lt;gregwhitworth> Proposed Resolution: UAs should ignore exif data that comes after image data<br>
&lt;smfr> q-<br>
&lt;gregwhitworth> astearns: I thought loading the image included looking for exif data<br>
&lt;gregwhitworth> TabAtkins: yeah, it's not a precise term<br>
&lt;futhark> [leaving]<br>
&lt;gregwhitworth> RESOLVED: UAs should ignore exif data that comes after image data<br>
&lt;astearns> ack fantasai<br>
&lt;Zakim> fantasai, you wanted to ask if we should also say authors SHOULD NOT use such images?<br>
&lt;gregwhitworth> TabAtkins: I think we should do an authors MUST NOT<br>
&lt;gregwhitworth> fantasai: we should give a clear reason as to why to do that<br>
&lt;fantasai> s/reason/guidance/<br>
&lt;fantasai> s/as to why to/to not/<br>
&lt;gregwhitworth> astearns: if we put in author guidance, we should make sure to put in example of why they shouldn't<br>
&lt;gregwhitworth> Proposed Resolution: Authors MUST NOT put exif data at the end of the image<br>
&lt;gregwhitworth> Resolved: Authors MUST NOT put exif data after the image data<br>
&lt;TabAtkins> oh time to rejoin huh<br>
&lt;TabAtkins> we're having fun in our breakout, new meeting, csswg is passe<br>
&lt;iank_> TabAtkins: do you trigger the rejoin? or do we have to opt-in?<br>
&lt;TabAtkins> there's a "return to main call" button on your video<br>
&lt;iank_>  ah saw the big button thingy<br>
&lt;fremy> ScribeNick: fremy<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/4929#issuecomment-714572885 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 22 October 2020 15:27:22 UTC