1789468 - Add telemetry probe to measure from AsyncOpen to the transaction being added to the HttpConnectionManager

Status: RESOLVED FIXED

(Core :: Networking, task, P2)

Description

[Andrew Creskey [:acreskey]]

When we create a new network request, the timing from HttpChanneChild::AsyncOpen in the content process to the point where we add the transaction to the HttpConnectionManager is all internal to Gecko.
This is notably before the issuance of the DNS request, which is captured here: http_sub_dns_issue_time.

We should measure this to see if it can be improved.

Comment 1

[Andrew Creskey [:acreskey]]

Attachment: Bug 1789468 - Add telemetry probe to measure from AsyncOpen to the transaction being added to the HttpConnectionManager r=necko-reviewers

Comment 2

[Andrew Creskey [:acreskey]]

Attachment: async_open_to_transaction_pending_data_review.txt

Comment 3

[Chris H-C :chutten]

Comment on attachment 9294484 [details]
async_open_to_transaction_pending_data_review.txt

DATA COLLECTION REVIEW RESPONSE:

Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes.

Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

If the request is for permanent data collection, is there someone who will monitor the data over time?

[e-mail], [e-mail] will be responsible for the permanent collections.

Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1, Technical.

Is the data collection request for default-on or default-off?

Default on for all channels.

Does the instrumentation include the addition of any new identifiers?

No.

Is the data collection covered by the existing Firefox privacy notice?

Yes.

Does the data collection use a third-party collection tool?

No.

---

Result: datareview+

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d8b002b9c8ba
Add telemetry probe to measure from AsyncOpen to the transaction being added to the HttpConnectionManager r=necko-reviewers,kershaw

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/d8b002b9c8ba