Closed
Bug 1692886
Opened 3 years ago
Closed 3 years ago
Telemetry::HTTP_TRANSACTION_RESTART_REASON is not recorded properly
Categories
(Core :: Networking: HTTP, defect, P2)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
87 Branch
| Tracking | Status | |
|---|---|---|
| firefox87 | --- | fixed |
People
(Reporter: kershaw, Assigned: kershaw)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
Just found that this probe Telemetry::HTTP_TRANSACTION_RESTART_REASON is not recorded correctly.
The reason is that we use mRestartCount as a condition to record this probe when the transaction is restarted at first attempt. However, mRestartCount is already increased in Restart() so the code in this block is never reached.
|
Assignee | |
Comment 1•3 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d1168bf01fb4 Record HTTP_TRANSACTION_RESTART_REASON correctly r=necko-reviewers,dragana
|
||
Comment 3•3 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox87:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•