Closed Bug 888576 Opened 7 years ago Closed 7 years ago

Follow the spec for document.body

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Details

(Keywords: dev-doc-needed)

Attachments

(1 file)

Attached patch Patch v1Splinter Review
No description provided.
Attachment #769333 - Flags: review?(mounir)
Comment on attachment 769333 [details] [diff] [review]
Patch v1

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

r=me assuming that other vendors implement document.body the same way and we are not getting ourselves incompatible by aligning with the spec.
Attachment #769333 - Flags: review?(mounir) → review+
Other vendors don't match the spec either, but they do so in different ways.
Comment on attachment 769333 [details] [diff] [review]
Patch v1

Lets get Olli do a second check at this.
Attachment #769333 - Flags: superreview?(bugs)
Comment on attachment 769333 [details] [diff] [review]
Patch v1

Well, I do prefer this behavior. And good to land this kind of change relatively
early in a cycle.
Attachment #769333 - Flags: superreview?(bugs) → superreview+
https://hg.mozilla.org/mozilla-central/rev/52058abdb6e9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.