Remove the hack around otoro orientation issue when capturing still images

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: cjones, Assigned: m1)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(blocking-basecamp:+, firefox17 fixed, firefox18 fixed)

Details

(Whiteboard: [fota] QARegressExclude)

Attachments

(1 attachment, 1 obsolete attachment)

We should normalize all these device quirks at the platform level.  We've already busted another device from the hack.
Created attachment 673505 [details] [diff] [review]
Normalize camera sensor orientation
(Assignee)

Updated

6 years ago
blocking-basecamp: --- → ?
(Assignee)

Updated

6 years ago
Duplicate of this bug: 802785
This blocks correctly-oriented photos on the commercial phone.
blocking-basecamp: ? → +
(Assignee)

Updated

6 years ago
Attachment #673505 - Flags: review?(jones.chris.g)
Attachment #673505 - Flags: review?(jones.chris.g) → review+
The PR of course looks fine, but please give us until Monday to land until we can work out something for otoro/unagi dogfooders.  Sorry about that.
Assignee: nobody → mvines
Created attachment 674898 [details] [diff] [review]
Trivial rebase

Carrying r+ forward
Attachment #673505 - Attachment is obsolete: true
Attachment #674898 - Flags: review+
(Assignee)

Updated

6 years ago
Attachment #673505 - Attachment is obsolete: false
Comment on attachment 674898 [details] [diff] [review]
Trivial rebase

nm.  can't drive hg properly yet...
Attachment #674898 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Whiteboard: [checkin-needed:aurora]
Note to Moz dogfooders / bisecters: this patch and the associated gaia commit will result in rotated images being captured by the cameras on otoro and unagi.  That's 100% expected and understood.  You won't see this bug in full flashes.

mvines was kind enough to land that workaround for us.  However, we need to ship this workaround in a FOTA update to dogfooders.  (Or have them push a new file to device.)
I'll land this on Aurora after it lands on m-c. No need to request checkin on that part as long as it's got blocking-basecamp+ or approval-aurora+.
Keywords: checkin-needed
Whiteboard: [checkin-needed:aurora]
Also FTR, the FOTA wheels are in motion.
https://hg.mozilla.org/mozilla-central/rev/450f060c4577
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
status-firefox17: --- → fixed
status-firefox18: --- → fixed
Duplicate of this bug: 805737
Duplicate of this bug: 805782
Duplicate of this bug: 806356
Whiteboard: [fota]
Depends on: 806582

Comment 20

6 years ago
Defect is verified as fixed using the Unagi device, build 20130114073222. 
Needs to be verified on an Otoro device.

Updated

6 years ago
Whiteboard: [fota] → [fota] QARegressExclude
You need to log in before you can comment on or make changes to this bug.