The weather suggestion can flicker and pop in
Categories
(Firefox :: Address Bar, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox109 | --- | fixed |
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR
- If weather suggestions are already disabled (and they're currently disabled by default), skip to the next step. Otherwise disable them and open a new window
- Click in the urlbar to show top sites
- Enable weather suggestions and make sure a suggestion is successfully prefetched
- Click in the urlbar again
There's flickering in the view: First the top sites are shown without the weather suggestion, and then the weather suggestion pops in. It will keep happening in the window for as long as it's open, weather suggestions are enabled, and the top sites don't change.
The problem is that the view's cached top-sites context is created in step 2 and it's never invalidated or recached. That's because the recache logic only checks the first result after a search finishes, which is a problem because now we're showing results in the top sites context that aren't actually top sites, namely the weather suggestion. We should instead check for the presence of a top site result anywhere in the results list.
Assignee | ||
Comment 1•2 years ago
|
||
This fixes the bug, adds a task to the existing test, and improves the test by
not waiting for the search to finish before checking the view. It also
simplifies the test a little by collecting URLs from rows into an array and
using Assert.deepEqual()
instead of comparing rows one by one.
Comment 3•2 years ago
|
||
bugherder |
Description
•