Move feature policy vibrate tests to correct directory

third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/feature_policy/FeaturePolicy.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/json/JSONValues.h"
 #include "platform/network/HTTPParsers.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "platform/wtf/PtrUtil.h"
 
 namespace blink {
 
 WebFeaturePolicyFeature GetWebFeaturePolicyFeature(const String& feature) {
   if (feature == "fullscreen")
     return WebFeaturePolicyFeature::kFullscreen;
   if (feature == "payment")
     return WebFeaturePolicyFeature::kPayment;
-  if (feature == "vibrate")
-    return WebFeaturePolicyFeature::kVibrate;
   if (RuntimeEnabledFeatures::featurePolicyExperimentalFeaturesEnabled()) {
+    if (feature == "vibrate")
+      return WebFeaturePolicyFeature::kVibrate;
     if (feature == "camera")
       return WebFeaturePolicyFeature::kCamera;
     if (feature == "eme")
       return WebFeaturePolicyFeature::kEme;
     if (feature == "microphone")
       return WebFeaturePolicyFeature::kMicrophone;
     if (feature == "speaker")
       return WebFeaturePolicyFeature::kSpeaker;
     if (feature == "cookie")
       return WebFeaturePolicyFeature::kDocumentCookie;
     if (feature == "domain")
       return WebFeaturePolicyFeature::kDocumentDomain;
     if (feature == "docwrite")
       return WebFeaturePolicyFeature::kDocumentWrite;
     if (feature == "geolocation")
       return WebFeaturePolicyFeature::kGeolocation;
     if (feature == "midi")
       return WebFeaturePolicyFeature::kMidiFeature;
     if (feature == "notifications")
       return WebFeaturePolicyFeature::kNotifications;
     if (feature == "push")
       return WebFeaturePolicyFeature::kPush;
     if (feature == "sync-script")
       return WebFeaturePolicyFeature::kSyncScript;
     if (feature == "sync-xhr")
       return WebFeaturePolicyFeature::kSyncXHR;
     if (feature == "webrtc")
       return WebFeaturePolicyFeature::kWebRTC;
   }
   return WebFeaturePolicyFeature::kNotFound;

[iclelland, 2017/04/19 17:23:28]

Can you check to see if kNotFound is used anywhere else in the codebase? I
suspect we can get rid of it, that it was only used here.

 }
 
 WebParsedFeaturePolicy ParseFeaturePolicy(const String& policy,
                                           RefPtr<SecurityOrigin> origin,
                                           Vector<String>* messages) {
   Vector<WebParsedFeaturePolicyDeclaration> whitelists;
 
   // Use a reasonable parse depth limit; the actual maximum depth is only going
   // to be 4 for a valid policy, but we'll give the featurePolicyParser a chance
   // to report more specific errors, unless the string is really invalid.
@@ skipping 62 matching lines @@
   for (const WebFeaturePolicyFeature feature : features) {
     WebParsedFeaturePolicyDeclaration whitelist;
     whitelist.feature = feature;
     whitelist.origins = Vector<WebSecurityOrigin>(1UL, {origin});
     whitelists.push_back(whitelist);
   }
   return whitelists;
 }
 
 }  // namespace blink