Summary: | Use Element& in StyleResolveTree | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antti Koivisto <koivisto> | ||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Antti Koivisto
2013-08-30 10:59:03 PDT
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. |