Replicate a parsed feature policy representation so it doesn't need to be parsed in the browser pro…

content/common/frame_replication_state.h

Index: content/common/frame_replication_state.h
diff --git a/content/common/frame_replication_state.h b/content/common/frame_replication_state.h
index cc8d4780d30615511950eba6ba1b31482c13bbea..8fcae1ca3e93f7b484463d536792dc631a87cac7 100644
--- a/content/common/frame_replication_state.h
+++ b/content/common/frame_replication_state.h
@@ -20,6 +20,19 @@ enum class WebSandboxFlags;
 
 namespace content {
 
+// This struct holds feature policy whitelist data that needs to be replicated
+// between a RenderFrame and any of its associated RenderFrameProxies. It forms
+// part of the FrameReplicationState below.

[alexmos, 2016/12/01 02:04:19]

nit: Maybe also mention that this is a whitelist for one feature, and the full
feature policy is a list of these.

[raymes, 2016/12/01 03:03:09]

Done.

+struct CONTENT_EXPORT FeaturePolicyParsedWhitelist {
+  FeaturePolicyParsedWhitelist();
+  FeaturePolicyParsedWhitelist(const FeaturePolicyParsedWhitelist& fppw);
+  ~FeaturePolicyParsedWhitelist();
+
+  std::string feature_name;
+  bool matches_all_origins;
+  std::vector<url::Origin> origins;
+};
+
 // This structure holds information that needs to be replicated between a
 // RenderFrame and any of its associated RenderFrameProxies.
 struct CONTENT_EXPORT FrameReplicationState {
@@ -84,9 +97,9 @@ struct CONTENT_EXPORT FrameReplicationState {
   // scratch.
   std::string unique_name;
 
-  // Feature policy header. May be empty if no header was sent with the
+  // Parsed feature policy header. May be empty if no header was sent with the
   // document.
-  std::string feature_policy_header;
+  std::vector<FeaturePolicyParsedWhitelist> feature_policy_header;

[alexmos, 2016/12/01 02:04:19]

Do we need to keep calling this feature_policy_header and not just
feature_policy?  Presumably yes, since a frame's feature policy might also
depend on <iframe> enable/disable attributes?

[raymes, 2016/12/01 03:03:09]

Exactly - this is to distinguish it from the attribute that we plan to add.

 
   // Accumulated CSP headers - gathered from http headers, <meta> elements,
   // parent frames (in case of about:blank frames).