Add logs for better overall network request/context tracking

RESOLVED FIXED in Firefox 57

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mayhemer, Assigned: mayhemer)

Tracking

56 Branch
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [necko-active])

Attachments

(1 attachment, 1 obsolete attachment)

No description provided.
Posted patch backup1 (obsolete) — Splinter Review
Whiteboard: [necko-active]
This is needed to track request context, page loads and channels in logan [1] for network diagnostics/breakdown within a page load.

Regarding the nsDocument logs, there are few more than just ctor and dtor already and those are there since stone age.  I don't assume there would be any analytic tool that could break ;)  I believe the same applies to PresShell logs.

[1] https://janbambas.cz/moz/logan/
Attachment #8895002 - Attachment is obsolete: true
Attachment #8898426 - Flags: review?(jduell.mcbugs)
Comment on attachment 8898426 [details] [diff] [review]
v1 (just adds few logs, simple)

No reaction from Jason...   Dragana, this is a simple patch, nothing more than added LOGs.  Thanks.
Attachment #8898426 - Flags: review?(jduell.mcbugs) → review?(dd.mozilla)
Attachment #8898426 - Flags: review?(dd.mozilla) → review+
Thanks.
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/464568af8eb1
Logs for better network requests and context tracking. r=mcmanus
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/464568af8eb1
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.