687662 - Add nsIID xpconnect test coverage

Status: RESOLVED DUPLICATE of bug 693341

(Core :: XPConnect, defect)

Description

[Bobby Holley (:bholley)]

When I define an IDL method with the following signature, it ceases to be visible to script:

nsIID testNSIID(in nsIID a, inout nsIID b);

When I use nsIIDRef, I get crashes. I may just be doing something wrong, and I don't have time to dive into it now. So I'm going to skip nsIID coverage in bug 684327, and use this bug as a reminder to go back and investigate.

Comment 1

[Bobby Holley (:bholley)]

(In reply to Bobby Holley (:bholley) from comment #0)
> When I define an IDL method with the following signature, it ceases to be
> visible to script:
> 
> nsIID testNSIID(in nsIID a, inout nsIID b);

This turned out to be because of the IsMethodReflectable logic on XPCConvert. That stuff is all going away in bug 692342. After that, nsIID, nsIIDRef, and nsIIDPtr will all be handled the same way in XPConnect. This will, unfortunately, mean that nsIID won't do the right thing for scriptable methods, so I've filed bug 695938.

 
> When I use nsIIDRef, I get crashes.

The XPIDL compiler shouldn't have let me do that. I filed bug 695856 for this.

The actual test coverage this bug was going to generate has been rolled into bug 693341. Marking dupe.