305528 - Cannot setTimeout window.focus or window.alert

Status: VERIFIED FIXED

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

Description

[Boris Zbarsky [:bzbarsky]]

This is fallout from splitwindow, based on the regression date range (which is
somewhere in
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2005-07-30+07%3A00%3A00&maxdate=2005-07-31+12%3A00%3A00&cvsroot=%2Fcvsroot
).  To reproduce, load the URL in the URL bar and note that the JS console shows
an error instead of an alert saying "success" coming up.  The error is:

Error: uncaught exception: [Exception... "Illegal operation on WrappedNative
prototype object"  nsresult: "0x8057000c (NS_ERROR_XPC_BAD_OP_ON_WN_PROTO)" 
location: "native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0"  data: no]

Comment 1

[Boris Zbarsky [:bzbarsky]]

Requesting blocking so that we at least evaluate what's going on here...

Comment 2

[Benjamin Smedberg]

Just for kicks, the same thing happens if you're using script directly instead
of a javascript: URI, right?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Yes.

Comment 4

[Johnny Stenback (:jst)]

Attachment: Make xpconnect attempt to use obj's outer if it can't find a wrapper using the given obj

This makes XPCWrappedNative::GetWrappedNativeOfJSObject() use obj's outer if it
can't find a wrapper with the given obj, which in this case is an inner window,
but the function we're calling comes from the outer window.

Comment 5

[Brendan Eich [:brendan]]

Comment on attachment 193751 [details] [diff] [review]
Make xpconnect attempt to use obj's outer if it can't find a wrapper using the given obj

Pre-emptive a=me as well as sr=me.  This is it.

/be

Comment 6

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Comment on attachment 193751 [details] [diff] [review]
Make xpconnect attempt to use obj's outer if it can't find a wrapper using the given obj

r=shaver

Comment 7

[Johnny Stenback (:jst)]

Fixed on trunk and branch.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED using build 2005-09-03-05 on Windows XP SeaMonkey trunk.