Update AboutWelcome default screens based on pin prompt segmentation and trigger auto pinning when OS-level consent prompt will show
Categories
(Firefox :: Messaging System, enhancement, P1)
Tracking
()
People
(Reporter: mviar, Assigned: mviar)
References
(Blocks 1 open bug)
Details
(Keywords: perf-alert)
Attachments
(1 file)
Update AboutWelcome defaults to not show the pin option on Windows builds that support showing an OS-level user consent prompt for pinning. Use an action_only message to trigger pinning for these users instead. Also, add a targeting constant using Windows build / UBR for whether or not a user will see an OS-level pin consent prompt when pinning is triggered.
Updated•2 months ago
|
Comment 5•2 months ago
•
|
||
Revert for causing xpcshell failures on test_OnboardingMessageProvider.js.
LATER EDIT: we also found that it caused these eme job failures: https://treeherder.mozilla.org/logviewer?job_id=578720976&repo=autoland&task=C4BhyVTWSTC82OGLqhOa7w.0&lineNumber=5945
Comment 8•2 months ago
•
|
||
Reverted this because it was causing wpt failures.
- Revert link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | leakcheck large nsGlobalWindowInner | /encrypted-media
Please also check these wd failures.
(In reply to Serban Stanca [:SerbanS] from comment #8)
Reverted this because it was causing wpt failures.
- Revert link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | leakcheck large nsGlobalWindowInner | /encrypted-media
Please also check these wd failures.
Thanks for flagging, I didn't see this issue in my try run. I've updated the patch to gate auto-firing the pin action in automation and am re-running the failing jobs on try.
Comment 10•2 months ago
|
||
Comment 11•2 months ago
|
||
| bugherder | ||
Updated•2 months ago
|
Comment 12•2 months ago
|
||
| perf-alert | ||
Perfherder has detected a browsertime performance change from push 1ab252154cab21afed20d53196a2d8c370377817.
No action is required from the author; this comment is provided for informational purposes only.
| Improvement | Test | Platform | Options | Absolute values [old vs new] | Performance Profiles |
|---|---|---|---|---|---|
| 6% | welcome LastVisualChange (doc) | windows11-64-24h2-shippable | fission warm webrender | 2,132.84 ms -> 1,999.88 ms | Before/After |
| 6% | welcome LastVisualChange (doc) | windows11-64-24h2-shippable | cold fission webrender | 2,126.55 ms -> 1,995.22 ms | Before/After |
Need Help or Information?
If you have any questions, please reach out to bacasandrei@mozilla.com. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.
Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.
Comment 14•1 month ago
|
||
I have verified this enhancement using the latest Firefox Nightly 155.0a1 (Build ID: 20260812080401) and Firefox Beta 154.0b10 (Build ID: 20260720195614) builds on Windows 10 x64, macOS 15.7.3, and Ubuntu 24.04.1.
- I confirm the following:
- For builds that do NOT show an OS‑level pin prompt:
- The OS‑level pin prompt doesn’t appear as expected when the pin action is triggered.
- The “Easy Setup” screen still shows both the pin checkbox and the default‑browser checkbox, with no changes in behavior. (except on Linux OS where the pin feature is not supported)
- For builds that DO show an OS‑level pin prompt:
- The OS‑level pin prompt appears as expected when the pin action is triggered.
- After accepting it, Firefox is correctly pinned.
- After unpinning Firefox and restarting the browser with the same profile, I confirm that it does not automatically re‑pin itself and the OS‑level prompt does not appear again.
- The pin checkbox is no longer shown on the “Easy Setup” screen and the default‑browser checkbox remains visible.
- Existing‑user check (for builds that DO show the OS‑level pin prompt):
- Using an already‑used profile where Firefox was unpinned, I restarted Firefox and confirm that it does not silently re‑pin itself. The new behavior only triggers on a genuinely new profile’s first startup.
Updated•1 month ago
|
Description
•