Bug 100191 - [chromium] [meta] Add auto-detection of image orientation from EXIF information
Summary: [chromium] [meta] Add auto-detection of image orientation from EXIF information
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 19688 100076 100144 100164 100179 100197 100200 100319 100320 100401 100414 100434 100456 100698 100767
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-23 21:24 PDT by noel gordon
Modified: 2012-11-26 20:11 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description noel gordon 2012-10-23 21:24:17 PDT
http://recursive-design.com/blog/2012/07/28/exif-orientation-handling-is-a-ghetto and I concur.

CG added EXIF support in bug 19688 for imageDocuments and for page <img> via the page WebKitShouldRespectImageOrientation setting.

We should add this EXIF support to the chromium port for imageDocument only, with the page setting to permit testing (leave the setting off-by-default) since auto-orientation of page <img> tags should be opt-in (author controlled) ...

[css3-images] Honoring EXIF orientation from images
  http://lists.w3.org/Archives/Public/www-style/2011Dec/0001.html
[css3-images] Processing model for transformations (image() vs. image-orientation)
  http://lists.w3.org/Archives/Public/www-style/2011Dec/0002.html
http://dev.w3.org/csswg/css4-images/#image-orientation
  img { image-orientation: from-image }

So in this bug, be aware of the CSS image-orientation work bug 89052 and how the platform code needs updating to examine an image renderer->style()->imageOrientation() at draw time, see bug 91566 and bug 92330, let's not break them.
Comment 1 Nico Weber 2012-10-23 21:49:06 PDT
The image-only exif rotation bits are done in bug 100164 and bug 100179 (and http://trac.webkit.org/changeset/132260). From what I can tell, this should not collide with the bugs linked above.
Comment 2 Nico Weber 2012-10-23 22:43:51 PDT
Bug 100200 will clash with one of davidbarr's changes, but in a minor way (the code it adds will need to grow 2 additional lines to deal with the passed-in orientation he's adding.)
Comment 3 Nico Weber 2012-10-26 10:25:30 PDT
This is mostly done.
Comment 4 noel gordon 2012-11-05 06:31:36 PST
http://crbug.com/56845 marked fixed, all done for Chrome M24.