Rename nsINode::GetOwnerDoc to nsINode::OwnerDoc

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
8 years ago
5 months ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(8 attachments, 1 obsolete attachment)

.
Assignee: nobody → Olli.Pettay
Posted patch initial patch (obsolete) — Splinter Review
...but I think I should remove some unneeded null checks while I'm changing the
code.
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.
Attachment #567366 - Attachment is obsolete: true
Attachment #567405 - Flags: review?
Attachment #567405 - Flags: review? → review?(jst)
Uploaded to try.

There can be still some more useless null checks, but this should cover
most common cases.
Attachment #567429 - Flags: review?(jst)
Posted patch -wSplinter Review
Attachment #567405 - Flags: review?(jst) → review+
Attachment #567429 - Flags: review?(jst) → review+
s/PR_TRUE/true/ patch caused quite a few merging problems.
https://hg.mozilla.org/mozilla-central/rev/9e6aa5ee6425
https://hg.mozilla.org/mozilla-central/rev/41a01f78db7a
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Just in case we want to backout strong parent 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.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.