Comment 212 for bug 113201

Revision history for this message
In , Christian Biesinger (cbiesinger) wrote :

(In reply to comment #138)
> Part of the merge involved a new call to SetupReplacementChannel() in
> ProcessFallback(). I pass in false as 'forProxy', but this is just a guess as
> to the desired behavior. Does it seem correct?

yeah that seems right.

are you going to add a test in netwerk/test/unit? you'd just have to set the PAC pref using the prefservice, can be a data: URI that just always returns DIRECT I think. and then check that stuff gets correctly transferred.