1220822 - Rewrite the nsINode::OwnerDoc comment to not contradict itself and relate it to HTML5 terms

Status: RESOLVED FIXED

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

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

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

Comment on attachment 8682143 [details] [diff] [review]
patch

>+   * For all other cases (this node is nsIContent, not nsIDocument) GetOwnerDoc
This is a bit oddly said, since not all other cases are nsIContent objects.
(we have still nsIAttribute)

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: patch

Removes parenthetical.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/696ea2cdc032

Comment 5

[Mats Palmgren (inactive)]

+   * For all other cases GetOwnerDoc and GetOwnerDocument behave identically.

What is that GetOwnerDoc you speak of above?  Shouldn't it say:

For all other cases OwnerDoc and ...

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/696ea2cdc032

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/696ea2cdc032

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5bcb60614d01

Comment 9

[Jonathan Watt [:jwatt]]

(In reply to Mats Palmgren (:mats) from comment #5)
> +   * For all other cases GetOwnerDoc and GetOwnerDocument behave
> identically.
> 
> What is that GetOwnerDoc you speak of above?  Shouldn't it say:
> 
> For all other cases OwnerDoc and ...

Thanks for noticing that! I took the liberty of using you as the reviewer. ;)

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5bcb60614d01