Closed
Bug 1011101
Opened 11 years ago
Closed 11 years ago
[Camera][Gecko] Enable auto focus moving callbacks by default
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.0 S2 (23may)
People
(Reporter: dmarcos, Assigned: aosmond)
Details
Attachments
(1 file)
710 bytes,
patch
|
dhylands
:
review+
|
Details | Diff | Splinter Review |
Now we have to pref in to have the feature enabled. We need to add in build/config/custom-prefs.js the following lines:
user_pref("camera.control.face_detection.enabled", true);
user_pref("camera.control.autofocus_moving_callback.enabled", true);
The feature should be now enabled by default with no extra configuration
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Face detection was already turned on in bug999396, just auto focus remaining
Attachment #8423868 -
Flags: review?(dhylands)
Assignee | ||
Updated•11 years ago
|
Summary: [Camera][Gecko] Enable face detection by default → [Camera][Gecko] Enable auto focus moving callbacks by default
Updated•11 years ago
|
Target Milestone: --- → 2.0 S2 (23may)
Comment 2•11 years ago
|
||
Comment on attachment 8423868 [details] [diff] [review]
bug1011101.patch, v1
Review of attachment 8423868 [details] [diff] [review]:
-----------------------------------------------------------------
looks good to me
Attachment #8423868 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Comment 4•11 years ago
|
||
May ignore Windows failures, not related to this submission. Emulator failures are intermittant and also unrelated to camera.
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Keywords: checkin-needed
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•