Lukewarm (0.4%) on Dromaeo/dom-attr.
Created attachment 143463 [details] A patchwork orange
Comment on attachment 143463 [details] A patchwork orange View in context: https://bugs.webkit.org/attachment.cgi?id=143463&action=review > Source/WebCore/dom/Element.cpp:2087 > + if (!hasAttrList()) > + return 0; > if (!attributeData()) > return 0; Should we combine these if statements?
Comment on attachment 143463 [details] A patchwork orange View in context: https://bugs.webkit.org/attachment.cgi?id=143463&action=review >> Source/WebCore/dom/Element.cpp:2087 >> return 0; > > Should we combine these if statements? Per IRC discussion, please convert the latter condition to an assertion.
Committed r118127: <http://trac.webkit.org/changeset/118127>