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

Side by Side Diff: third_party/WebKit/Source/core/frame/FrameOwner.h

Issue 2697713003: Initial Implementation of Iframe Attribute for Feature Policy (Part 2) (Closed)
Patch Set: Update rebase Created 3 years, 10 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 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef FrameOwner_h 5 #ifndef FrameOwner_h
6 #define FrameOwner_h 6 #define FrameOwner_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "core/dom/SandboxFlags.h" 9 #include "core/dom/SandboxFlags.h"
10 #include "platform/heap/Handle.h" 10 #include "platform/heap/Handle.h"
11 #include "platform/scroll/ScrollTypes.h" 11 #include "platform/scroll/ScrollTypes.h"
12 #include "public/platform/WebFeaturePolicy.h"
12 #include "public/platform/WebVector.h" 13 #include "public/platform/WebVector.h"
13 #include "public/platform/modules/permissions/permission.mojom-blink.h" 14 #include "public/platform/modules/permissions/permission.mojom-blink.h"
14 15
15 namespace blink { 16 namespace blink {
16 17
17 class Frame; 18 class Frame;
18 19
19 // Oilpan: all FrameOwner instances are GCed objects. FrameOwner additionally 20 // Oilpan: all FrameOwner instances are GCed objects. FrameOwner additionally
20 // derives from GarbageCollectedMixin so that Member<FrameOwner> references can 21 // derives from GarbageCollectedMixin so that Member<FrameOwner> references can
21 // be kept (e.g., Frame::m_owner.) 22 // be kept (e.g., Frame::m_owner.)
(...skipping 22 matching lines...) Expand all
44 // https://html.spec.whatwg.org/multipage/browsers.html#browsing-context-conta iner 45 // https://html.spec.whatwg.org/multipage/browsers.html#browsing-context-conta iner
45 virtual AtomicString browsingContextContainerName() const = 0; 46 virtual AtomicString browsingContextContainerName() const = 0;
46 virtual ScrollbarMode scrollingMode() const = 0; 47 virtual ScrollbarMode scrollingMode() const = 0;
47 virtual int marginWidth() const = 0; 48 virtual int marginWidth() const = 0;
48 virtual int marginHeight() const = 0; 49 virtual int marginHeight() const = 0;
49 virtual bool allowFullscreen() const = 0; 50 virtual bool allowFullscreen() const = 0;
50 virtual bool allowPaymentRequest() const = 0; 51 virtual bool allowPaymentRequest() const = 0;
51 virtual AtomicString csp() const = 0; 52 virtual AtomicString csp() const = 0;
52 virtual const WebVector<mojom::blink::PermissionName>& delegatedPermissions() 53 virtual const WebVector<mojom::blink::PermissionName>& delegatedPermissions()
53 const = 0; 54 const = 0;
55 virtual const WebVector<WebFeaturePolicyFeature>& allowedFeatures() const = 0;
54 }; 56 };
55 57
56 // TODO(dcheng): This class is an internal implementation detail of provisional 58 // TODO(dcheng): This class is an internal implementation detail of provisional
57 // frames. Move this into WebLocalFrameImpl.cpp and remove existing dependencies 59 // frames. Move this into WebLocalFrameImpl.cpp and remove existing dependencies
58 // on it. 60 // on it.
59 class CORE_EXPORT DummyFrameOwner 61 class CORE_EXPORT DummyFrameOwner
60 : public GarbageCollectedFinalized<DummyFrameOwner>, 62 : public GarbageCollectedFinalized<DummyFrameOwner>,
61 public FrameOwner { 63 public FrameOwner {
62 USING_GARBAGE_COLLECTED_MIXIN(DummyFrameOwner); 64 USING_GARBAGE_COLLECTED_MIXIN(DummyFrameOwner);
63 65
(...skipping 18 matching lines...) Expand all
82 int marginHeight() const override { return -1; } 84 int marginHeight() const override { return -1; }
83 bool allowFullscreen() const override { return false; } 85 bool allowFullscreen() const override { return false; }
84 bool allowPaymentRequest() const override { return false; } 86 bool allowPaymentRequest() const override { return false; }
85 AtomicString csp() const override { return nullAtom; } 87 AtomicString csp() const override { return nullAtom; }
86 const WebVector<mojom::blink::PermissionName>& delegatedPermissions() 88 const WebVector<mojom::blink::PermissionName>& delegatedPermissions()
87 const override { 89 const override {
88 DEFINE_STATIC_LOCAL(WebVector<mojom::blink::PermissionName>, permissions, 90 DEFINE_STATIC_LOCAL(WebVector<mojom::blink::PermissionName>, permissions,
89 ()); 91 ());
90 return permissions; 92 return permissions;
91 } 93 }
94 const WebVector<WebFeaturePolicyFeature>& allowedFeatures() const override {
95 DEFINE_STATIC_LOCAL(WebVector<WebFeaturePolicyFeature>, features, ());
96 return features;
97 }
92 98
93 private: 99 private:
94 // Intentionally private to prevent redundant checks when the type is 100 // Intentionally private to prevent redundant checks when the type is
95 // already DummyFrameOwner. 101 // already DummyFrameOwner.
96 bool isLocal() const override { return false; } 102 bool isLocal() const override { return false; }
97 bool isRemote() const override { return false; } 103 bool isRemote() const override { return false; }
98 }; 104 };
99 105
100 } // namespace blink 106 } // namespace blink
101 107
102 #endif // FrameOwner_h 108 #endif // FrameOwner_h
OLDNEW
« no previous file with comments | « content/test/data/allowed_frames.html ('k') | third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698