337205 - Move docshell changes for session storage into a new interface

Status: RESOLVED FIXED

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

Description

[Neil Deakin]

Comment 1

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

bryner is concerned about shipping a variant nsIDocShell in alpha2, because he has a C++-based extension that he wants to have work in both 1.5 and 2.0a2.  Given that there are different IIDs for the two nsIDocShells, I think that's feasible, and a relatively rare case.

So in my opinion the current path is preferable to the other available choices (back out storage and not get impl feedback from interested web authors until b1; or slip the alpha for this bug to get fixed).  But others' opinions might differ!

Comment 2

[Johnny Stenback (:jst)]

Attachment: Revert the changes to nsIDocShell and put them in a new interface

This reverts all the nsIDocShell changes in bug 335540 and splits them out into a nsIDocShell_MOZILLA_1_8_BRANCH interface. Pretty straight forward stuff.

Comment 3

[Darin Fisher]

This should block the release of "bon echo alpha 2" in my opinion.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 221393 [details] [diff] [review]
Revert the changes to nsIDocShell and put them in a new interface

r+sr+a=bzbarsky

Comment 5

[Mike Schroepfer]

You should land this once the tree bustage is fixed.

Comment 6

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

jst landed this last night, FIXED.