1548646 - Preference name for "security.identitypopup.recordEventElemetry" misspelled

Status: RESOLVED FIXED

(Firefox :: Protections UI, defect, P5)

Description

[Jared Wein [:jaws] (please needinfo? me)]

The preference name is "security.identitypopup.recordEventElemetry" but it should be "security.identitypopup.recordEventTelemetry" (missing "T").

Both places where the preference is referenced will need to be updated. This will not affect Telemetry collection, but it would reset the pref for users who have toggled it independently. Therefore, we should continue reading the old spelling of the preference when checking the pref.

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

See https://searchfox.org/mozilla-central/search?q=security.identitypopup.recordEventElemetry&path= for the references.

Comment 2

[Sachin Titus]

Hello, I am willing to work on this issue

Comment 3

[Sachin Titus]

Could you provide more details please? Like What component I have to work on?

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

Thanks for your interest. You can work on this. See the link in comment 1 for what files need changing.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The priority flag is not set for this bug.
:johannh, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 6

[premk]

Attachment: Patch for bug 1548646

Files firefox.js and BrowserGlue.jsm amended. Patch attached.

Comment 7

[premk]

:jaws please review the patch.

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 9069580 [details] [diff] [review]
Patch for bug 1548646

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

::: browser/components/BrowserGlue.jsm
@@ +1450,5 @@
>      }
>    },
>  
>    _recordContentBlockingTelemetry() {
> +    let recordIdentityPopupEvents = Services.prefs.getBoolPref("security.identitypopup.recordEventTelemetry");

As I mentioned in comment 0, "Therefore, we should continue reading the old spelling of the preference when checking the pref."

Comment 9

[premk]

This is what I've understood so far about the bug. Correct me if I'm wrong.

Two references are made to this preference in the codebase (BrowserGlue.jsm and firefox.js)

firefox.js is basically the master of list of all default preferences.
Any preference change made by the user in (about:config) is stored under user preferences (prefs.js)

BrowserGlue.jsm pulls preferences from firefox.js and overrides it with values set in pref.js.

Won't it cause conflict if we update the spelling in one file (firefox.js) but not the other (browserglue.jsm)?

Comment 10

[Jared Wein [:jaws] (please needinfo? me)]

If a user selectively changes a pref in firefox.js, and we remove that preference and replace it with a new name, the user-changed preference will remain. In that event, we can still check the old preference name to see if they have opted in to a specific behavior before checking the new preference name's value.

So the code in BrowserGlue.jsm can do as follows:

let recordIdentityPopupEvents = Services.prefs.prefHasUserValue("security.identitypopup.recordEventElemetry") ?  
                                Services.prefs.getBoolPref("security.identitypopup.recordEventElemetry") :
                                Services.prefs.getBoolPref("security.identitypopup.recordEventTelemetry");

Comment 11

[premk]

Attachment: Revised patch for Bug 1548646

Got it. Revised patch attached.

Comment 12

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 9069731 [details] [diff] [review]
Revised patch for Bug 1548646

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

thanks!

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4f4de4742c98
correct misspelled preference name security.identitypopup.recordEventElemetry r=jaws

Comment 14

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/4f4de4742c98