Closed Bug 1960900 Opened 10 months ago Closed 9 months ago

USB remains after address temporarily modified then reset

Categories

(Firefox :: Address Bar, defect, P1)

Firefox 139
defect

Tracking

()

VERIFIED FIXED
140 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr128 --- unaffected
firefox137 --- unaffected
firefox138 --- unaffected
firefox139 --- verified
firefox140 --- verified

People

(Reporter: ke5trel, Assigned: daisuke)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [sng-scotchbonnet-followon])

Attachments

(3 files)

STR:

  1. Visit a site like https://mozilla.org.
  2. Click in the address bar.
  3. Modify the address.
  4. Press Esc key several times to reset address.

Expected:
Unified Search Button (USB) disappears when address unmodified.

Actual:
USB remains visible for unmodified address, potentially confusing site identity (Bug 1928154) until page is reloaded.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=c25d04a68ad925a428a52a6d4d76a2559eb3ef49&tochange=094882056dd4739c2799876a466680d0b8190494

Regressed by Bug 1957630.

:daisuke, since you are the author of the regressor, bug 1957630, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(daisuke)

Thank you very much for your report, Kestrel!

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Flags: needinfo?(daisuke)
Severity: -- → S3
Priority: -- → P1
Whiteboard: [sng-scotchbonnet-followon]

Set release status flags based on info from the regressing bug 1957630

:daleharvey do you think you can review this patch? Last week of betas is next week and would be good to land this during early beta so that it has some bake time.

Flags: needinfo?(dharvey)
Pushed by dakatsuka.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/75c313938c8e Not use isUnifiedSearchButtonAvailable upon reverting urlbar r=daleharvey
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 140 Branch

The patch landed in nightly and beta is affected.
:daisuke, is this bug important enough to require an uplift?

For more information, please visit BugBot documentation.

Flags: needinfo?(daisuke)

We are in the last week of beta for Fx139 with only 2 beta build left in the cycle.
:daisuke, could you please take a look at Comment 8?

Flags: needinfo?(dharvey)

Thank you very much for the notice, Nonal!
I will do it.

Flags: needinfo?(daisuke)
Attachment #9487146 - Flags: approval-mozilla-beta?

firefox-beta Uplift Approval Request

  • User impact if declined: Unified Search Button still be shown even after address temporarily modified then reset
  • Code covered by automated testing: yes
  • Fix verified in Nightly: yes
  • Needs manual QE test: no
  • Steps to reproduce for manual QE testing: No
  • Risk associated with taking this patch: Low
  • Explanation of risk level: Very small patch and affects only for Unified Search Button
  • String changes made/needed: None
  • Is Android affected?: no
Attachment #9487146 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Reproduced the issue with Firefox 139.0a1 (2025-04-16) on macOS 14.
The issue is verified fixed with Firefox 140.0a1 (20250520213629) and Firefox 139.0b9 (20250516091640) on macOS 14, Windows 10 and Ubuntu 22.

Status: RESOLVED → VERIFIED
Has STR: --- → yes
QA Whiteboard: [qa-ver-done-c140/b139]
QA Contact: dlucaci
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: