Closed Bug 1643687 Opened 1 year ago Closed 1 year ago

Search bar becomes unresponsive if using paste and go context menu option with a string while search tip is displayed

Categories

(Firefox :: Address Bar, defect, P1)

Desktop
All
defect
Points:
2

Tracking

()

VERIFIED FIXED
Firefox 79
Iteration:
79.1 - June 1 - June 14
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- fixed
firefox77 --- unaffected
firefox78 --- verified
firefox79 --- verified

People

(Reporter: atrif, Assigned: harry)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image adress_bar_0.gif

Affected versions

  • 78.0b3 (20200604213430)
  • 79.0a1 (20200604220012)

Affected platforms

  • macOS 10.12
  • Windows 10x64

Steps to reproduce

  1. Open Firefox and about:config and set devtools.chrome.enabled:true.
  2. Open browser console and type (async function() { let { ProfileAge } = ChromeUtils.import("resource://gre/modules/ProfileAge.jsm"); let age = await ProfileAge(); age._times = { firstUse: 1368255600000, created: 1368255600000 }; await age.writeTimes(); })();.
  3. Restart the browser and copy a random string in the clipboard (e.g firefox).
  4. Open a new tab and select Paste and go from the context menu option into the address bar.

Expected result

  • The copied string is successfully pasted and searched.

Actual result

Regression range

  • Firefox 77.0.1 (20200602222727) is not affected. I tried searching for one but I don’t know how to trigger the search tip in mozregression. It seems that it doesn't work by following the above steps. I will search for one manually ASAP.

Notes

  • Attached a screen recording with the issue.
Has Regression Range: --- → no
Has STR: --- → yes

maybe related to my changes in bug 1636583, there may be unexpected handling of tips?

Severity: -- → S3
Priority: -- → P1

This can be reproduced when using the search tips "cheat" pref, setting browser.urlbar.searchTips.test.ignoreShowLimits to true.

Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 79.1 - June 1 - June 14
Points: --- → 2

(In reply to Harry Twyford [:harry] from comment #2)

This can be reproduced when using the search tips "cheat" pref, setting browser.urlbar.searchTips.test.ignoreShowLimits to true.

Confirming the regression range with mozregression. Thank you!

Last good revision: 8f6fbfc7558c71bf283696578d148389fc7c2392
First bad revision: 3883a1bffcbee3c00df6c0388c16b86d707e36ec
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=8f6fbfc7558c71bf283696578d148389fc7c2392&tochange=3883a1bffcbee3c00df6c0388c16b86d707e36ec

Has Regression Range: no → yes
Regressed by: 1636583
Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/89dc62c137a1
Resolve incompatibility between Search Tips and Paste & Go. r=mak

Comment on attachment 9156266 [details]
Bug 1643687 - Resolve incompatibility between Search Tips and Paste & Go. r?mak

Beta/Release Uplift Approval Request

  • User impact if declined: Users being shown Search Tips might be able to break the address bar. Search Tips are shown mostly to new profiles. Also Search Tips are launching on ESR for the first time in 78.
  • 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: See comment 0 and comment 2.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): The fix is targeted specifically towards Urlbar Tips, which are shown in pretty limited circumstances. This fix wouldn't be able to break any other part of the Urlbar.
  • String changes made/needed:
Attachment #9156266 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
QA Whiteboard: [qa-triaged]

Verified fixed with Firefox 79.0a1 (20200614215300) on Windows 10x64, macOS 10.12 and Ubuntu 18.04. The search bar works as expected when using Paste and Go context menu option while search tip is displayed.

Comment on attachment 9156266 [details]
Bug 1643687 - Resolve incompatibility between Search Tips and Paste & Go. r?mak

approved for 78.0b8

Attachment #9156266 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified using Firefox 78.0b8 (20200615143809) from comment 10 on Windows 10x64, macOS 10.12 and Ubuntu 18.04

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