1023043 - [Flame][v2.0] taking picture with front camera in landscape mode, the orientation is reversed while preview it

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Vendcom, defect)

Description

[Brian Huang [:brianhuang][:brhuang]]

Attachment: orientation reserved picture

Build Information

Gaia      901646a3279c66daa7621bebb62641779d8ae22c
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/d6376896f57c
BuildID   20140609160201
Version   32.0a2
ro.build.version.incremental=94
ro.build.date=Tue May 20 09:29:20 CST 2014

Device: Flame

Description

Taking picture with front camera. Using preview or gallery, the orientation is reversed. See attached picture.

Steps to Reproduce
1. launch camera
2. Switch camera to front facing camera
3. take pictures with landscape mode
4. Preview or use Gallery to view the pictures
5. The orientation is reserved

Expected Results
1. The orientation is correct

Actual Results
1. The orientation is wrong and reserved.

Reproduction Frequency: Always occurs

Comment 1

[Marcia Knous [:marcia]]

Able to reproduce with Base Image H on Flame using:

Gaia   f42ebc93554979501d3ac52bcf9e69cb4b310a4f
SourceStamp 9dc0ffca10f4
BuildID 20140610040208
Version 33.0a1 and

Gaia   8d865839d932bfbd5e157f376f74d8cb12bfdd51
SourceStamp 1d4046a8cb6c
BuildID 20140610000223
Version 32.0a2

Comment 2

[Lionel Mauritson]

B2G Inbound Regression Window

Last Working
Device: Flame 2.0
BuildID: 20140517113002
Gaia: d364c473012b7731688fe0ef4bca92358cee18c5
Gecko: f913ad39fa23
Version: 32.0a1 (2.0) 
Firmware Version: v10G-2

First Broken
Device: Flame 2.0
Build ID: 20140519053002
Gaia: 8a2352d5b7be27ec4b1ea18c680ebcd0b6d34348
Gecko: 8508ab79921f
Version: 32.0a1 (2.0) 
Firmware Version: v10G-2

First Broken Gaia Last Working Gecko: Issue DOES reproduce.
Gaia: 8a2352d5b7be27ec4b1ea18c680ebcd0b6d34348
Gecko: f913ad39fa23

Last Working Gaia First Broken Gecko: Issue does NOT reproduce.
Gaia: d364c473012b7731688fe0ef4bca92358cee18c5
Gecko: 8508ab79921f

Gaia Pushlog:
https://github.com/mozilla-b2g/gaia/compare/d364c473012b7731688fe0ef4bca92358cee18c5...8a2352d5b7be27ec4b1ea18c680ebcd0b6d34348

Unfortunately I was not able to narrow it down any further due to a lack of available builds.

Comment 3

[Diego Marcos [:dmarcos]]

Flame camera is not inserting the orientation flag in the EXIF headers and gallery and preview default to 0.

I attached two pictures. One taken with Flame and the other with Hamachi. To read the Orientation flag on OSX you can use exiftool (brew install exiftool):

exiftool -Orientation /picture/directory/image.jpg

Mike, Who should be looking into this?

Comment 4

[Diego Marcos [:dmarcos]]

Attachment: flame.jpg

Comment 5

[Diego Marcos [:dmarcos]]

Attachment: hamachi.jpg

Comment 6

[Mike Habicher [:mikeh] (high bugzilla latency)]

On every other platform, we expect the camera library to construct a JPEG with an EXIF header that includes the orientation field. This is a vendor issue.

Comment 7

[Mike Habicher [:mikeh] (high bugzilla latency)]

I have confirmed Diego's results in comment 3: the images emitted by the front-facing Flame camera (in -all- device orientations) do not contain an EXIF 'orientation' tag. HOWEVER, only images taken with a landscape device orientation are upside down; images taken with the device upside down appear correctly. This suggests that despite the absence of an 'orientation' tag, the camera library/driver is attempting to adjust the emitted image in response to the orientation information we pass to it through KEY_ROTATION.

This is with:
- basebuild: v121-2
- gecko:     m-c:aab3362f97e9
- gaia:      master:90777363ed0a4e6d32612074a12fb2c73c353a25
- buildid:   20140612160203
- version:   33.0a1
- ro.build.version.incremental=108
- ro.build.date=Tue Jun 10 19:40:40 CST 2014

This is POVB: the camera library is incorrectly handling device orientation information.

Comment 9

[Jason Smith [:jsmith]]

Francis - Can you have someone from T2M take a look?

Comment 10

[Francis Lee [:frlee]]

hi Yanan, Youlong,

could you please check this issue? This issue can be reproduced on V121-2 w/0 changing gaia/gecko version.

Comment 11

[Francis Lee [:frlee]]

hi John,

may you confirm this is the same issue as we met in Vixen? if so, this should not be a vendor issue.

thank you very much

Comment 12

[John Hu [:johnhu][:johu][:醬糊小弟]]

Francis,

No, this is different issue. The Vixen issue is caused by locking wrong orientation in screen. But it is wrong orientation flag embedded in image.

Comment 14

[Mike Lien[:mlien]]

verify with Flame KK build v162-3, it's fine

Comment 15

[BMO Automation]

Firefox OS is not being worked on