*->&
Created attachment 210137 [details] patch
Comment on attachment 210137 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=210137&action=review r=me. Obligatory whining about places where reasons for dereference safety is not immediately clear: > Source/WebCore/dom/ContainerNode.cpp:108 > static inline void attachChild(Node* child) This should take a Node&. > Source/WebCore/dom/ContainerNode.cpp:116 > static inline void detachChild(Node* child) This should take a Node&. > Source/WebCore/dom/Element.cpp:2303 > + Style::resolveTree(*existingPseudoElement, needsStyleRecalc() ? Style::Force : change); This function should take a PseudoElement& instead of a PseudoElement*. > Source/WebCore/style/StyleResolveTree.cpp:472 > + PostAttachCallbackDisabler callbackDisabler(¤t); We should make this RAII object take a ContainerNode& instead.
https://trac.webkit.org/r154903