Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

Unified Diff: content/common/feature_policy/feature_policy.cc

Issue 2766213002: Update feature list (Closed)
Patch Set: Initial Impl Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/feature_policy/feature_policy.cc
diff --git a/content/common/feature_policy/feature_policy.cc b/content/common/feature_policy/feature_policy.cc
index f045f5a827524f01c6829e5785a6c2727207463a..068d8983c52e75a6b64f96a54fe7a825d5b76527 100644
--- a/content/common/feature_policy/feature_policy.cc
+++ b/content/common/feature_policy/feature_policy.cc
@@ -189,32 +189,38 @@ void FeaturePolicy::AddContainerPolicy(
// static
const FeaturePolicy::FeatureList& FeaturePolicy::GetDefaultFeatureList() {
CR_DEFINE_STATIC_LOCAL(FeatureList, default_feature_list,
- ({{blink::WebFeaturePolicyFeature::DocumentCookie,
- FeaturePolicy::FeatureDefault::EnableForAll},
- {blink::WebFeaturePolicyFeature::DocumentDomain,
- FeaturePolicy::FeatureDefault::EnableForAll},
- {blink::WebFeaturePolicyFeature::DocumentWrite,
- FeaturePolicy::FeatureDefault::EnableForAll},
+ ({{blink::WebFeaturePolicyFeature::Camera,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::Eme,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
{blink::WebFeaturePolicyFeature::Fullscreen,
FeaturePolicy::FeatureDefault::EnableForSelf},
{blink::WebFeaturePolicyFeature::Geolocation,
FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::Microphone,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
{blink::WebFeaturePolicyFeature::MidiFeature,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::Payment,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::Speaker,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::Vibrate,
+ FeaturePolicy::FeatureDefault::EnableForSelf},
+ {blink::WebFeaturePolicyFeature::DocumentCookie,
iclelland 2017/03/22 20:40:39 Can you put a comment above this line -- something
+ FeaturePolicy::FeatureDefault::EnableForAll},
+ {blink::WebFeaturePolicyFeature::DocumentDomain,
+ FeaturePolicy::FeatureDefault::EnableForAll},
+ {blink::WebFeaturePolicyFeature::DocumentWrite,
FeaturePolicy::FeatureDefault::EnableForAll},
{blink::WebFeaturePolicyFeature::Notifications,
FeaturePolicy::FeatureDefault::EnableForAll},
- {blink::WebFeaturePolicyFeature::Payment,
- FeaturePolicy::FeatureDefault::EnableForSelf},
{blink::WebFeaturePolicyFeature::Push,
FeaturePolicy::FeatureDefault::EnableForAll},
{blink::WebFeaturePolicyFeature::SyncScript,
FeaturePolicy::FeatureDefault::EnableForAll},
{blink::WebFeaturePolicyFeature::SyncXHR,
FeaturePolicy::FeatureDefault::EnableForAll},
- {blink::WebFeaturePolicyFeature::Usermedia,
- FeaturePolicy::FeatureDefault::EnableForAll},
- {blink::WebFeaturePolicyFeature::Vibrate,
- FeaturePolicy::FeatureDefault::EnableForSelf},
{blink::WebFeaturePolicyFeature::WebRTC,
FeaturePolicy::FeatureDefault::EnableForAll}}));
return default_feature_list;

Powered by Google App Engine
This is Rietveld 408576698