Boris Zbarsky [:bzbarsky] Assignee
	Description • 12 years ago

Dromaeo's dom-attr test uses an <h2>.

You can see a before/after at http://dromaeo.com/?id=186938,186941 (esp. the getAttribute and element.property lines).

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 3 • 12 years ago

Comment on attachment 695425 [details] [diff] [review]
part 1.  Rename nsHTMLHeadingElement to mozilla::dom::HTMLHeadingElement.

Review of attachment 695425 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/html/content/src/nsHTMLHeadingElement.cpp
@@ +9,5 @@
>  #include "nsStyleConsts.h"
>  #include "nsMappedAttributes.h"
>  #include "nsRuleData.h"
>  #include "mozAutoDocUpdate.h"
> +#include "mozilla/dom/HTMLHeadingElement.h"

Include this first, to make sure the header doesn't depend on anything included before it.

::: content/html/content/src/HTMLHeadingElement.h
@@ +36,5 @@
> +
> +  virtual bool ParseAttribute(int32_t aNamespaceID,
> +                                nsIAtom* aAttribute,
> +                                const nsAString& aValue,
> +                                nsAttrValue& aResult);

Indentation

@@ +39,5 @@
> +                                const nsAString& aValue,
> +                                nsAttrValue& aResult);
> +  NS_IMETHOD_(bool) IsAttributeMapped(const nsIAtom* aAttribute) const;
> +  nsMapRuleToAttributesFunc GetAttributeMappingFunction() const;
> +  virtual nsresult Clone(nsINodeInfo *aNodeInfo, nsINode **aResult) const;

* to the left

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 4 • 12 years ago

> Include this first, to make sure the header doesn't depend on anything included before
> it.

Eh.  OK.

Will fix the ParseAttribute indent.  No plans to "fix" the copied code.  ;)

	Peter Van der Beken [:peterv]
	Comment 5 • 12 years ago

Hmm, need to add include guards to HTMLHeadingElement.h.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 6 • 12 years ago

Good catch.  Done.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 7 • 12 years ago

   https://hg.mozilla.org/integration/mozilla-inbound/rev/dc6168662099
   https://hg.mozilla.org/integration/mozilla-inbound/rev/f5638aa67e23

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 9 • 12 years ago

Pushed the test fix as https://hg.mozilla.org/integration/mozilla-inbound/rev/21865911745c pending review.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 10 • 12 years ago

As expected, this gave us wins on dom-attr, ranging from 5% to 17% depending on the build configuration.

	Daniel Holbert [:dholbert]
	Comment 11 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/dc6168662099
https://hg.mozilla.org/mozilla-central/rev/f5638aa67e23
https://hg.mozilla.org/mozilla-central/rev/21865911745c