Change search settings redirects to General section of about:preferences

VERIFIED FIXED in Firefox 55

Status

()

Firefox
Preferences
P1
normal
VERIFIED FIXED
a year ago
11 months ago

People

(Reporter: Petruta Rasa [Away. Please needinfo? bogdan.maris@softvision.ro], Assigned: rickychien)

Tracking

({regression})

55 Branch
Firefox 56
regression
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox54 unaffected, firefox55+ verified, firefox56 verified)

Details

(Whiteboard: [photon-preference])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(3 attachments)

[Note]:
- This doesn't reproduce on Firefox 54 beta 13 with one-off searches from awesome bar force enabled 

[Affected versions]:
- Firefox 55 beta 1

[Affected platforms]:
- Win 10 64-bit
- Mac OS X 10.12.5
- Ubuntu 14.04 64-bit

[Steps to reproduce]:
1. Click the magnifying glass from Search toolbar and select "Change Search Settings" button
OR 
Select "Change search settings" gear icon from url dropdown

[Expected result]:
- about:preferences#search opens

[Actual result]:
- about:preferences#general-search opens with focus on General section from about:preferences

[Regression range]:
- This could be caused by 1335907 (about:preferences was reorganized) which didn't ride the 55 train. I will continue the investigation.

[Additional notes]:
- The issue also occurs for other about:preferences redirects. Eg: Go to about:telemetry and click the "Change" links - about:preferences#privacy-reports is shown in the url while General section is focused.
about:preferences#search should go to the right place in both cases, if preferences have been reorganized, it will be in #general, otherwise it will be about:preferences#search.

What's the bug? Are we not going to the screen that allows to disable search suggestions in the Location Bar in both cases?
Created attachment 8877119 [details]
about:preferences#general-search

Firefox 55 beta doesn't recognize the new urls from Preferences reorg so it will show the main section.

Change Search Settings should redirect to about:preferences#search.
Hey Tim, can you or someone else working on preference look in to this? This is tracking 55.
Flags: needinfo?(timdream)
Let's take this to triage today. Thanks.
Flags: needinfo?(timdream)
Priority: -- → P1
Whiteboard: [photon-preference][triage]
regression in beta from the pref reorg, tracking for 55.
tracking-firefox55: --- → +
will be fixed in 55 beta.
Whiteboard: [photon-preference][triage] → [photon-preference]
(Assignee)

Updated

a year ago
Assignee: nobody → rchien
Status: NEW → ASSIGNED
Flags: qe-verify+
Note that his issue can be reproduced easily in Nightly by setting `browser.preferences.useOldOrganization = true` in about:config.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Duplicate of this bug: 1373834
Jaws, can you take a look and help review the patch? thanks
Flags: needinfo?(jaws)

Comment 13

a year ago
mozreview-review
Comment on attachment 8877927 [details]
Bug 1372528 - Change search settings redirects to General section of about:preferences

https://reviewboard.mozilla.org/r/149332/#review155418

::: browser/components/search/content/search.xml:1476
(Diff revision 2)
>  
>        <method name="showSettings">
>          <body><![CDATA[
>            BrowserUITelemetry.countSearchSettingsEvent(this.telemetryOrigin);
> +
> +          if (Preferences.get("browser.preferences.useOldOrganization")) {

I don't see the Preferences object referenced in this file already. Have you tested this to confirm that it will work?
Attachment #8877927 - Flags: review?(jaws) → review+
Jared, this is triggered by browser's url search bar not in Preferences itself.

I've tested and confirmed that clicking "Change Search Settings" button can redirect to the right preferences sub-panel whatever opt-in or opt-out the `browser.preferences.useOldOrganization`.
(Assignee)

Updated

a year ago
Flags: needinfo?(jaws)

Comment 15

a year ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f4a46277935b
Change search settings redirects to General section of about:preferences r=jaws

Comment 16

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f4a46277935b
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Confirmed that this grafts cleanly to Beta. Please request approval when you get a chance.
status-firefox56: unaffected → fixed
status-firefox-esr52: --- → unaffected
Flags: needinfo?(rchien)
Comment on attachment 8877927 [details]
Bug 1372528 - Change search settings redirects to General section of about:preferences

Approval Request Comment
[Feature/Bug causing the regression]: unknown
[User impact if declined]: Search toolbar's "Change Search Settings" doesn't work
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: see STR in description
[List of other uplifts needed for the feature/fix]: no
[Is the change risky?]: no
[Why is the change risky/not risky?]: fixed button behavior will not crash entire Firefox.
[String changes made/needed]: none
Flags: needinfo?(rchien)
Attachment #8877927 - Flags: approval-mozilla-beta?
Comment on attachment 8877927 [details]
Bug 1372528 - Change search settings redirects to General section of about:preferences

fix search settings link, beta55+
Attachment #8877927 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 20

a year ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/eddd20864565
status-firefox55: affected → fixed
Created attachment 8880497 [details] [diff] [review]
Patch to fix beta55 browser_urlBarSearchSuggestions_opt-out.js
Flags: needinfo?(ryanvm)
Thanks for the quick fix :)
https://hg.mozilla.org/releases/mozilla-beta/rev/a6aa59c61c8d
Flags: needinfo?(ryanvm)

Updated

a year ago
Duplicate of this bug: 1376001

Comment 24

a year ago
I have reproduced this bug with Nightly 56.0a1 (2017-06-13) on Windows 8, 64-bit.

The bug's fix is now verified on Latest Beta 55.0b6

Build ID 	20170629005143
User Agent 	Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0

[bugday-20170628]
Verified as fixed using Firefox 55 beta 6 under Win 10 64 bit, Ubuntu 14.04 32-bit and Mac OS X 10.12 and logged follow-up bug 1378136.
Status: RESOLVED → VERIFIED
status-firefox55: fixed → verified
status-firefox56: fixed → verified
Flags: qe-verify+

Updated

11 months ago
Flags: qe-verify+
QA Contact: hani.yacoub
You need to log in before you can comment on or make changes to this bug.