Closed
Bug 1487204
Opened 6 years ago
Closed 6 years ago
[macOS 10.14] Add platform support for calling authorizationStatusForMediaType and requestAccessForMediaType from JS
Categories
(Core :: Widget: Cocoa, enhancement, P1)
Tracking
()
RESOLVED
FIXED
mozilla64
People
(Reporter: haik, Assigned: haik)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
spohl
:
review+
pascalc
:
approval-mozilla-beta-
|
Details |
This bug is filed to cover adding support for authorizationStatusForMediaType and requestAccessForMediaType when we are running on macOS 10.14. On Mojave, access to the camera/mic from an application prompts the user to allow or deny access. 10.14 includes API's to query the current permission status and to request the permission.
I'm splitting the platform work out from bug 1479051 to this bug. See bug 1479051 for additional details.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → haftandilian
Priority: -- → P1
Assignee | ||
Comment 1•6 years ago
|
||
Add a new interface nsIOSPermissionRequest for querying the
staus of access permissions for audio/video media capture and
requesting access to audio/video capture devices. Provides an
implementation for macOS 10.14 and a default implementation
(nsOSPermissionRequestBase) for earlier macOS versions and other
platforms. The default implementation always returns status
indicating access is allowed.
Comment 2•6 years ago
|
||
Comment on attachment 9005012 [details]
Bug 1487204 - Add platform support for calling authorizationStatusForMediaType and requestAccessForMediaType from JS r?spohl
Stephen A Pohl [:spohl] has approved the revision.
Attachment #9005012 -
Flags: review+
Pushed by haftandilian@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cc550315583a
Add platform support for calling authorizationStatusForMediaType and requestAccessForMediaType from JS r=spohl
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Comment 5•6 years ago
|
||
Is this something we should consider for Beta uplift?
Flags: needinfo?(haftandilian)
Assignee | ||
Comment 6•6 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #5)
> Is this something we should consider for Beta uplift?
Once bug 1479051 is landed, we should consider uplifting the two to be ready for the release of macOS 10.14. Bug 1470833 is another 10.14 bug we will likely want to uplift, but it isn't in Nightly yet.
Flags: needinfo?(haftandilian)
Comment 7•6 years ago
|
||
(In reply to Haik Aftandilian [:haik] from comment #6)
> (In reply to Ryan VanderMeulen [:RyanVM] from comment #5)
> > Is this something we should consider for Beta uplift?
>
> Once bug 1479051 is landed, we should consider uplifting the two to be ready
> for the release of macOS 10.14. Bug 1470833 is another 10.14 bug we will
> likely want to uplift, but it isn't in Nightly yet.
Tracking for 63 as 10.14 is likely to be released end of September.
tracking-firefox63:
--- → +
Comment 8•6 years ago
|
||
Is this something we want to still consider for Beta uplift with a week left before the RC?
Flags: needinfo?(haftandilian)
QA Contact: mstange
Updated•6 years ago
|
QA Contact: mstange
Assignee | ||
Comment 9•6 years ago
|
||
(In reply to (PTO 8-Oct) Ryan VanderMeulen [:RyanVM] from comment #8)
> Is this something we want to still consider for Beta uplift with a week left
> before the RC?
No. This bug lays the groundwork for bug 1479051 which I hope to land this week and wouldn't want to uplift to Beta so late in the cycle.
Flags: needinfo?(haftandilian)
Comment 10•6 years ago
|
||
marking as wontfix for 63 as per comment #9
Assignee | ||
Comment 11•6 years ago
|
||
I've received some uplift requests for bug 1479051 and have filed the request for Beta on 1479051. I will file the uplift request for this bug this morning.
Assignee | ||
Comment 12•6 years ago
|
||
Comment on attachment 9005012 [details]
Bug 1487204 - Add platform support for calling authorizationStatusForMediaType and requestAccessForMediaType from JS r?spohl
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: N/A, changes in macOS Mojave
User impact if declined: This is the platform support for bug 1479051 with impact "User may get into a situation where they have allowed cam/mic permission in Firefox for a site, but Firefox does not have access to the cam/mic and as a result those devices do not work. From the site's perspective, the cam/mic appear to work, but the received camera stream is just black pixels."
Is this code covered by automated tests?: No
Has the fix been verified in Nightly?: Yes
Needs manual test from QE?: No
If yes, steps to reproduce:
List of other uplifts needed: None
Risk to taking this patch: Medium
Why is the change risky/not risky? (and alternatives if risky): Changes are partially covered by automated tests for WebRTC. The changes are not very complex, only affect Mac builds, and some of the code is only used on Mojave.
String changes made/needed: None
Attachment #9005012 -
Flags: approval-mozilla-beta?
Comment 13•6 years ago
|
||
Comment on attachment 9005012 [details]
Bug 1487204 - Add platform support for calling authorizationStatusForMediaType and requestAccessForMediaType from JS r?spohl
Not taking as we are not taking 1479051, thanks.
Attachment #9005012 -
Flags: approval-mozilla-beta? → approval-mozilla-beta-
Assignee | ||
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•