Closed
Bug 982071
Opened 11 years ago
Closed 11 years ago
Unusable Camera app when triggered from pick activity for file upload
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect)
Tracking
(blocking-b2g:1.4+, b2g-v1.4 verified, b2g-v2.0 verified)
People
(Reporter: gerard-majax, Assigned: wilsonpage)
References
Details
(Keywords: regression, Whiteboard: [landed-on-master])
Attachments
(3 files)
See the attached screenshot: Camera app does not start properly.
STR:
0. Open browser
1. Go to http://m.imgur.com/upload
2. Tap "Device", then select Photo in the activity chooser
Expected:
Camera app starts correctly
Actual:
Confere screenshot. Camera app not usable.
This is reproducing on both Camera app from the current master and from camera-new-features branch.
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → 1.4?
Reporter | ||
Comment 1•11 years ago
|
||
logcat shows:
E/GeckoConsole( 1275): [JavaScript Error: "TypeError: setting is undefined" {file: "app://camera.gaiamobile.org/js/main.js" line: 4292}]
Updated•11 years ago
|
Keywords: regressionwindow-wanted
Updated•11 years ago
|
QA Contact: mvaughan
Comment 2•11 years ago
|
||
Blocking just by going with broken functionality criteria
Assignee: nobody → jdarcangelo
blocking-b2g: 1.4? → 1.4+
Comment 3•11 years ago
|
||
The following regression window was found using b2g-inbound builds:
- Last Working -
Device: Buri ENG v1.4 MOZ RIL
BuildID: 20140225203818
Gaia: 6003c319780f49871dbe91ce5af32f5fc8ec06d9
Gecko: 4fe213a22c59
Version: 30.0a1
Firmware Version: V1.2-device.cfg
- First Broken -
Device: Buri ENG v1.4 MOZ RIL
BuildID: 20140225213719
Gaia: d62991e8b97d382f39b44856ea996fb1bb90a459
Gecko: 02217076e7d5
Version: 30.0a1
Firmware Version: V1.2-device.cfg
**This looks to be a gaia issue**
Push Log: https://github.com/mozilla-b2g/gaia/compare/6003c319780f49871dbe91ce5af32f5fc8ec06d9...d62991e8b97d382f39b44856ea996fb1bb90a459
Keywords: regressionwindow-wanted
Updated•11 years ago
|
Severity: normal → blocker
Comment 5•11 years ago
|
||
Setting NEEDINFO for Wilson:
Wilson, the error is happening on the following line:
https://github.com/mozilla-b2g/gaia/blob/camera-new-features/apps/camera/js/lib/setting-alias.js#L56
E/GeckoConsole( 914): [JavaScript Error: "TypeError: setting is undefined" {file: "app://camera.gaiamobile.org/js/main.js" line: 4703}]
This seems to be an issue with the SettingAlias model and I have some questions as to the flow of execution with it. Ping me when you're online so I can discuss with you. Thanks!
Flags: needinfo?(wilsonpage)
Comment 6•11 years ago
|
||
Wilson is out on 3/19 and 3/21. Wilson, please take a look when you are back tomorrow.
Thanks
Hema
Assignee | ||
Comment 7•11 years ago
|
||
Attachment #8394305 -
Flags: review?(jdarcangelo)
Flags: needinfo?(wilsonpage)
Updated•11 years ago
|
Attachment #8394305 -
Flags: review?(jdarcangelo) → review+
Assignee | ||
Updated•11 years ago
|
QA Whiteboard: [branch-camera-new-features fixed]
Assignee | ||
Updated•11 years ago
|
Assignee: jdarcangelo → wilsonpage
Updated•11 years ago
|
QA Whiteboard: [branch-camera-new-features fixed]
Whiteboard: [branch-camera-new-features reviewed-ready-to-land]
Comment 8•11 years ago
|
||
Landed on camera-new-features:
https://github.com/mozilla-b2g/gaia/commit/d8fd55a7f1ef63ff24e3374abb11ea06849be07d
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [branch-camera-new-features reviewed-ready-to-land] → [branch-camera-new-features fixed]
Comment 9•11 years ago
|
||
This needs to land on the main Gaia branch before this gets marked fixed. Otherwise, someone will just open a new bug for this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 10•11 years ago
|
||
Diego: We already landed this on camera-new-features, we need to also land on master.
Attachment #8395133 -
Flags: review?(dmarcos)
Comment 11•11 years ago
|
||
Yes. This will land on master once we merge camera-new-features. Partner requirement
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Comment 12•11 years ago
|
||
(In reply to Diego Marcos [:dmarcos] from comment #11)
> Yes. This will land on master once we merge camera-new-features. Partner
> requirement
Please see https://bugzilla.mozilla.org/show_bug.cgi?id=984730#c19.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•11 years ago
|
Attachment #8395133 -
Flags: review?(dmarcos) → review+
Updated•11 years ago
|
Attachment #8395133 -
Flags: review+ → review-
Comment 13•11 years ago
|
||
Justin. This is 1.4 blocker. Your PR should go on top of v1.4 branch instead of master
Comment 14•11 years ago
|
||
Can we merge this with the rest of camera-new-features branch?
Comment 15•11 years ago
|
||
Sorry, I can re-submit the PR, but let's wait to see what the plan is for landing camera-new-features today first.
Updated•11 years ago
|
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Whiteboard: [branch-camera-new-features fixed] → [landed-on-master]
Comment 16•11 years ago
|
||
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
Comment 17•11 years ago
|
||
The bug is no longer reproduces on 1.4 and master build
Camera app starts and taking pictures correctly
1.4 Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140410000201
Gaia: 9b2da43dfee3792cd311ae55f0b06272313208f0
Gecko: 9d9ead7d6afa
Version: 30.0a2
Firmware Version: v1.2-device.cfg
1.5 Environmental Variables:
Device: Buri 1.5 MOZ
BuildID: 20140410040201
Gaia: 9d0b1bdf746823a94b13e6574c1d8304dc584763
Gecko: 690c810c8e3e
Version: 31.0a1
Firmware Version: v1.2-device.cfg
You need to log in
before you can comment on or make changes to this bug.
Description
•