965145 - [Camera] refactor .sensorAngle patch in bug 947956 for new camera app

Status: RESOLVED FIXED

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

Description

[KM Lee [:rexboy] (inactivate after Sep. 22)]

+++ This bug was initially created as a clone of Bug #947956 +++

We've landed the .sensorAngle patch on bug 947956 for v1.3.
Now we can try to refactor the patch to gaia master for new camera app.

Comment 1

[Inder]

Preeti / Diego -- this is causing the camera preview to be upside down on QRD devices. This is a blocker for us and we need it fixed by 3/24.

Thanks.

Comment 2

[Preeti Raghunath(:Preeti)]

Hema

Can you please review this bug for 1.4?

Comment 3

[Hema Koka [:hema]]

rexboy: since you already have a patch for master on bug: https://bugzilla.mozilla.org/show_bug.cgi?id=947956 that has some comments that you need to address. Can you take this bug and fix it?

Thanks
Hema

Comment 4

[KM Lee [:rexboy] (inactivate after Sep. 22)]

I’m currently working on another project and I’m afraid that I can’t put a patch before 3/24. If someone can work on this, I can help him / try to describe how the patch is working.

Comment 5

[Justin D'Arcangelo [:justindarc]]

KM: Do you have a patch for this already? I assigned this to myself and was about to start looking into it.

Comment 6

[KM Lee [:rexboy] (inactivate after Sep. 22)]

What I have are just the same with patches on bug 947956.
https://github.com/mozilla-b2g/gaia/pull/15188 The patch committed to v1.3.

What I did before for refactoring to v1.4 (on bug 947956; and it's not committed in since wilsonpage told me there would have following refactors) was almost just move the setPreviewSize function to where it should be in v1.4; but there should be refactors since then so you may need to take a look on the new codebase.

Comment 7

[Justin D'Arcangelo [:justindarc]]

Attachment: pull-request (camera-new-features)

Comment 8

[Inder]

:justin -- I was trying out the patch (comment 7) and noticed it doesn't apply cleanly. Can you please upload an updated patch. We can then verify if it fixes our issue or not.
Thanks.

Comment 9

[Justin D'Arcangelo [:justindarc]]

Inder: The patch needs to be applied on top of the `camera-new-features` branch:

https://github.com/mozilla-b2g/gaia/tree/camera-new-features

Comment 10

[David Flanagan [:djf]]

Comment on attachment 8393733 [details] [review]
pull-request (camera-new-features)

Clearing the review request and setting feedback+.  With github issues addressed this looks good to me, but since we're both flying blind, let's ask QC to try the patch on a QRD before we r+ and land it.

Comment 11

[David Flanagan [:djf]]

Inder: are you able to test by applying Justin's patch on top of the camera-new-features branch?  That is the branch that Leyla is currently testing on.  We hope to have that merged to master soon

Comment 12

[Hema Koka [:hema]]

Inder/Leyla, please help test the patch

Comment 13

[Hema Koka [:hema]]

Eric, 

Would you be able to help test this patch out from "camera-new-features" branch using the QRD that you have so we can move forward with this?

Thanks
Hema

Comment 14

[Eric Chang [:ericcc] [:echang]]

Currently I have no working QRD to test with, there is some issue my gecko on my device, so I have tried to reproduce bugs with Fugu and Buri.

Comment 15

[Inder]

Tried out this patch on camera-new-features branch and it fixes the camera preview upside down issue on QRD!

Justin/Hema - can we get a version of the patch which works for current gaia v1.4 branch so we can land in the tree temporarily and remove the FC blocker.

Comment 16

[Hema Koka [:hema]]

inder: the camera guys are also actively working on addressing bugs on the new features branch to move a bunch of madai features to master/1.4 -- just context switching back and forth for specific bugs will cause some inefficiencies especially when we are racing to cram a bunch of camera features into 1.4. 

since you have verified this on branch that it works - can removing the FC blocker tag off of this/landing on master/1.4 wait for a few more days please (mid next week). Does this block any further testing for 1.4? If yes, we will get on figuring out a version of the patch landed first on master/1.4 (justin is also feeling sick today and needs to take a break!)

Comment 17

[Inder]

Hema: I understand that most of your team is busy with camera-new-features branch but the test teams have put it as a FC blocker as I mentioned earlier so we need a patch for 1.4.
Justin: sorry to hear this. Get well soon.

Thanks,
-Inder

Comment 18

[Diego Marcos [:dmarcos]]

David gave an r+ with the condition that it had to be verified on the QRD device. Comment #15 confirms that the patch fixes the bug on QRD

Comment 19

[Justin D'Arcangelo [:justindarc]]

Landed on camera-new-features:

https://github.com/mozilla-b2g/gaia/commit/fc797effd2457d3ba48159a4087b91c3721d59fc

Comment 20

[Jason Smith [:jsmith]]

The branch this patch was filed against was master, so this isn't closed until the patch lands on master.

Comment 21

[David Flanagan [:djf]]

The fix for this bug is on top of a number of other patches on our camera-new-features branch and cannot easily be uplifted to 1.4. 

Since no one at Mozilla is able to get the camera app to run at all on our QRD devices, we won't be able to test the patch if we tried to create a version that applied cleanly to 1.4.

Therefore we think that this fix will have to wait until Wednesday.  We'll land camera-new-features to master early Tuesday morning, and then uplift to 1.4 about 24 hours after that.

Comment 22

[Hema Koka [:hema]]

landed on master along with other camera features

Comment 23

[David Flanagan [:djf]]

Bulk edit for camera bugs.

If earlier comments do not show how this bug landed to master, it probably landed as part of https://github.com/mozilla-b2g/gaia/pull/17599 which merged the camera-new-features branch into master.

This bug was uplifted from master to v1.4 as part of https://github.com/mozilla-b2g/gaia/commit/a8190d08e61316a86bba572ba8d894d081a20530