1430603 - Restrict BROWSER_*_DEFAULT probes to parent-process only

Status: RESOLVED FIXED

(Firefox :: General, defect, P3)

Description

[Chris H-C :chutten]

Several "browser is default" probes are set to be recorded in both "main" and "content" processes, but are only ever recorded in the "main" process.

Please go through and set the "record_in_processes" field within toolkit/components/telemetry/Histograms.json to be ["main"] for the following probes:

BROWSER_IS_ASSIST_DEFAULT
BROWSER_IS_USER_DEFAULT
BROWSER_IS_USER_DEFAULT_ERROR
BROWSER_SET_DEFAULT_ALWAYS_CHECK
BROWSER_SET_DEFAULT_DIALOG_PROMPT_RAWCOUNT
BROWSER_SET_DEFAULT_ERROR
BROWSER_SET_DEFAULT_RESULT

Comment 1

[Swapnil P (:DarthSwap)]

I'll be working on this.

Comment 2

[Swapnil P (:DarthSwap)]

Attachment: Bug1430603.patch

I have changed all the requested probes to ["main"] only.

Comment 3

[Chris H-C :chutten]

Comment on attachment 8942719 [details] [diff] [review]
Bug1430603.patch

Review of attachment 8942719 [details] [diff] [review]:
-----------------------------------------------------------------

Thank you, this looks good. Does it build properly?

Comment 4

[Swapnil P (:DarthSwap)]

I ran the automated tests and they were fine, if that's what you were asking?

Comment 5

[Chris H-C :chutten]

I wanted to make sure that you ran `./mach build` and it completed successfully. This file doesn't get rebuilt in an artefact build, so I have to ask :)

I have marked this bug for inclusion in Firefox. Thank you for your contribution!

Would you like some help finding something new to work on? If so, please let me know what you'd like to work on and maybe I can help you find something.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/41bb3e170bf6
Set all BROWSER_*_DEFAULT probes just to ["main"]. r=chutten.

Comment 7

[Swapnil P (:DarthSwap)]

(In reply to Chris H-C :chutten from comment #5)
> I wanted to make sure that you ran `./mach build` and it completed
> successfully. This file doesn't get rebuilt in an artefact build, so I have
> to ask :)

Oh, my bad. Yes, I just did run ./mach build and it completed.

> 
> I have marked this bug for inclusion in Firefox. Thank you for your
> contribution!
> 
> Would you like some help finding something new to work on? If so, please let
> me know what you'd like to work on and maybe I can help you find something.

I want to work on bugs that are a little bit more difficult than good first contributions. I want to increase my contributions in any way. Any help is much appreciated. Thanks! :)

Comment 8

[Chris H-C :chutten]

I appreciate the enthusiasm! If you'd like to work a little in Python, bug 1426460 is about making it so we fail later with all the parser errors, instead of failing immediately with just the first error. And there's bug 1207948 which involves treating Android a little differently in about:telemetry.

Do either of those seem interesting?

Comment 9

[Swapnil P (:DarthSwap)]

Yes, thank you!

Comment 10

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/41bb3e170bf6