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

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

Issue 2743053003: [Reland #1] Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: Fix DOM object leaks. Diff this against Patch Set 10. Created 3 years, 9 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/public/web/WebFrameOwnerProperties.h
diff --git a/third_party/WebKit/public/web/WebFrameOwnerProperties.h b/third_party/WebKit/public/web/WebFrameOwnerProperties.h
index 5ba8d741452c0cd03e6bcd0a8fc674957c44ba90..e047c8617a1e0ec31e8e6529542f9629fc57a3bf 100644
--- a/third_party/WebKit/public/web/WebFrameOwnerProperties.h
+++ b/third_party/WebKit/public/web/WebFrameOwnerProperties.h
@@ -23,6 +23,7 @@ struct WebFrameOwnerProperties {
int marginHeight;
bool allowFullscreen;
bool allowPaymentRequest;
+ bool isDisplayNone;
WebString requiredCsp;
WebVector<mojom::PermissionName> delegatedPermissions;
@@ -34,7 +35,8 @@ struct WebFrameOwnerProperties {
marginWidth(-1),
marginHeight(-1),
allowFullscreen(false),
- allowPaymentRequest(false) {}
+ allowPaymentRequest(false),
+ isDisplayNone(false) {}
#if INSIDE_BLINK
WebFrameOwnerProperties(
@@ -44,6 +46,7 @@ struct WebFrameOwnerProperties {
int marginHeight,
bool allowFullscreen,
bool allowPaymentRequest,
+ bool isDisplayNone,
const WebString& requiredCsp,
const WebVector<mojom::PermissionName>& delegatedPermissions,
const WebVector<WebFeaturePolicyFeature>& allowedFeatures)
@@ -53,6 +56,7 @@ struct WebFrameOwnerProperties {
marginHeight(marginHeight),
allowFullscreen(allowFullscreen),
allowPaymentRequest(allowPaymentRequest),
+ isDisplayNone(isDisplayNone),
requiredCsp(requiredCsp),
delegatedPermissions(delegatedPermissions),
allowedFeatures(allowedFeatures) {}

Powered by Google App Engine
This is Rietveld 408576698