1566763 - Review DEVTOOLS_ABOUTDEBUGGING_OPENED_COUNT after removing old aboutdebugging

Status: RESOLVED FIXED

(DevTools :: about:debugging, task, P3)

Description

[Julian Descottes [:jdescottes]]

The old aboutdebugging used to record the opened_counts in the following histogram: DEVTOOLS_ABOUTDEBUGGING_OPENED_COUNT

This was recorded via the following code:
https://searchfox.org/mozilla-central/rev/22b330ecb3edba1536a54887060cbdd09db21c59/devtools/client/aboutdebugging/components/Aboutdebugging.js#89

  componentDidMount() {
    window.addEventListener("hashchange", this.onHashChange);

    // aboutdebugging is not connected with a toolbox so we pass -1 as the
    // toolbox session id.
    this.props.telemetry.toolOpened("aboutdebugging", -1, this);
  }

The new aboutdebugging however only uses events for telemetry. The most similar event here would be dt_adbg - open_adbg.

We are removing the old aboutdebugging in https://bugzilla.mozilla.org/show_bug.cgi?id=1539461 so we should decide either to remove the probe or to call toolOpened for the new aboutdebugging (maybe we want to keep the histogram to easily compare with other devtools UIs which rely on toolOpened?)

Comment 1

[Julian Descottes [:jdescottes]]

Hi Harald

More telemetry questions! :)
This histogram used to record old aboutdebugging open counts. Do we want to keep it and use it from the new aboutdebugging, or do we feel like the events are enough now?

Comment 2

[:Harald Kirschner :digitarald]

It would be interesting to see if we regress, and making sure we validate the event telemetry numbers with the histogram as comparison – at least until we have confidence. Does that make sense or am I missing something?

What's the work in collecting the same telemetry for the new about:debugging?

Comment 3

[Manish Kumar]

It is interesting

Comment 4

[Julian Descottes [:jdescottes]]

(In reply to :Harald Kirschner :digitarald from comment #2)

It would be interesting to see if we regress, and making sure we validate the event telemetry numbers with the histogram as comparison – at least until we have confidence. Does that make sense or am I missing something?

That sounds good, I also think it will be easier to compare.

What's the work in collecting the same telemetry for the new about:debugging?

Should be pretty easy, we just have to call telemetry.toolOpened("aboutdebugging", -1, this); when opening the new about:debugging.

Comment 5

[Julian Descottes [:jdescottes]]

Attachment: Bug 1566763 - Call telemetry.toolOpened/Closed from new aboutdebugging

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/92064bff2df5
Call telemetry.toolOpened/Closed from new aboutdebugging r=remote-debugging-reviewers,Ola

Comment 7

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/92064bff2df5