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

Unified Diff: third_party/WebKit/public/web/WebFrameOwnerProperties.h

Issue 2564633002: Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: Add new layout tests. Created 4 years 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/public/web/WebFrameOwnerProperties.h
diff --git a/third_party/WebKit/public/web/WebFrameOwnerProperties.h b/third_party/WebKit/public/web/WebFrameOwnerProperties.h
index ec2f2a39fbde73ecfe86303fa75df5165a735a34..734c29650756cfa8a1c12efa9927dcd2689ae92f 100644
--- a/third_party/WebKit/public/web/WebFrameOwnerProperties.h
+++ b/third_party/WebKit/public/web/WebFrameOwnerProperties.h
@@ -19,6 +19,7 @@ struct WebFrameOwnerProperties {
int marginWidth;
int marginHeight;
bool allowFullscreen;
+ bool isDisplayNone;
WebString requiredCsp;
WebVector<WebPermissionType> delegatedPermissions;
@@ -26,7 +27,8 @@ struct WebFrameOwnerProperties {
: scrollingMode(ScrollingMode::Auto),
marginWidth(-1),
marginHeight(-1),
- allowFullscreen(false) {}
+ allowFullscreen(false),
+ isDisplayNone(false) {}
#if INSIDE_BLINK
WebFrameOwnerProperties(
@@ -34,12 +36,14 @@ struct WebFrameOwnerProperties {
int marginWidth,
int marginHeight,
bool allowFullscreen,
+ bool isDisplayNone,
const WebString& requiredCsp,
const WebVector<WebPermissionType>& delegatedPermissions)
: scrollingMode(static_cast<ScrollingMode>(scrollingMode)),
marginWidth(marginWidth),
marginHeight(marginHeight),
allowFullscreen(allowFullscreen),
+ isDisplayNone(isDisplayNone),
requiredCsp(requiredCsp),
delegatedPermissions(delegatedPermissions) {}
#endif

Powered by Google App Engine
This is Rietveld 408576698