(In reply to comment #126)
> Hmm. So this patch as it stands doesn't carry over "custom" headers set by
> XMLHttpRequest? That seems like it could be a problem, no?
>
Correct. The patch is not meant to be applied. However, SetupReplacementChannel already has the problem. The patch from shaver/gavin just makes the problem worse because we are calling SetupReplacementChannel more often.
(In reply to comment #126)
> Hmm. So this patch as it stands doesn't carry over "custom" headers set by
> XMLHttpRequest? That seems like it could be a problem, no?
>
Correct. The patch is not meant to be applied. However, SetupReplacemen tChannel already has the problem. The patch from shaver/gavin just makes the problem worse because we are calling SetupReplacemen tChannel more often.