Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(572)

Side by Side Diff: Source/core/html/HTMLFrameOwnerElement.h

Issue 838903002: Replicate sandbox flags for OOPIF (Blink part 2) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@iframe-sandbox-flags-part1
Patch Set: Address Daniel's comments. Move sandbox flags inheritance to FrameLoader::effectiveSandboxFlags(). Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 70
71 class UpdateSuspendScope { 71 class UpdateSuspendScope {
72 public: 72 public:
73 UpdateSuspendScope(); 73 UpdateSuspendScope();
74 ~UpdateSuspendScope(); 74 ~UpdateSuspendScope();
75 75
76 private: 76 private:
77 void performDeferredWidgetTreeOperations(); 77 void performDeferredWidgetTreeOperations();
78 }; 78 };
79 79
80 // FrameOwner overrides:
81 virtual bool isLocal() const { return true; }
82 virtual void dispatchLoad() override;
83 virtual SandboxFlags sandboxFlags() const override { return m_sandboxFlags; }
84
80 virtual void trace(Visitor*) override; 85 virtual void trace(Visitor*) override;
81 86
82 protected: 87 protected:
83 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&); 88 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&);
84 void setSandboxFlags(SandboxFlags); 89 void setSandboxFlags(SandboxFlags);
85 90
86 bool loadOrRedirectSubframe(const KURL&, const AtomicString& frameName, bool lockBackForwardList, ContentSecurityPolicyDisposition shouldCheckContentSecurit yPolicy); 91 bool loadOrRedirectSubframe(const KURL&, const AtomicString& frameName, bool lockBackForwardList, ContentSecurityPolicyDisposition shouldCheckContentSecurit yPolicy);
87 92
88 private: 93 private:
89 virtual bool isKeyboardFocusable() const override; 94 virtual bool isKeyboardFocusable() const override;
90 virtual bool isFrameOwnerElement() const override final { return true; } 95 virtual bool isFrameOwnerElement() const override final { return true; }
91 96
92 // FrameOwner overrides:
93 virtual bool isLocal() const { return true; }
94 virtual SandboxFlags sandboxFlags() const override;
95 virtual void dispatchLoad() override;
96
97 RawPtrWillBeMember<Frame> m_contentFrame; 97 RawPtrWillBeMember<Frame> m_contentFrame;
98 RefPtrWillBeMember<Widget> m_widget; 98 RefPtrWillBeMember<Widget> m_widget;
99 SandboxFlags m_sandboxFlags; 99 SandboxFlags m_sandboxFlags;
100 }; 100 };
101 101
102 DEFINE_ELEMENT_TYPE_CASTS(HTMLFrameOwnerElement, isFrameOwnerElement()); 102 DEFINE_ELEMENT_TYPE_CASTS(HTMLFrameOwnerElement, isFrameOwnerElement());
103 103
104 class SubframeLoadingDisabler { 104 class SubframeLoadingDisabler {
105 STACK_ALLOCATED(); 105 STACK_ALLOCATED();
106 public: 106 public:
(...skipping 23 matching lines...) Expand all
130 static WillBeHeapHashCountedSet<RawPtrWillBeMember<Node>>& disabledSubtreeRo ots(); 130 static WillBeHeapHashCountedSet<RawPtrWillBeMember<Node>>& disabledSubtreeRo ots();
131 131
132 RawPtrWillBeMember<Node> m_root; 132 RawPtrWillBeMember<Node> m_root;
133 }; 133 };
134 134
135 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement, FrameOwner, owner, owner->isLocal(), ow ner.isLocal()); 135 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement, FrameOwner, owner, owner->isLocal(), ow ner.isLocal());
136 136
137 } // namespace blink 137 } // namespace blink
138 138
139 #endif // HTMLFrameOwnerElement_h 139 #endif // HTMLFrameOwnerElement_h
OLDNEW
« no previous file with comments | « no previous file | Source/core/html/HTMLFrameOwnerElement.cpp » ('j') | Source/core/loader/FrameLoader.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698