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

RESOLVED FIXED in 1.4 S2 (28feb)

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: itsay, Assigned: itsay)

Tracking

({meta})

unspecified
1.4 S2 (28feb)
x86
macOS
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

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

(Assignee)

Description

5 years ago
This is the user story meta bug for webRTC
(Assignee)

Updated

5 years ago
Blocks: 922527
blocking-b2g: --- → 1.3+
Whiteboard: [ucid:WebRTC7, ft:media-recording, 1.3:p1][NPOTB]
(Assignee)

Updated

5 years ago
Summary: [Meta][User Story] getUserMedia in WebRTC-video for image/video (For Camera) → [Meta][User Story] getUserMedia API for image/video capture (WebRTC-Video)
(Assignee)

Updated

5 years ago
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]
(Assignee)

Comment 3

5 years ago
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]
(Assignee)

Comment 4

5 years ago
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
(Assignee)

Comment 10

5 years ago
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
Last Resolved: 5 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: 947853
No longer blocks: 922527
(Assignee)

Comment 14

5 years ago
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]
(Assignee)

Comment 15

5 years ago
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.
(Assignee)

Comment 17

5 years ago
Points taken. Thank you for the follow up.
Depends on: 970183
Blocks: 923364
(Assignee)

Comment 18

5 years ago
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]
(Assignee)

Updated

5 years ago
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
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Depends on: 977956
No longer depends on: 977956
Depends on: 977971
No longer depends on: 977971
Depends on: 978660
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
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Depends on: 1035819
Flags: in-moztrap?(jsmith)
You need to log in before you can comment on or make changes to this bug.