Commit a2d89c5a authored by lunalu's avatar lunalu Committed by Commit bot

Ship navigator.vibrate behind featurePolicyExperimentalFeatures flag

Due to complexity between user gesture and vibrate, we decided to remove vibrate from V1 feature policy.

Review-Url: https://codereview.chromium.org/2801153002
Cr-Commit-Position: refs/heads/master@{#462893}
parent b4c4b350
...@@ -81,6 +81,7 @@ bool NavigatorVibration::vibrate(Navigator& navigator, ...@@ -81,6 +81,7 @@ bool NavigatorVibration::vibrate(Navigator& navigator,
// TODO(lunalu): When FeaturePolicy is ready, take out the check for the // TODO(lunalu): When FeaturePolicy is ready, take out the check for the
// runtime flag. Please pay attention to the user gesture code below. // runtime flag. Please pay attention to the user gesture code below.
if (RuntimeEnabledFeatures::featurePolicyEnabled() && if (RuntimeEnabledFeatures::featurePolicyEnabled() &&
RuntimeEnabledFeatures::featurePolicyExperimentalFeaturesEnabled() &&
!frame->isFeatureEnabled(blink::WebFeaturePolicyFeature::Vibrate)) { !frame->isFeatureEnabled(blink::WebFeaturePolicyFeature::Vibrate)) {
frame->domWindow()->printErrorMessage( frame->domWindow()->printErrorMessage(
"Navigator.vibrate() is not enabled in feature policy for this " "Navigator.vibrate() is not enabled in feature policy for this "
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment