The default bug view has changed. See this FAQ.

Remove nsAXPCNativeCallContext::{GetRetValPtr,GetReturnValueWasSet,SetReturnValueWasSet}

RESOLVED FIXED in mozilla11

Status

()

Core
XPConnect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

({meta})

Trunk
mozilla11
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
We can now return jsvals through IDL rather than using these APIs.
(Assignee)

Comment 1

5 years ago
(As requested in bug 705186 comment 1.)
(Assignee)

Updated

5 years ago
Depends on: 705333
(Assignee)

Updated

5 years ago
Depends on: 705344
(Assignee)

Updated

5 years ago
Depends on: 705355
(Assignee)

Updated

5 years ago
Depends on: 705357
(Assignee)

Updated

5 years ago
Depends on: 705444
(Assignee)

Comment 2

5 years ago
Created attachment 579449 [details] [diff] [review]
Remove nsAXPCNativeCallContext::{GetRetValPtr,GetReturnValueWasSet,SetReturnValueWasSet}; s
Attachment #579449 - Flags: review?(bobbyholley+bmo)
We also need to remove support for this in XPCWrappedNative.cpp, no?
(Assignee)

Comment 4

5 years ago
Created attachment 579468 [details] [diff] [review]
Patch v2

Well... Yes.
Attachment #579449 - Attachment is obsolete: true
Attachment #579449 - Flags: review?(bobbyholley+bmo)
Attachment #579468 - Flags: review?(bobbyholley+bmo)
(Assignee)

Comment 5

5 years ago
https://tbpl.mozilla.org/?tree=Try&rev=92daa6355bef
Comment on attachment 579468 [details] [diff] [review]
Patch v2

r=bholley. Also, sr=bz from IRC.

Thanks for doing this. :-)
Attachment #579468 - Flags: review?(bobbyholley+bmo) → review+
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/dddd8c46f9a9

MDC doesn't know about nsAXPCNativeCallContext, and I claim that's a good thing, so no dev-doc-needed.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.