Comment 253 for bug 113201

Revision history for this message
In , Jmuizelaar (jmuizelaar) wrote :

It does help. Not calling addDocumentRedirect() when the redirect is internal fixes the test case.

So now, to fix it properly:

1. AddDocumentRedirect() doesn't check for internal redirects. It seems reasonable that it should ignore internal redirects.

2. nsDocShell::OnRedirectStateChange() also calls AddDocumentRedirect() and gavin was wondering whether nsDocShell::OnRedirectStateChange() should be called on internal redirects.