682420 - Rename nsINode::GetOwnerDoc to nsINode::OwnerDoc

Status: RESOLVED FIXED

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

Description

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

.

Comment 1

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

Attachment: initial patch

...but I think I should remove some unneeded null checks while I'm changing the
code.

Comment 2

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

Attachment: s/GetOwnerDoc()/OwnerDoc()/

This is script generated patch, but I had to fix few whitespace issues manually
when the change was done inside a macro.

Another patch coming which will remove some null checks.

Comment 3

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

Attachment: Remove useless null checks

Uploaded to try.

There can be still some more useless null checks, but this should cover
most common cases.

Comment 4

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

Attachment: -w

Comment 5

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

Attachment: s/GetOwnerDoc()/OwnerDoc()/, up-to-date

s/PR_TRUE/true/ patch caused quite a few merging problems.

Comment 6

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

Attachment: Remove useless null checks, up-to-date

Comment 7

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

https://hg.mozilla.org/mozilla-central/rev/9e6aa5ee6425
https://hg.mozilla.org/mozilla-central/rev/41a01f78db7a

Comment 8

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

Attachment: backout for 10, part 1

Just in case we want to backout strong parent node from 10...

Comment 9

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

Attachment: backout for 10, part 2

Comment 10

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

Attachment: backout strongparent node from 10

This patch is really not for this bug, but want to back it up somewhere.
And it is needed if we want to have non-strong-parent in 10.