Closed
Bug 1857244
Opened 2 years ago
Closed 2 years ago
Netmonitor offline mode is only reset when setting `No throttling`
Categories
(DevTools :: Netmonitor, defect, P2)
DevTools
Netmonitor
Tracking
(firefox121 verified, firefox122 verified)
VERIFIED
FIXED
121 Branch
People
(Reporter: valentin, Assigned: bomsy)
References
(Blocks 1 open bug, Regressed 1 open bug)
Details
Attachments
(1 file)
- Open tab
- set offline throttling mode in the netmonitor
- open http://example.com (Should display OFFLINE error page)
- Set GPRS throttling profile in netmonitor
- Refresh the page (Page should load, but instead still displays offline error page)
Comment 1•2 years ago
|
||
Thanks for the STRs, bug for a new feature should be fixed rapidly.
Severity: -- → S3
Priority: -- → P2
| Assignee | ||
Comment 2•2 years ago
|
||
Updated•2 years ago
|
Assignee: nobody → hmanilla
Status: NEW → ASSIGNED
Comment 3•2 years ago
|
||
There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:bomsy, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.
Flags: needinfo?(jdescottes)
Flags: needinfo?(hmanilla)
| Assignee | ||
Comment 4•2 years ago
|
||
We need to land this fix with a test (which is in Bug 1855129). I'll get back to finishing the test in a few.
Flags: needinfo?(hmanilla)
Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/78c127f4a596
[devtools] Reset netmonitor offline when throttling mode is changed r=devtools-reviewers,jdescottes
Comment 6•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox121:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Updated•2 years ago
|
QA Whiteboard: [qa-121b-p2]
Comment 7•2 years ago
|
||
Reproduced this issue on an affected nightly build from 2023-10-05 on Win 10.
Verified as fixed on Firefox 121.0b4 (20231127091758) and 122.0a1 (20231129050021) on Ubuntu 22, macOS 11 and Windows 10.
Updated•1 year ago
|
Flags: needinfo?(jdescottes)
You need to log in
before you can comment on or make changes to this bug.
Description
•