Closed
Bug 703715
Opened 14 years ago
Closed 14 years ago
Add Telemetry probes for sync XHR
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: smaug, Assigned: smaug)
Details
Attachments
(1 file)
2.42 KB,
patch
|
sicking
:
review+
taras.mozilla
:
review+
|
Details | Diff | Splinter Review |
Sync XHR in Window context is terrible, and if just possible we should get rid of it.
(it may takes years to achieve that)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → bugs
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #575543 -
Flags: review?(tglek)
Attachment #575543 -
Flags: review?(jonas)
Comment on attachment 575543 [details] [diff] [review]
patch
Assuming this will let us see how man async as well as sync requests are made.
It'd be awesome if we could get a count on how many sites out there use sync XHR, but I don't see a way of doing that without effectively sending the domain name of the site to us, which would be terrible privacy.
Attachment #575543 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 3•14 years ago
|
||
This counts all the open() calls, and either increases the async bar (0), or sync bar(1), so yes
it is easy to check how many XHR open calls there has been, and how many of those have been sync.
Comment 4•14 years ago
|
||
Comment on attachment 575543 [details] [diff] [review]
patch
dont need empty lines in TelemetryHistograms.h
Attachment #575543 -
Flags: review?(tglek) → review+
Assignee | ||
Comment 5•14 years ago
|
||
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•