Closed Bug 1237775 Opened 4 years ago Closed 4 years ago
Turn off network change monitoring on Linux
Because of some problems with network change events on Linux, we will turn it off on beta and aurora. It is only a pref change. On nightly it will left on and will be fixed soon.
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
Comment on attachment 8705395 [details] [diff] [review] bug_1237775.patch Review of attachment 8705395 [details] [diff] [review]: ----------------------------------------------------------------- ::: modules/libpref/init/all.js @@ +1196,5 @@ > pref("network.allow-experiments", true); > > // Allow the network changed event to get sent when a network topology or > // setup change is noticed while running. > +#if defined(XP_UNIX) trailing whitespace nit
Attachment #8705395 - Flags: review?(mcmanus) → review+
Comment on attachment 8705704 [details] [diff] [review] bug_1237775.patch Approval Request Comment [Feature/regressing bug #]: Bug 1191253 [User impact if declined]: This is only Linux bug. For some users ipv6 addresses are refresh often (even every 3-4 seconds) and each refresh firefox wrongly detects as network change event which causes the idle connections to be closed. So if a server takes some time to respond connection will be closed. Bugs: 1234548, 1237583, 1235509, 1238062, 1234751. [Describe test coverage new/current, TreeHerder]: All reporters reported that flipping the pref solves the problem: https://bugzilla.mozilla.org/show_bug.cgi?id=1237583#c2 https://bugzilla.mozilla.org/show_bug.cgi?id=1238062#c3 https://bugzilla.mozilla.org/show_bug.cgi?id=1234548#c13 https://bugzilla.mozilla.org/show_bug.cgi?id=1234548#c18 https://bugzilla.mozilla.org/show_bug.cgi?id=1234548#c20 [Risks and why]: This is only pref change (network.notify.changed set by default to false on linux only) and with this pref change the behaviour will be the same as no ff42. We will fix this problem on the current nightly (ff46) therefore we are not flipping the pref on nightly. [String/UUID change made/needed]: none
I think we want to document that.
Comment on attachment 8705704 [details] [diff] [review] bug_1237775.patch Sure, let's disable it.
Attachment #8705704 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 8705704 [details] [diff] [review] bug_1237775.patch Based on the stricter uplift criteria for Beta44, this fits into major regressions bucket. Based on the uplift request this bug seems like something introduced in 43 and we should be trying to fix in 44, at least by turning the pref off. Beta44+
Attachment #8705704 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.