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

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

Issue 2743053003: [Reland #1] Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: rebaseline. Created 3 years, 8 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; }
93 AtomicString csp() const override { return nullAtom; } 94 AtomicString csp() const override { return nullAtom; }
94 const WebVector<WebFeaturePolicyFeature>& allowedFeatures() const override; 95 const WebVector<WebFeaturePolicyFeature>& allowedFeatures() const override;
95 96
96 DECLARE_VIRTUAL_TRACE(); 97 DECLARE_VIRTUAL_TRACE();
97 98
98 protected: 99 protected:
99 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&); 100 HTMLFrameOwnerElement(const QualifiedName& tagName, Document&);
100 void setSandboxFlags(SandboxFlags); 101 void setSandboxFlags(SandboxFlags);
101 102
102 bool loadOrRedirectSubframe(const KURL&, 103 bool loadOrRedirectSubframe(const KURL&,
103 const AtomicString& frameName, 104 const AtomicString& frameName,
104 bool replaceCurrentItem); 105 bool replaceCurrentItem);
105 bool isKeyboardFocusable() const override; 106 bool isKeyboardFocusable() const override;
106 107
107 void disposeWidgetSoon(FrameViewBase*); 108 void disposeWidgetSoon(FrameViewBase*);
109 void frameOwnerPropertiesChanged();
108 110
109 private: 111 private:
110 // Intentionally private to prevent redundant checks when the type is 112 // Intentionally private to prevent redundant checks when the type is
111 // already HTMLFrameOwnerElement. 113 // already HTMLFrameOwnerElement.
112 bool isLocal() const final { return true; } 114 bool isLocal() const final { return true; }
113 bool isRemote() const final { return false; } 115 bool isRemote() const final { return false; }
114 116
115 bool isFrameOwnerElement() const final { return true; } 117 bool isFrameOwnerElement() const final { return true; }
116 118
117 virtual ReferrerPolicy referrerPolicyAttribute() { 119 virtual ReferrerPolicy referrerPolicyAttribute() {
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 167
166 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement, 168 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement,
167 FrameOwner, 169 FrameOwner,
168 owner, 170 owner,
169 owner->isLocal(), 171 owner->isLocal(),
170 owner.isLocal()); 172 owner.isLocal());
171 173
172 } // namespace blink 174 } // namespace blink
173 175
174 #endif // HTMLFrameOwnerElement_h 176 #endif // HTMLFrameOwnerElement_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698