811663 - Create a DocumentFragment header

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Boris Zbarsky [:bzbarsky]]

Actually, nevermind.  This isn't doable because we use it in nsGenericHTMLElement.

As long as I'm here, I'll just morph the bug.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Rename nsDocumentFragment to mozilla::dom::DocumentFragment.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Create a DocumentFragment.h header.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 681410 [details] [diff] [review]
part 1.  Rename nsDocumentFragment to mozilla::dom::DocumentFragment.


>+  DocumentFragment *it = new DocumentFragment(nodeInfo.forget());
DocumentFragmeent* it =..
>   if (!it) {
>     return NS_ERROR_OUT_OF_MEMORY;
>   }
And while you're here, you could remove OOM check.


>-DOMCI_NODE_DATA(DocumentFragment, nsDocumentFragment)
You had to move this to elsewhere because of namespaces, right?

Comment 5

[Boris Zbarsky [:bzbarsky]]

> And while you're here, you could remove OOM check.

Done.

> You had to move this to elsewhere because of namespaces, right?

Correct.

Comment 6

[Boris Zbarsky [:bzbarsky]]

   https://hg.mozilla.org/integration/mozilla-inbound/rev/6fd28026e072
   https://hg.mozilla.org/integration/mozilla-inbound/rev/7c24b8d92e1c

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6fd28026e072
https://hg.mozilla.org/mozilla-central/rev/7c24b8d92e1c