Bug 101207

Summary: [Cairo] Make Cairo honor image orientation
Product: WebKit Reporter: Dominik Röttsches (drott) <d-r>
Component: PlatformAssignee: Dominik Röttsches (drott) <d-r>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, gustavo, gyuyoung.kim, kenneth, mrobinson, pnormand, rakuco, tmpsantos, webkit.review.bot, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 95299    
Attachments:
Description Flags
Patch
none
Patch, v2. Syntax fix for efl-wk1 TestExpectations.
none
Patch, v3, comment punctuation fixed.
none
Patch, v4, I mean, really. none

Description Dominik Röttsches (drott) 2012-11-05 03:56:46 PST
Analogously to bug 100179, Cairo needs to honor decoded image orientation info so that EXIF orientation is respected, see bug 100191.
Comment 1 Dominik Röttsches (drott) 2012-11-05 07:14:55 PST
Created attachment 172331 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2012-11-05 07:20:14 PST
Comment on attachment 172331 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=172331&action=review

Nice!

> Source/WebCore/platform/graphics/cairo/BitmapImageCairo.cpp:118
> +        // ImageOrientation expects the origin to be at (0, 0)

Punctuation mark at end
Comment 3 Dominik Röttsches (drott) 2012-11-05 07:25:58 PST
Created attachment 172332 [details]
Patch, v2. Syntax fix for efl-wk1 TestExpectations.
Comment 4 Dominik Röttsches (drott) 2012-11-05 07:28:53 PST
Created attachment 172333 [details]
Patch, v3, comment punctuation fixed.
Comment 5 Dominik Röttsches (drott) 2012-11-05 07:38:52 PST
Created attachment 172334 [details]
Patch, v4, I mean, really.
Comment 6 WebKit Review Bot 2012-11-05 08:24:16 PST
Comment on attachment 172334 [details]
Patch, v4, I mean, really.

Clearing flags on attachment: 172334

Committed r133482: <http://trac.webkit.org/changeset/133482>
Comment 7 WebKit Review Bot 2012-11-05 08:24:21 PST
All reviewed patches have been landed.  Closing bug.