Last Comment Bug 703715 - Add Telemetry probes for sync XHR
: Add Telemetry probes for sync XHR
Product: Core
Classification: Components
Component: DOM (show other bugs)
: unspecified
: x86_64 All
-- normal (vote)
: ---
Assigned To: Olli Pettay [:smaug]
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2011-11-18 12:38 PST by Olli Pettay [:smaug]
Modified: 2011-11-19 11:26 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (2.42 KB, patch)
2011-11-18 13:35 PST, Olli Pettay [:smaug]
jonas: review+
taras.mozilla: review+
Details | Diff | Splinter Review

Description User image Olli Pettay [:smaug] 2011-11-18 12:38:48 PST
Sync XHR in Window context is terrible, and if just possible we should get rid of it.
(it may takes years to achieve that)
Comment 1 User image Olli Pettay [:smaug] 2011-11-18 13:35:34 PST
Created attachment 575543 [details] [diff] [review]
Comment 2 User image Jonas Sicking (:sicking) No longer reading bugmail consistently 2011-11-18 13:40:51 PST
Comment on attachment 575543 [details] [diff] [review]

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.
Comment 3 User image Olli Pettay [:smaug] 2011-11-18 13:44:54 PST
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 User image (dormant account) 2011-11-18 14:11:19 PST
Comment on attachment 575543 [details] [diff] [review]

dont need empty lines in TelemetryHistograms.h

Note You need to log in before you can comment on or make changes to this bug.