Consolidate more MutationObserverRegistration logic in Node
Created attachment 166520 [details] Patch
Comment on attachment 166520 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166520&action=review > Source/WebCore/ChangeLog:12 > + This patch inverts that behavior so that there's no need to return the I like it better without "inverts that behavior". It confused me.
Comment on attachment 166520 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166520&action=review > Source/WebCore/dom/Node.cpp:2484 > Vector<OwnPtr<MutationObserverRegistration> >* registry = ensureRareData()->ensureMutationObserverRegistry(); On a completely unrelated note, we need to invent a variant of HashMap that behaves like Vector when the map is small.
Created attachment 166527 [details] Patch for landing
Comment on attachment 166520 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=166520&action=review >> Source/WebCore/dom/Node.cpp:2484 >> Vector<OwnPtr<MutationObserverRegistration> >* registry = ensureRareData()->ensureMutationObserverRegistry(); > > On a completely unrelated note, we need to invent a variant of HashMap that behaves like Vector when the map is small. Definitely, there are several places recently where I've wanted something like that.
Comment on attachment 166527 [details] Patch for landing Clearing flags on attachment: 166527 Committed r130069: <http://trac.webkit.org/changeset/130069>
All reviewed patches have been landed. Closing bug.