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

Side by Side Diff: Source/core/dom/Element.cpp

Issue 236653002: Oilpan: move mutation observers to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebased + explicitly dispose() mutation observer registrations always (non-Oilpan also.) Created 6 years, 8 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Peter Kelly (pmk@post.com) 4 * (C) 2001 Peter Kelly (pmk@post.com)
5 * (C) 2001 Dirk Mueller (mueller@kde.org) 5 * (C) 2001 Dirk Mueller (mueller@kde.org)
6 * (C) 2007 David Smith (catfish.man@gmail.com) 6 * (C) 2007 David Smith (catfish.man@gmail.com)
7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved. 7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
8 * (C) 2007 Eric Seidel (eric@webkit.org) 8 * (C) 2007 Eric Seidel (eric@webkit.org)
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 2774 matching lines...) Expand 10 before | Expand all | Expand 10 after
2785 } 2785 }
2786 2786
2787 if (oldValue != newValue) { 2787 if (oldValue != newValue) {
2788 if (inActiveDocument()) 2788 if (inActiveDocument())
2789 document().ensureStyleResolver().ensureUpdatedRuleFeatureSet().sched uleStyleInvalidationForAttributeChange(name, *this); 2789 document().ensureStyleResolver().ensureUpdatedRuleFeatureSet().sched uleStyleInvalidationForAttributeChange(name, *this);
2790 2790
2791 if (isUpgradedCustomElement()) 2791 if (isUpgradedCustomElement())
2792 CustomElement::attributeDidChange(this, name.localName(), oldValue, newValue); 2792 CustomElement::attributeDidChange(this, name.localName(), oldValue, newValue);
2793 } 2793 }
2794 2794
2795 if (OwnPtr<MutationObserverInterestGroup> recipients = MutationObserverInter estGroup::createForAttributesMutation(*this, name)) 2795 if (OwnPtrWillBeRawPtr<MutationObserverInterestGroup> recipients = MutationO bserverInterestGroup::createForAttributesMutation(*this, name))
2796 recipients->enqueueMutationRecord(MutationRecord::createAttributes(this, name, oldValue)); 2796 recipients->enqueueMutationRecord(MutationRecord::createAttributes(this, name, oldValue));
2797 2797
2798 InspectorInstrumentation::willModifyDOMAttr(this, oldValue, newValue); 2798 InspectorInstrumentation::willModifyDOMAttr(this, oldValue, newValue);
2799 } 2799 }
2800 2800
2801 void Element::didAddAttribute(const QualifiedName& name, const AtomicString& val ue) 2801 void Element::didAddAttribute(const QualifiedName& name, const AtomicString& val ue)
2802 { 2802 {
2803 attributeChanged(name, value); 2803 attributeChanged(name, value);
2804 InspectorInstrumentation::didModifyDOMAttr(this, name.localName(), value); 2804 InspectorInstrumentation::didModifyDOMAttr(this, name.localName(), value);
2805 dispatchSubtreeModifiedEvent(); 2805 dispatchSubtreeModifiedEvent();
(...skipping 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
3273 || isHTMLObjectElement(*this) 3273 || isHTMLObjectElement(*this)
3274 || isHTMLAppletElement(*this) 3274 || isHTMLAppletElement(*this)
3275 || isHTMLCanvasElement(*this)) 3275 || isHTMLCanvasElement(*this))
3276 return false; 3276 return false;
3277 if (FullscreenElementStack::isActiveFullScreenElement(this)) 3277 if (FullscreenElementStack::isActiveFullScreenElement(this))
3278 return false; 3278 return false;
3279 return true; 3279 return true;
3280 } 3280 }
3281 3281
3282 } // namespace WebCore 3282 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698