Entering text in the urlbar ending with a carriage return (0D), line feed (0A), or both selects search shortcut in first position of Top Sites list
Categories
(Firefox :: Address Bar, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox88 | --- | wontfix |
firefox89 | --- | wontfix |
firefox90 | --- | verified |
People
(Reporter: jscher2000, Assigned: daisuke)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta-
|
Details | Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:88.0) Gecko/20100101 Firefox/88.0
Steps to reproduce:
Select and copy a cell in Excel or text in a text document including the subsequent line break, paste to the address bar, and press Enter.
Actual results:
If a pinned search shortcut is in the first position of the Top Sites list and the user is displaying Top Sites in the address bar autocomplete drop-down -- this is the default configuration -- instead of handling the address or query string normally, Firefox behaves as though the user clicked the first Top Site, shows the relevant search engine name at the left end of the address bar, and discards the pasted input.
(As reported in: https://support.mozilla.org/questions/1335231 and https://www.reddit.com/r/firefox/comments/n5tait/annoying_blue_amazon_search_box_when_opening_a/ )
Expected results:
Firefox should disregard the 0D 0A on URL bar input.
Reporter | ||
Updated•3 years ago
|
Reporter | ||
Comment 1•3 years ago
|
||
Mozregression points to bug 1185358 as the patch behind this change.
Updated•3 years ago
|
Updated•3 years ago
|
Comment 2•3 years ago
|
||
Thanks for debugging this, jscher. It looks like the urlbar view must be opened at least once before pasting and pressing enter, and the pasted string must end in a CR, LF, or CR+LF. Here are the STR that work for me:
- Open a new tab
- Click the urlbar or press accel+L open the view with your top sites
- Paste a string that ends in a line feed (0A), carriage return (0D), or CR+LF (0D0A)
- Observe that the view does not open like it should
- Press enter
- Observe that Firefox behaves as if the first result in the view was picked (I'm guessing -- my first result is the Amazon shortcut/keyword offer, and it gets picked)
Comment 3•3 years ago
|
||
Daisuke, do you have any time to look into this? No problem if not, I probably will if you don't.
Updated•3 years ago
|
Assignee | ||
Comment 4•3 years ago
|
||
Sure! I will take this issue.
Updated•3 years ago
|
Assignee | ||
Comment 6•3 years ago
|
||
Pushed by dakatsuka.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84eab701c0e6 Request autocomplete upon pasting value. r=adw
Comment 8•3 years ago
|
||
bugherder |
Comment 9•3 years ago
|
||
The patch landed in nightly and beta is affected.
:daisuke, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 10•3 years ago
|
||
Comment on attachment 9221468 [details]
Bug 1709971: Request autocomplete upon pasting value.
Beta/Release Uplift Approval Request
- User impact if declined: Even if user pastes ipaddress/url that ends with \r by middle clicking, then type enter, it can not go to the page.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The main changes are only 6 lines, and it affects only pasting by middle clicking.
- String changes made/needed: none
Comment 11•3 years ago
|
||
Comment on attachment 9221468 [details]
Bug 1709971: Request autocomplete upon pasting value.
We shipped 87 and 88 with this bug, it recently landed on nightly and we are nearing the beta cycle, I think it should ride the 90 train, thanks.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Was not able to reproduce the issue at all by using Firefox 80.0, 80.0.1, Nightly 90.0a1 (2021-05-06) under both macOS 10.15.7 and Windows 10.
Jscher2000, could you verify please on latest builds if the issue is fixed for you? Thank you!
Reporter | ||
Comment 14•3 years ago
|
||
(In reply to Catalin Sasca, QA [:csasca] from comment #13)
Was not able to reproduce the issue at all by using Firefox 80.0, 80.0.1, Nightly 90.0a1 (2021-05-06) under both macOS 10.15.7 and Windows 10.
Jscher2000, could you verify please on latest builds if the issue is fixed for you? Thank you!
Version 90.0b4 (DevEd) looks good.
Description
•