(In reply to comment #95)
> Gavin, can you load the POST result, then breakpoing in CanSavePresention and
> navigate away from it? Does it get bfcached? If not, which exact reason
> causes CanSavePresentation to return false?
With or without the patch applied (i.e. whether or not the dialog appears), the CanSavePresentation called with nsLocation::SetHref on the stack (i.e. the one that navigates away from the POST in the testcase) returns false due to the subframe check:
> Is doing loads directly from the onload handler relevant?
No - I can reproduce the problem by driving the test manually (e.g. submitting the iframe's form with a javascript: URI, and then just going back to the form page, and forward to the POST result manually with the back/forward buttons).
(In reply to comment #95)
> Gavin, can you load the POST result, then breakpoing in CanSavePresention and
> navigate away from it? Does it get bfcached? If not, which exact reason
> causes CanSavePresentation to return false?
With or without the patch applied (i.e. whether or not the dialog appears), the CanSavePresentation called with nsLocation::SetHref on the stack (i.e. the one that navigates away from the POST in the testcase) returns false due to the subframe check:
http:// bonsai. mozilla. org/cvsblame. cgi?file= mozilla/ docshell/ base/nsDocShell .cpp&rev= 1.894&mark= 5292#5261
> Is doing loads directly from the onload handler relevant?
No - I can reproduce the problem by driving the test manually (e.g. submitting the iframe's form with a javascript: URI, and then just going back to the form page, and forward to the POST result manually with the back/forward buttons).