Add an iframe permissions= attribute for implementing permission delegation

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

Index: third_party/WebKit/Source/core/html/HTMLIFrameElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLIFrameElement.h b/third_party/WebKit/Source/core/html/HTMLIFrameElement.h
index 752d0c86fd74027d6b6bfba9a66867b963511cd0..885e35763d76143a600d6d055afc9d6ef5bd6567 100644
--- a/third_party/WebKit/Source/core/html/HTMLIFrameElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLIFrameElement.h
@@ -24,19 +24,26 @@
 #ifndef HTMLIFrameElement_h
 #define HTMLIFrameElement_h
 
+#include "core/CoreExport.h"
 #include "core/html/HTMLFrameElementBase.h"
+#include "core/html/HTMLIFrameElementPermissions.h"
 #include "core/html/HTMLIFrameElementSandbox.h"
+#include "public/platform/WebVector.h"
+#include "public/platform/modules/permissions/WebPermissionType.h"
 
 namespace blink {
 
-class HTMLIFrameElement final : public HTMLFrameElementBase, public DOMTokenListObserver {
+class CORE_EXPORT HTMLIFrameElement final : public HTMLFrameElementBase {
     DEFINE_WRAPPERTYPEINFO();
-    USING_GARBAGE_COLLECTED_MIXIN(HTMLIFrameElement);
 public:
     DECLARE_NODE_FACTORY(HTMLIFrameElement);
     DECLARE_VIRTUAL_TRACE();
     ~HTMLIFrameElement() override;
     DOMTokenList* sandbox() const;
+    DOMTokenList* permissions() const;
+
+    void sandboxValueWasSet();
+    void permissionsValueWasSet();
 
 private:
     explicit HTMLIFrameElement(Document&);
@@ -55,16 +62,21 @@ private:
     void didLoadNonEmptyDocument() override { m_didLoadNonEmptyDocument = true; }
     bool isInteractiveContent() const override;
 
-    void valueWasSet() override;
-
     ReferrerPolicy referrerPolicyAttribute() override;
 
     bool allowFullscreen() const override { return m_allowFullscreen; }
 
+    const WebVector<WebPermissionType>* delegatedPermissions() const override { return &m_delegatedPermissions; }

[sof, 2016/06/28 05:29:00]

What is the assumed lifetime of this value? i.e., you're returning a
non-traceable pointer into a Blink heap-allocated object (which might be garbage
collected.)

[raymes, 2016/06/28 06:49:46]

The pointer is just used to reduce copied but the value is used synchronously
and will get copied into the frame owner properties struct.

[sof, 2016/06/28 06:57:48]

ok, it seems unnecessary to resort to returning a pointer here if the call site
substitutes |nullptr| with an empty WebVector.

[raymes, 2016/06/29 05:54:47]

I was trying to reduce copies :) I've returned a const reference instead, which
requires using a static local in the places where null is currently returned.
Otherwise I can return by copy if you feel strongly.

[sof, 2016/06/29 06:10:28]

When & where to optimize is a perennial question, but I think this is fine.

+
+    bool initializePermissionsAttribute();
+
     AtomicString m_name;
     bool m_didLoadNonEmptyDocument;
     bool m_allowFullscreen;
     Member<HTMLIFrameElementSandbox> m_sandbox;
+    Member<HTMLIFrameElementPermissions> m_permissions;
+
+    WebVector<WebPermissionType> m_delegatedPermissions;
 
     ReferrerPolicy m_referrerPolicy;
 };