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.
It does help. Not calling addDocumentRedi rect() when the redirect is internal fixes the test case.
So now, to fix it properly:
1. AddDocumentRedi rect() doesn't check for internal redirects. It seems reasonable that it should ignore internal redirects.
2. nsDocShell: :OnRedirectStat eChange( ) also calls AddDocumentRedi rect() and gavin was wondering whether nsDocShell: :OnRedirectStat eChange( ) should be called on internal redirects.