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

Unified Diff: third_party/WebKit/Source/web/RemoteFrameOwner.h

Issue 2564633002: Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/RemoteFrameOwner.h
diff --git a/third_party/WebKit/Source/web/RemoteFrameOwner.h b/third_party/WebKit/Source/web/RemoteFrameOwner.h
index a724b55abf49e587f18470a8279df53f4531f446..b2cfd3ee958e657eb9ed4a6aa5a34165498fc0d2 100644
--- a/third_party/WebKit/Source/web/RemoteFrameOwner.h
+++ b/third_party/WebKit/Source/web/RemoteFrameOwner.h
@@ -47,6 +47,7 @@ class RemoteFrameOwner final
int marginHeight() const override { return m_marginHeight; }
bool allowFullscreen() const override { return m_allowFullscreen; }
bool allowPaymentRequest() const override { return m_allowPaymentRequest; }
+ bool isDisplayNone() const override { return m_isDisplayNone; }
AtomicString csp() const override { return m_csp; }
const WebVector<WebPermissionType>& delegatedPermissions() const override {
return m_delegatedPermissions;
@@ -64,6 +65,7 @@ class RemoteFrameOwner final
void setAllowPaymentRequest(bool allowPaymentRequest) {
m_allowPaymentRequest = allowPaymentRequest;
}
+ void setIsDisplayNone(bool isDisplayNone) { m_isDisplayNone = isDisplayNone; }
void setCsp(const WebString& csp) { m_csp = csp; }
void setDelegatedpermissions(
const WebVector<WebPermissionType>& delegatedPermissions) {
@@ -88,6 +90,7 @@ class RemoteFrameOwner final
int m_marginHeight;
bool m_allowFullscreen;
bool m_allowPaymentRequest;
+ bool m_isDisplayNone;
WebString m_csp;
WebVector<WebPermissionType> m_delegatedPermissions;
};

Powered by Google App Engine
This is Rietveld 408576698