The default bug view has changed. See this FAQ.

Make nsIDocument::AddXMLEventsContent return void

VERIFIED FIXED in mozilla7

Status

()

Core
DOM
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 542776 [details] [diff] [review]
Patch v1

None of its callers check the return value, which can only be NS_OK because of infallible new.
Flags: in-testsuite-
Attachment #542776 - Flags: review?(Olli.Pettay)
Comment on attachment 542776 [details] [diff] [review]
Patch v1

You need to update the IID of nsIDocument
Attachment #542776 - Flags: review?(Olli.Pettay) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/mozilla-central/rev/4caf28f35715
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7

Comment 3

6 years ago
From the code changes made to:
content/base/public/nsIDocument.h
content/base/src/nsDocument.cpp
content/base/src/nsDocument.h

it is obvious that AddXMLEventsContent now returns void instead of nsresult.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.