842700 - [B2G][Gallery] Switching between portrait and landscape mode while in Edit window does ridiculous changes to an image

Status: VERIFIED WORKSFORME

(Firefox OS Graveyard :: Gaia::Camera, defect)

Description

[Natalya Kot [:nkot]]

Description:
When switching between portrait and landscape modes while editing picture it is changing image appearance very bad - stretching it our or shrinks - see screenshots 

Repro Steps:
1) Updated to Unagi Build ID: 20130219070202
2) Launch Gallery app
3) While in the portrait mode tap any picture to open full view
4) Tap edit icon in the bottom tool bar
5) Switch orientation to landscape
6) Play with different editing options
7) Switch between portrait and landscape while editing

Expected:
*Image will appear normally and will only be adjusted accordingly to editing selection

Actual:
*Image stretches out or shrinks while editing 

Repro frequency:
100%

*Notes:
Gecko: c8d5dd5b0d452be57dddfedb4c1c4c61a920761e
Gaia: edaca00b1eb7534120b6255db5d5200fb1d86d65

*Screenshots attached

Comment 1

[Marcia Knous [:marcia]]

Can you please attach the screenshots?  Thanks.

Comment 2

[Natalya Kot [:nkot]]

Attachment: screenshot 1

Comment 3

[Natalya Kot [:nkot]]

Attachment: screenshot 2

Comment 4

[Marcia Knous [:marcia]]

Where did you get those images? I am having difficult reproducing this with images I am taking from the device.

Comment 5

[Natalya Kot [:nkot]]

it repros for any image in the gallery, both camera taken and transferred,

Comment 6

[Natalya Kot [:nkot]]

these SRT could be better:

Device is in the portrait mode
1. open gallery
2. select any image
3. select edit icon
4. rotate device to landscape => you 'll get stretched out image
5. select "crop" option then "exposure"
6. rotate back to the portrait mode => image will shrink

Comment 7

[John Hammink]

David - do you have any ideas how this is supposed to work?  Some clarification could be helpful here.

Comment 8

[David Flanagan [:djf]]

No real clarification needed: the image just needs to be resized correctly.  There's only really one right way to do it.  This is not a regression. Just something that never got implemented.

Jon Hylands has just done work on this part of the app, so I'm cc'ing him in case he wants to take it.

If this gets tef+'ed and has no owner, please assign to me and I'll fix it right away.

John Hammink: I'm surprised that this hasn't been flagged by QA before this, but the fact that it did get flagged shows that the system works! :-)  Thanks for bringing it to my attention.

Comment 9

[Allen Maxwell]

No longer reproducible on v1.0.1, Mozilla RIL, or Commercial RIL
 
Resolving as Works-For-Me on 3/29 Commercial (v1.1) RIL:
Build ID: 20130329070203
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/5cc5df16447a
Gaia: 26b463f14caa11e0fc64fda09a17054da4bea68b