Replicate feature policy container policies.

third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h

Index: third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h b/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
index 6ff75dfc26c709727cc0b8a8cf3e177441eccceb..9ff725dcaccd3d7c735bae8d74c03c0428c8a9ce 100644
--- a/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
@@ -25,6 +25,7 @@
 #include "core/dom/Document.h"
 #include "core/frame/FrameOwner.h"
 #include "core/html/HTMLElement.h"
+#include "platform/feature_policy/FeaturePolicy.h"
 #include "platform/heap/Handle.h"
 #include "platform/scroll/ScrollTypes.h"
 #include "platform/weborigin/SecurityPolicy.h"
@@ -93,6 +94,7 @@ class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
   bool isDisplayNone() const override { return !m_widget; }
   AtomicString csp() const override { return nullAtom; }
   const WebVector<WebFeaturePolicyFeature>& allowedFeatures() const override;
+  const WebParsedFeaturePolicy& containerPolicy() const override;
 
   DECLARE_VIRTUAL_TRACE();
 
@@ -107,6 +109,7 @@ class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
 
   void disposeWidgetSoon(FrameViewBase*);
   void frameOwnerPropertiesChanged();
+  void updateContainerPolicy();
 
  private:
   // Intentionally private to prevent redundant checks when the type is
@@ -123,6 +126,11 @@ class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
   Member<Frame> m_contentFrame;
   Member<FrameViewBase> m_widget;
   SandboxFlags m_sandboxFlags;
+
+  WebParsedFeaturePolicy m_containerPolicy;
+
+  // Tracks the absolute URL represented by the contents of the src attribute.
+  KURL m_absoluteURL;

[alexmos, 2017/04/06 00:44:22]

Why do we need to track this?  Could you get it instead just by looking up the
src attribute?  (I recall some Blink folks weren't fond of tracking too many
(slightly) different kinds of URLs, at least on Document.  dcheng@ knows the
historical perspective on that)

[iclelland, 2017/04/09 03:25:54]

My concern here was that the absolute URL isn't just what's in the src
attribute, various kinds of relative URLs need to combine with the parent
frame's base to determine how the src attribute is interpreted.

I was thinking that that meant we needed to hold on to it for longer than just
in |HTMLFrameOwnerElement::loadOrRedirectSubframe|, but as Luna pointed out, the
new code doesn't actually use it anywhere else, so I don't think we need to
store it.

I've removed it, and, since this class doesn't have any other concept of
location, moved the responsibility for calculating the relevant origin into
HTMLFrameElementBase.

 };
 
 DEFINE_ELEMENT_TYPE_CASTS(HTMLFrameOwnerElement, isFrameOwnerElement());