533401 - Names added through external DOM classinfo don't resolve on global object

Status: RESOLVED FIXED

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

Description

[Peter Van der Beken [:peterv]]

Attachment: v1

This blocks the CoolIris extension from working in 1.9.2.

The check for the non-null mData->mProtoChainInterface in nsDOMClassInfo::PostCreatePrototype fails (because CoolIris doesn't supply a protochain interface), and so we never define the name on the global object. That check originally was in nsDOMClassInfo::PostCreate where it was fine (only need to set up prototype chain if there's a non-nsISupports interface on the prototype chain), but in PostCreatePrototype it looks unneeded. Only the debugging code needs the check, because it uses mProtoChainInterface to check its name. The code in ResolvePrototype, which gets called from PostCreatePrototype, looks like it can deal with null or nsISupports for the mProtoChainInterface.

Comment 1

[Mike Beltzner [:beltzner, not reading bugmail]]

a=beltzner to land on trunk when reviewed

Comment 2

[Shawn Wilsher :sdwilsh]

Is it possible to make an automated test case for this so we don't break this in the future again?

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 416523 [details] [diff] [review]
v1

Looks good.

Comment 4

[Peter Van der Beken [:peterv]]

Yeah, I've been looking at how to test this.

Comment 5

[Peter Van der Beken [:peterv]]

Attachment: Part 2 v1

We need an additional fix so that |new foo()| also works. Already got r=jst in person on this one.

Comment 6

[Johnny Stenback (:jst)]

Now that we know this fixes everything we need to fix to get cooliris back in business, we should block on this for 1.9.2.

Comment 7

[Peter Van der Beken [:peterv]]

http://hg.mozilla.org/mozilla-central/rev/abd2dcd924d9
http://hg.mozilla.org/mozilla-central/rev/c7bbc4ce86fa

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/f3f8b87e11e9
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/0b5a1d59f5f6

Still working on the test.