1868553 - [Experiment] Users cannot enroll in the “Ask User to Set Default PDF Handler” experiment due to unmatched targeting

Status: VERIFIED FIXED

(Firefox :: Messaging System, defect)

Description

[Alice Varro [avarro], Ecosystem QA]

[Notes]:

• According to the targeting of the Experimenter ticket, the experiment should target users having a default PDF handler other than the current Firefox installation.

[Affected versions]:

• Firefox Release 120.0.1 - Build ID: 20231129155202
• Firefox Beta 121.0b7 - Build ID: 20231204092159 (Release channel)

[Affected Platforms]:

• Windows 10x64
• Windows 11 x64.

[Prerequisites]:

• Have a new Firefox profile created and opened.
• Have this “user.js” file saved on your computer.
• Have a different browser than Firefox set as the default PDF handler.

[Steps to reproduce]:

1. Navigate to the “about:support” page and paste the “user.js” file into the User Profile.
2. Restart the browser (Ctrl/Cmd + Shift + J and Ctrl/Cmd + Option + R).
3. Navigate to the “about:studies” page.
4. Observe the displayed experiments in the “Active Studies” section.

[Expected result]:

• The users are successfully enrolled in the “Ask User to Set Default PDF Handler” experiment and the experiment is displayed as active in the “about:studies” page.

[Actual result]:

• Users cannot enroll in the “Ask User to Set Default PDF Handler” experiment due to unmatched targeting

[Additional Notes]:

• This issue is reproducible on both branches of the experiment.
• Attaching a screen recording of the issue here.

Comment 1

[[e-mail]]

Here's the JEXL from the experiment:

(browserSettings.update.channel == "release") && ('pdf' in isDefaultHandler && !isDefaultHandler['pdf'] && 'knownBrowser' in defaultPDFHandler && defaultPDFHandler['knownBrowser']) && (version|versionCompare('120.!') >= 0)

This is evaluating to false at about:newtab#devtools-targeting on Firefox 120 release channel. It should not be.

If I remove the two "in" statements, it evaluates to true:

(browserSettings.update.channel == "release") && (!isDefaultHandler['pdf'] && defaultPDFHandler['knownBrowser']) && (version|versionCompare('120.!') >= 0)

Not sure what's going on there...

Comment 2

[Robin Steuber (she/her) [:bytesized]]

https://github.com/mozilla/experimenter/pull/9908

Comment 3

[Robin Steuber (she/her) [:bytesized]]

PR is closed and changes have propagated to https://experimenter.services.mozilla.com/nimbus/set-default-pdf-handler-rollout/summary

Comment 4

[Alice Varro [avarro], Ecosystem QA]

I've verified this issue using Firefox Release 120.0.1 (build ID: 20231129155202) on Windows 10 x64 and I got successfully enrolled in the experiment. I'll close the issue as Verified.
Thank you!