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

Unified Diff: Source/core/svg/SVGElement.h

Issue 334223009: Oilpan: Mark SVGElement::InvalidationGuard and SVGElement::InstanceUpdateBlocker STACK_ALLOCATED(). (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/svg/SVGElement.h
diff --git a/Source/core/svg/SVGElement.h b/Source/core/svg/SVGElement.h
index 9e3070679bf0fce2393bfaef099691f578357437..753a5aff8e1fafa1139d5c2594a49914cbddbe90 100644
--- a/Source/core/svg/SVGElement.h
+++ b/Source/core/svg/SVGElement.h
@@ -160,22 +160,25 @@ public:
bool inUseShadowTree() const;
class InvalidationGuard {
+ STACK_ALLOCATED();
WTF_MAKE_NONCOPYABLE(InvalidationGuard);
public:
InvalidationGuard(SVGElement* element) : m_element(element) { }
haraken 2014/06/17 00:52:48 Add explicit.
tkent 2014/06/17 01:40:47 This CL is not responsible for existing style issu
~InvalidationGuard() { m_element->invalidateInstances(); }
+
private:
- SVGElement* m_element;
+ RawPtrWillBeMember<SVGElement> m_element;
};
class InstanceUpdateBlocker {
+ STACK_ALLOCATED();
WTF_MAKE_NONCOPYABLE(InstanceUpdateBlocker);
public:
InstanceUpdateBlocker(SVGElement* targetElement);
haraken 2014/06/17 00:52:48 Add explicit.
~InstanceUpdateBlocker();
private:
- SVGElement* m_targetElement;
+ RawPtrWillBeMember<SVGElement> m_targetElement;
};
void invalidateInstances();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698