Open Bug 1392272 Opened 5 years ago Updated 7 days ago

[Win] Optimize the way we resolve proxy settings

Categories

(Core :: Networking: HTTP, enhancement, P3)

enhancement

Tracking

()

ASSIGNED
Tracking Status
firefox96 --- fixed

People

(Reporter: mayhemer, Assigned: kershaw)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open, perf, Whiteboard: [necko-next][proxy])

Attachments

(2 files)

With the default prefs we do async proxy resolution for every channel.  Most users these days have Windows set to "no proxy" in the Windows internet settings.  We should detect changes by observing for either registry changes or doing some other magic and bypass proxy resolution completely when no proxy has been specified.

I can see 20+ ms (on a fast machine, in an opt build!) on EVERY channel.  This also includes main thread dispatch.

Not sure we could have this for 57, but would definitely be nice.

Loosely blocking CDP.

https://stackoverflow.com/questions/6192563/detect-windows-ie-proxy-settings-changes
One simple solution (some probability of regression prove..) could be to refresh the registry settings only when loading a top level document and not for sub-resources.
Keywords: perf
Whiteboard: [necko-next]
Whiteboard: [necko-next] → [necko-next][proxy]
Assign to myself, but maybe fix later, so keep necko-next.
Assignee: nobody → xeonchen
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P3 → P2
Assignee: xeonchen → nobody
Priority: P2 → P3
Assignee: nobody → kershaw
Status: NEW → ASSIGNED
Duplicate of this bug: 1652083

Add leave-open, since linux and android parts are not completed.

Keywords: leave-open
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1c1c60a1f6ef
P1: [windows] Monitor system proxy changes, r=necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/931db25d14fd
P2: [osx] Monitor system proxy changes, r=necko-reviewers,dragana
Regressions: 1749501
No longer blocks: 1652083
Severity: normal → --
You need to log in before you can comment on or make changes to this bug.