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

Unified Diff: Source/core/css/resolver/StyleResolverState.cpp

Issue 5216392399814656: Clear StyleResolverState after each resolve. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Updated after levi's revert Created 7 years, 5 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
« no previous file with comments | « Source/core/css/resolver/StyleResolverState.h ('k') | Source/core/html/canvas/CanvasRenderingContext2D.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/css/resolver/StyleResolverState.cpp
diff --git a/Source/core/css/resolver/StyleResolverState.cpp b/Source/core/css/resolver/StyleResolverState.cpp
index 9ab8a9d2cd4c2b906ef9b615fe6e46539c027f3b..bc6399a8462e9fc6f3c06d570e90f19cde26fd8b 100644
--- a/Source/core/css/resolver/StyleResolverState.cpp
+++ b/Source/core/css/resolver/StyleResolverState.cpp
@@ -47,11 +47,23 @@ ElementResolveContext::ElementResolveContext(Element* element)
m_rootElementStyle = documentElement && element != documentElement ? documentElement->renderStyle() : documentStyle;
}
+StyleResolveScope::StyleResolveScope(StyleResolverState* state, Document* document, Element* e, RenderStyle* parentStyle, RenderRegion* regionForStyling)
+ : m_state(state)
+{
+ m_state->initForStyleResolve(document, e, parentStyle, regionForStyling);
+}
+
+StyleResolveScope::~StyleResolveScope()
+{
+ m_state->clear();
+}
+
void StyleResolverState::clear()
{
// FIXME: Use m_elementContent = ElementContext() instead.
m_elementContext.deprecatedPartialClear();
+ m_style = 0;
m_parentStyle = 0;
m_regionForStyling = 0;
m_elementStyleResources.clear();
« no previous file with comments | « Source/core/css/resolver/StyleResolverState.h ('k') | Source/core/html/canvas/CanvasRenderingContext2D.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698