New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 666765 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 689802
Owner:
OOO until 4th Feb
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Feature



Sign in to add a comment

Feature Policy: Support Camera, Microphone, and Speaker API's

Project Member Reported by iclell...@chromium.org, Nov 18 2016

Issue description

Add support for features "camera", "speaker", and "microphone" in Feature Policy.

The features will be defined in FeaturePolicy.cpp, replacing "usermedia". 

If not allowed by policy, the call to these features will result in the error callback being called with a NotAllowedError.

Those features are permissions-based API, so that should integrate with Permission Delegation.
 

Comment 1 by mcasas@chromium.org, Nov 21 2016

Components: -Blink>WebRTC Blink>GetUserMedia
Owner: lunalu@chromium.org
Status: Started (was: Available)
Components: Blink>FeaturePolicy

Comment 4 by lunalu@chromium.org, Mar 21 2017

Status: Assigned (was: Started)
Summary: Feature Policy: Support Camera, Microphone, and Speaker API (was: Feature Policy: Support GetUserMedia API)

Comment 5 by lunalu@chromium.org, Mar 21 2017

Description: Show this description

Comment 6 by lunalu@chromium.org, Mar 21 2017

Cc: raymes@chromium.org
raymes, are you planning to work on those features? I am landing a CL to update the feature list which will unblock you. If you are not planning to work on these features, I can certainly work on them :)

Comment 7 by lunalu@chromium.org, Mar 21 2017

Summary: Feature Policy: Support Camera, Microphone, and Speaker API's (was: Feature Policy: Support Camera, Microphone, and Speaker API)
Blocking: -623682
Labels: -Feature-Policy Feature-Policy-V2
Not blocking initial launch.
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0725ef5c886c6386de8a5dbca0700ba79e7dace3

commit 0725ef5c886c6386de8a5dbca0700ba79e7dace3
Author: lunalu <lunalu@chromium.org>
Date: Tue Mar 28 19:16:33 2017

Update feature list

Update the list to match the spec draft
https://rawgit.com/clelland/feature-policy/update-spec/index.html#feature

I changed the order of the Enums following the spec.
I kept previously defined features, but replace usermedia
by camera, speaker, and microphone.

BUG= 666765 

Review-Url: https://codereview.chromium.org/2766213002
Cr-Commit-Position: refs/heads/master@{#460184}

[modify] https://crrev.com/0725ef5c886c6386de8a5dbca0700ba79e7dace3/content/common/feature_policy/feature_policy.cc
[modify] https://crrev.com/0725ef5c886c6386de8a5dbca0700ba79e7dace3/third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp
[modify] https://crrev.com/0725ef5c886c6386de8a5dbca0700ba79e7dace3/third_party/WebKit/public/platform/WebFeaturePolicy.h

Cc: -raymes@chromium.org lunalu@chromium.org
Owner: raymes@chromium.org
Cc: -lunalu@chromium.org loonyb...@chromium.org
Labels: Type-Feature
Raymes, this is already implemented, right? Can we close this issue (or merge into 689802)?
Mergedinto: 689802
Status: Duplicate (was: Assigned)

Sign in to add a comment