Same icon is presented inside Address/search bars for add search engines after they are removed
Categories
(Firefox :: Search, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox91 | --- | wontfix |
firefox92 | --- | wontfix |
firefox93 | --- | fix-optional |
People
(Reporter: atrif, Unassigned)
Details
(Keywords: regression)
Attachments
(1 file)
925.43 KB,
image/gif
|
Details |
Affected versions
- 91.0.1 (20210816143654)
- 92.0b6 (20210819185750)
- 93.0a1 (20210819214942)
Affected platforms
- Windows 10x64
- macOS 10.15
- Ubuntu 21.04
Preconditions
- search bar added to toolbar
Steps to reproduce
- Open Firefox and go to Youtube.com and add the search engine.
- In the same tab go to olx.ro and add the search engine.
- Go to about:preferences-> Search and remove the newly added search engines.
- Go to olx.ro page and click on the Search icon from the Search bar.
Expected result
- The search engines have the expected icon.
Actual result
- Search engines have the same icon.
Regression range
- This is working as expected on Firefox 36.0a1 (2014-11-02). Unfortunately after implementation of about:pages the remove button is not present e.g with 37.0a1 (2014-12-02), so I can’t find a proper regression range.
Notes
- Attached a screen recording.
Reporter | ||
Updated•3 years ago
|
Comment 1•3 years ago
|
||
Low priority, since this is an unusual action, and is easily solved by page reload.
Updated•3 years ago
|
Comment 2•3 years ago
|
||
I tried to do a regression but the builds are too old to generate one (based on the last line from mozregression->"57:22.67 INFO: There are no build artifacts for these changesets (they are probably too old)").
Last build that worked for me was firefox-37.0a1.en-US.win64-x86_64.zip, but I was able to add only one search item. Last pushlog generated is not concludent
55:45.14 INFO: Last good revision: bb8d6034f5f2 (2015-01-11)
55:45.14 INFO: First bad revision: 29b05d283b00 (2015-01-30)
55:45.14 INFO: Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=bb8d6034f5f2&tochange=29b05d283b00.
Most probably the build got broken between (2015-01-11) and (2015-01-30).
Comment 3•3 years ago
|
||
Thanks Monica! I think we can remove the regressionwindow-wanted
keyword based on the above.
Description
•