Add tests for FeaturePolicy integration with RenderFrameHost

content/common/feature_policy/feature_policy.cc

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 76ad76989dee2d051b19a7b32c2d50853f755d3b..29edc9c8240a6d8f049bbdcc3dc51eb717f7f88a 100644
--- a/content/common/feature_policy/feature_policy.cc
+++ b/content/common/feature_policy/feature_policy.cc
@@ -146,6 +146,10 @@ void FeaturePolicy::SetHeaderPolicy(
   }
 }
 
+bool FeaturePolicy::HasSetHeaderPolicy() const {
+  return !whitelists_.empty();

[alexmos, 2017/05/17 22:48:49]

Can you set an empty policy in the header, one that would keep whitelists_
empty?  If so, it seems like this would return the wrong thing here.

[raymes, 2017/05/21 22:11:20]

Ahh good catch. I've removed this for now, as right now we don't have the state
anywhere to check this. If you felt comfortable about it, I could add a bool
which was set to true in RenderFrameHostImpl::OnDidSetFeaturePolicyHeader but it
would only be for DCHECKing.

[alexmos, 2017/05/22 23:05:47]

I'm fine leaving it out until there's a more substantial need. :)

+}
+
 FeaturePolicy::FeaturePolicy(url::Origin origin,
                              const FeatureList& feature_list)
     : origin_(origin), feature_list_(feature_list) {}