Closed Bug 1730702 Opened 3 years ago Closed 3 years ago

The user is not redirected to the Preferences Search pane after clicking the “Customize in preferences” modal button if other than Preferences Search pane is already opened in an existing tab

Categories

(Firefox :: Address Bar, defect, P1)

Desktop
All
defect
Points:
2

Tracking

()

VERIFIED FIXED
94 Branch
Iteration:
94.1 - Sep 6 - Sept 19
Tracking Status
firefox92 --- verified
firefox93 --- verified
firefox94 --- verified

People

(Reporter: cfat, Assigned: adw)

References

Details

Attachments

(2 files)

[Affected versions]:

  • Firefox Beta 93.0b4 (Build ID: 20210912185727)
  • Firefox Nightly 94.0a1 (Build ID: 20210913213224)

[Affected Platforms]:

  • Windows 10 x64;
  • macOS 10.15.7;
  • Linux Ubuntu 20.04 x64;

[Prerequisites]:

  • Have the Session restore enabled.
  • Have the “about:preferences” page opened, with other than Search pane displayed (e.g. “Sync”).

[Steps to reproduce]:

  1. Open the Firefox browser with the profile from prerequisites.
  2. Set the following preferences:
    browser.urlbar.quicksuggest.enabled: true
    browser.urlbar.quicksuggest.scenario: online
    browser.urlbar.quicksuggest.shouldShowOnboardingDialog: true
    browser.urlbar.suggest.quicksuggest: false
    browser.urlbar.suggest.quicksuggest.sponsored: false
  3. Restart the browser.
  4. Click the “Customize in settings” button from the Onboarding modal.
  5. Observe the behavior.

[Expected result]:

  • The user is redirected to the already opened “about:preferences” page, with the Search pane in focus (Privacy pane after bug 1730580 lands).

[Actual result]:

  • The user is redirected to the already opened “about:preferences” page, with the Sync pane in focus.

[Notes]:

  • The following errors are displayed in the Browser Console when clicking the “Customize in settings” button from the Onboarding modal:

Nightly
[bug 1345098] Lazy browser prematurely inserted via 'loadURI' property access: getter@chrome://browser/content/tabbrowser.js:2271:39 switchIfURIInWindow@chrome://browser/content/browser.js:8634:11 switchToTabHavingURI@chrome://browser/content/browser.js:8657:26 openPreferences@chrome://browser/content/utilityOverlay.js:1045:20 maybeShowOnboardingDialog@resource:///modules/UrlbarQuickSuggest.jsm:265:11 tabbrowser.js:2271:25

TypeError: can't access property "gotoPref", browser.contentWindow is null

Beta
TypeError: browser.contentWindow is nullutilityOverlay.js:1064:5

  • Attached is a screen recording of the issue.
Depends on: 1727907

This is most likely a bug in the preferences UI and not on our side, but I'm not 100% sure. Either way, to see the problem, you have to start up Firefox with restored tabs, and one of those tabs has to be about:preferences, and the onboarding has to be triggered, all at the same time. Not impossible but it seems like an uncommon scenario.

Group: mozilla-employee-confidential
Assignee: nobody → adw
Status: NEW → ASSIGNED
Iteration: --- → 94.1 - Sep 6 - Sept 19
Points: --- → 2
Priority: -- → P1

When switching to an already open but lazy/unloaded about:preferences tab and
highlighting a section in it, wait for it to load first.

Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3b8a4f25b257
When switching to an already open but lazy/unloaded about:preferences tab and highlighting a section in it, wait for it to load first. r=dao
Flags: qe-verify+
Flags: in-testsuite-
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
See Also: → 1731061

Comment on attachment 9241256 [details]
Bug 1730702 - When switching to an already open but lazy/unloaded about:preferences tab and highlighting a section in it, wait for it to load first.

Beta/Release Uplift Approval Request

  • User impact if declined: Needed for important Firefox Suggest rollouts on 93 and 92
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: Should be clear from comment 0
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is a small change that waits for about:preferences to load before trying to highlight a section in it.
  • String changes made/needed:
Attachment #9241256 - Flags: approval-mozilla-release?
Attachment #9241256 - Flags: approval-mozilla-beta?

Comment on attachment 9241256 [details]
Bug 1730702 - When switching to an already open but lazy/unloaded about:preferences tab and highlighting a section in it, wait for it to load first.

Needed for experiments targeting Fx92. Approved for 93.0b6.

Attachment #9241256 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Depends on: 1731104

I have verified that the issue is no longer reproducible on Firefox 94, for non-US region (RO, DE). The user is correctly redirected to the Privacy pane, with the Address Bar - Firefox Suggest section highlighted. Tested using Firefox Nightly 94.0a1 (Build ID: 20210916125444) on Windows 10, macOS 11.1, and Ubuntu 20.04 x64.

By setting the US region, the issue is still reproducible. In order to track the issue, I have logged Bug 1731104.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-triaged]
  • I have verified that the issue is no longer reproducible on Firefox Beta 93.06 (Build ID: 20210916185649) for non-US region (RO, DE).
  • The user is correctly redirected to the Privacy pane, with the Address Bar - Firefox Suggest section highlighted. Tested across platforms Windows 10, macOS 11.1, and Ubuntu 20.04 x64.

Comment on attachment 9241256 [details]
Bug 1730702 - When switching to an already open but lazy/unloaded about:preferences tab and highlighting a section in it, wait for it to load first.

Approved for 92.0.1.

Attachment #9241256 - Flags: approval-mozilla-release? → approval-mozilla-release+
  • I have verified this issue using the Candidate 92.0.1 build (Build ID: 20210920202613) on Windows 10 x64, macOS 10.15.7 and Linux Ubuntu 20.04 x64.

  • The user is correctly redirected to the Privacy pane, with the Address Bar - Firefox Suggest section highlighted. Tested with non-US region (ro).

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.