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

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

Issue 2696283003: Revert of Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: 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 /* 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 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 bool canRenderFallbackContent() const override { return false; } 83 bool canRenderFallbackContent() const override { return false; }
84 void renderFallbackContent() override {} 84 void renderFallbackContent() override {}
85 AtomicString browsingContextContainerName() const override { 85 AtomicString browsingContextContainerName() const override {
86 return getAttribute(HTMLNames::nameAttr); 86 return getAttribute(HTMLNames::nameAttr);
87 } 87 }
88 ScrollbarMode scrollingMode() const override { return ScrollbarAuto; } 88 ScrollbarMode scrollingMode() const override { return ScrollbarAuto; }
89 int marginWidth() const override { return -1; } 89 int marginWidth() const override { return -1; }
90 int marginHeight() const override { return -1; } 90 int marginHeight() const override { return -1; }
91 bool allowFullscreen() const override { return false; } 91 bool allowFullscreen() const override { return false; }
92 bool allowPaymentRequest() const override { return false; } 92 bool allowPaymentRequest() const override { return false; }
93 bool isDisplayNone() const override { return !m_widget; }
94 AtomicString csp() const override { return nullAtom; } 93 AtomicString csp() const override { return nullAtom; }
95 const WebVector<mojom::blink::PermissionName>& delegatedPermissions() 94 const WebVector<mojom::blink::PermissionName>& delegatedPermissions()
96 const override; 95 const override;
97 96
98 DECLARE_VIRTUAL_TRACE(); 97 DECLARE_VIRTUAL_TRACE();
99 98
100 protected: 99 protected:
101 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&); 100 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&);
102 void setSandboxFlags(SandboxFlags); 101 void setSandboxFlags(SandboxFlags);
103 102
104 bool loadOrRedirectSubframe(const KURL&, 103 bool loadOrRedirectSubframe(const KURL&,
105 const AtomicString& frameName, 104 const AtomicString& frameName,
106 bool replaceCurrentItem); 105 bool replaceCurrentItem);
107 bool isKeyboardFocusable() const override; 106 bool isKeyboardFocusable() const override;
108 107
109 void disposeWidgetSoon(Widget*); 108 void disposeWidgetSoon(Widget*);
110 void frameOwnerPropertiesChanged();
111 109
112 private: 110 private:
113 // Intentionally private to prevent redundant checks when the type is 111 // Intentionally private to prevent redundant checks when the type is
114 // already HTMLFrameOwnerElement. 112 // already HTMLFrameOwnerElement.
115 bool isLocal() const final { return true; } 113 bool isLocal() const final { return true; }
116 bool isRemote() const final { return false; } 114 bool isRemote() const final { return false; }
117 115
118 bool isFrameOwnerElement() const final { return true; } 116 bool isFrameOwnerElement() const final { return true; }
119 117
120 virtual ReferrerPolicy referrerPolicyAttribute() { 118 virtual ReferrerPolicy referrerPolicyAttribute() {
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 166
169 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement, 167 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement,
170 FrameOwner, 168 FrameOwner,
171 owner, 169 owner,
172 owner->isLocal(), 170 owner->isLocal(),
173 owner.isLocal()); 171 owner.isLocal());
174 172
175 } // namespace blink 173 } // namespace blink
176 174
177 #endif // HTMLFrameOwnerElement_h 175 #endif // HTMLFrameOwnerElement_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698