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

Unified Diff: third_party/WebKit/Source/core/dom/Element.cpp

Issue 2564633002: Don't create layout objects for children of display-none iframes. (Closed)
Patch Set: . 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/Source/core/dom/Element.cpp
diff --git a/third_party/WebKit/Source/core/dom/Element.cpp b/third_party/WebKit/Source/core/dom/Element.cpp
index 7c286936357b055b17ba342c2581c8daf2be69cc..5c2895c103340ff6dd0ccf1f44d797c47896cd2d 100644
--- a/third_party/WebKit/Source/core/dom/Element.cpp
+++ b/third_party/WebKit/Source/core/dom/Element.cpp
@@ -1944,6 +1944,11 @@ StyleRecalcChange Element::recalcOwnStyle(StyleRecalcChange change,
INCREMENT_STYLE_STATS_COUNTER(document().styleEngine(), stylesChanged, 1);
}
+ // TODO: This is the wrong approach, but gets the job done. Find a more
+ // appropriate way of doing the same thing.
+ if (forceReattachDuringStyleRecalc())
esprehn 2016/12/10 02:20:55 Remove this, we don't want an extra branch for eve
+ localChange = Reattach;
+
if (localChange == Reattach) {
StyleReattachData styleReattachData;
styleReattachData.computedStyle = std::move(newStyle);

Powered by Google App Engine
This is Rietveld 408576698