>
> (In reply to Patrick McManus [:mcmanus] from comment #41)
> > ::: browser/base/content/test/browser_bug435325.js
> > @@ +40,5 @@
> > > finish();
> > > }
> > >
> > > registerCleanupFunction(function() {
> > > + Services.prefs.setIntPref("network.proxy.type", 2);
> >
> > you need at least push/pop. Please comment that this is necessary because
> > the mochitest proxy runs n localhost and localhost is now reachable even
> > when offline.
>
> Added the comments, but wasn't clear if the "push/pop" referred to something
> else?
by push/pop I mean you need to save the original state of the pref and restore that rather than setting it explicitly to "2" when we're done, in case that's not what is used in the future.
> base/content/ test/browser_ bug435325. js Function( function( ) { prefs.setIntPre f("network. proxy.type" , 2);
> (In reply to Patrick McManus [:mcmanus] from comment #41)
> > ::: browser/
> > @@ +40,5 @@
> > > finish();
> > > }
> > >
> > > registerCleanup
> > > + Services.
> >
> > you need at least push/pop. Please comment that this is necessary because
> > the mochitest proxy runs n localhost and localhost is now reachable even
> > when offline.
>
> Added the comments, but wasn't clear if the "push/pop" referred to something
> else?
by push/pop I mean you need to save the original state of the pref and restore that rather than setting it explicitly to "2" when we're done, in case that's not what is used in the future.