They're really not terribly similar, so having them be the same class doesn't make much sense. It leads to a ton of code that tests IsInnerWindow() to see if we need to forward or not. We'll still need the forwarding code, but we can avoid the if-statement.

who's going to take this?

I was hoping to get to this this quarter.

Andreas had discussed (IIRC with mrbkap) using harmony:proxies here. Possible? /be

Seems better to me to do that in a followup step. This is more aimed at using a different C++ class than at using a different JS class. One thing at a time...

We use proxies in JS now. We need to split the C++ classes so things are less implicitly unsafe for C++ users.

The plan here is roughly: 1. Move all methods off of nsIDOMWindow (bug 1216401) 2. Make it possible to specify in XPIDL whether a parameter is an inner or outer window. 3. Make an nsPIDOMWindowInner/Outer, and make every place in the code base use one or the other. 4. Move methods specific to inner/outer to the appropriate nsPIDOMWindowInner/Outer. 5. Split nsGlobalWindow.

(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #6) > The plan here is roughly: > > 1. Move all methods off of nsIDOMWindow (bug 1216401) > 2. Make it possible to specify in XPIDL whether a parameter is an inner or > outer window. > 3. Make an nsPIDOMWindowInner/Outer, and make every place in the code base > use one or the other. > 4. Move methods specific to inner/outer to the appropriate > nsPIDOMWindowInner/Outer. > 5. Split nsGlobalWindow. Steps 1-3 have already happened. Instead of continuing with steps 4 and 5, I have a new plan. JS security is now handled entirely by the JS engine and the wrapper system. We never create a wrapper for the outer window or expose it to JS. So instead of splitting nsGlobalWindow, we should instead move all of the methods/variables from the outer window to the docshell. Then we can kill the concept of inner and outer windows entirely, and nsGlobalWindow will be just the inner window. WindowProxy will exist solely as a JS abstraction, and where C++ needs to manipulate windows across navigations it will hold onto and use the docshell.

I'm going to be trying to push this through in Q4, so assigning this to myself.

This was fixed 2 months ago :-)