Closed Bug 923361 Opened 7 years ago Closed 7 years ago

[Meta][User Story] getUserMedia API for image/video capture and permission prompt

Categories

(Firefox OS Graveyard :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
1.4 S2 (28feb)

People

(Reporter: itsay, Assigned: itsay)

References

Details

(Keywords: meta, Whiteboard: [ucid:WebRTC7, 1.4, ft:multimedia-platform][NPOTB])

This is the user story meta bug for webRTC
blocking-b2g: --- → 1.3+
Whiteboard: [ucid:WebRTC7, ft:media-recording, 1.3:p1][NPOTB]
Summary: [Meta][User Story] getUserMedia in WebRTC-video for image/video (For Camera) → [Meta][User Story] getUserMedia API for image/video capture (WebRTC-Video)
Keywords: meta
Flags: in-moztrap?(jsmith)
Per discussion offline - targeted features don't block, so we don't need to continue blocking on this.
blocking-b2g: 1.3+ → -
Depends on: 912960
Depends on: 916524
Depends on: 853356
Depends on: 898949
Depends on: 914028
No longer depends on: 912960
blocking-b2g: - → ---
Update whiteboard tag to follow format [ucid:{id}, {release}:p{1,2}, ft:{team-id}]
Whiteboard: [ucid:WebRTC7, ft:media-recording, 1.3:p1][NPOTB] → [ucid:WebRTC7, 1.3:p1, ft:media-recording][NPOTB]
Update to P2 in whiteboard since this is target feature.
Whiteboard: [ucid:WebRTC7, 1.3:p1, ft:media-recording][NPOTB] → [ucid:WebRTC7, 1.3:p2, ft:media-recording][NPOTB]
Remove dependency on 898949.

This use story no longer depend on the bug 898949. Per discussion between Shih-Chiang and Fabrice while reviewing the 1st patch in bug 898949, we can only get the camera/video permission without the option for front and back camera. We will add the option in v1.4.
No longer depends on: 898949
(In reply to Ivan Tsay (:ITsay) from comment #4)
> Remove dependency on 898949.
> 
> This use story no longer depend on the bug 898949. Per discussion between
> Shih-Chiang and Fabrice while reviewing the 1st patch in bug 898949, we can
> only get the camera/video permission without the option for front and back
> camera. We will add the option in v1.4.

So what camera are we using here then for 1.3? Front or back camera?
Flags: needinfo?(schien)
Back camera will be used in default.
Flags: needinfo?(schien)
(In reply to Jason Smith [:jsmith] from comment #5)
> (In reply to Ivan Tsay (:ITsay) from comment #4)
> > Remove dependency on 898949.
> > 
> > This use story no longer depend on the bug 898949. Per discussion between
> > Shih-Chiang and Fabrice while reviewing the 1st patch in bug 898949, we can
> > only get the camera/video permission without the option for front and back
> > camera. We will add the option in v1.4.
> 
> So what camera are we using here then for 1.3? Front or back camera?

Jason, assuming this story will def go in 1.4. Correct?
(In reply to Candice Serran from comment #7)
> (In reply to Jason Smith [:jsmith] from comment #5)
> > (In reply to Ivan Tsay (:ITsay) from comment #4)
> > > Remove dependency on 898949.
> > > 
> > > This use story no longer depend on the bug 898949. Per discussion between
> > > Shih-Chiang and Fabrice while reviewing the 1st patch in bug 898949, we can
> > > only get the camera/video permission without the option for front and back
> > > camera. We will add the option in v1.4.
> > 
> > So what camera are we using here then for 1.3? Front or back camera?
> 
> Jason, assuming this story will def go in 1.4. Correct?

To my understanding, this is planning to land on Sunday in Taipei's timezone. So this will still be a part of 1.3.
(In reply to Jason Smith [:jsmith] from comment #8)
> (In reply to Candice Serran from comment #7)
> > (In reply to Jason Smith [:jsmith] from comment #5)
> > > (In reply to Ivan Tsay (:ITsay) from comment #4)
> > > > Remove dependency on 898949.
> > > > 
> > > > This use story no longer depend on the bug 898949. Per discussion between
> > > > Shih-Chiang and Fabrice while reviewing the 1st patch in bug 898949, we can
> > > > only get the camera/video permission without the option for front and back
> > > > camera. We will add the option in v1.4.
> > > 
> > > So what camera are we using here then for 1.3? Front or back camera?
> > 
> > Jason, assuming this story will def go in 1.4. Correct?
> 
> To my understanding, this is planning to land on Sunday in Taipei's
> timezone. So this will still be a part of 1.3.

Clarification - Monday in Taipei's timezone
Will try to land the test case for the bug 853356 Monday Taipei timezone. Currently we leave the bug 853356 open before the test case is landed. The functionality of this user story is already in v1.3.
This is now feature complete.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
(In reply to Jason Smith [:jsmith] from comment #11)
> This is now feature complete.

And now it isn't - bug 853356 managed to turn the tree orange in the integration tests & broke the contacts API permission prompt. It's being backed out.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Jason Smith [:jsmith] from comment #12)
> (In reply to Jason Smith [:jsmith] from comment #11)
> > This is now feature complete.
> 
> And now it isn't - bug 853356 managed to turn the tree orange in the
> integration tests & broke the contacts API permission prompt. It's being
> backed out.

Also - given the backout, this is now moving to 1.4.
Blocks: 1.4-multimedia
No longer blocks: 1.3-media-recording
Update whiteboard version to 1.4 and update title with the mention of permission prompt.
Also add back bug 898949 for the option of front/back permission prompt.
Depends on: 898949
Summary: [Meta][User Story] getUserMedia API for image/video capture (WebRTC-Video) → [Meta][User Story] getUserMedia API for image/video capture and permission prompt
Whiteboard: [ucid:WebRTC7, 1.3:p2, ft:media-recording][NPOTB] → [ucid:WebRTC7, 1.4:p2, ft:media-recording][NPOTB]
Flags: in-moztrap?(jsmith)
Whiteboard: [ucid:WebRTC7, 1.4:p2, ft:media-recording][NPOTB] → [ucid:WebRTC7, 1.4:p2, ft:multimedia-platform][NPOTB]
Hi Jason,

Are there already the test cases for this feature in v1.4?
Flags: in-moztrap?(jsmith)
(In reply to Ivan Tsay (:ITsay) from comment #15)
> Hi Jason,
> 
> Are there already the test cases for this feature in v1.4?

Not yet. Thanks for flagging. I'm looking into this.
Points taken. Thank you for the follow up.
Depends on: 970183
Blocks: 923364
Assign to EPM for tracking purpose
Assignee: nobody → itsay
Whiteboard: [ucid:WebRTC7, 1.4:p2, ft:multimedia-platform][NPOTB] → [ucid:WebRTC7, 1.4, ft:multimedia-platform][NPOTB]
Target Milestone: --- → 1.4 S2 (28feb)
Depends on: 973696
front/back camera support is being moved to 1.5
No longer depends on: 898949
Depends on: 898949
All dependencies closed.
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
No longer depends on: 977956
No longer depends on: 977971
No longer depends on: 978660
bug 942724 blocks a FC signoff - we're missing a key feature component here.
Status: RESOLVED → REOPENED
Depends on: 942724
Resolution: FIXED → ---
(In reply to Jason Smith [:jsmith] from comment #21)
> bug 942724 blocks a FC signoff - we're missing a key feature component here.

And now it's fixed.
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
Flags: in-moztrap?(jsmith)
You need to log in before you can comment on or make changes to this bug.