Closed Bug 1481680 Opened 5 years ago Closed 5 years ago

Add mochitests for search shortcuts and fix existing mochitests


(Firefox :: New Tab Page, enhancement, P3)




Firefox 64
64.1 - Sep 14
Tracking Status
firefox63 --- wontfix
firefox64 --- fixed


(Reporter: ahillier, Assigned: andreio)


(Blocks 1 open bug)



(1 file)


Having the search shortcuts experiment turned on by default messes up the existing mochitests; by adding pinned search shortcuts we change the TopSites behaviour (e.g., positions of default sites, number of sites, number of pinned sites) and the tests fail.

For the linked PR we disable the experiment for the tests. We should fix the existing tests and re-enable the experiment for them, and also add new mochitests to cover the behaviour of the new search shortcuts.
Iteration: --- → 63.4 - Aug 20
Priority: -- → P3
Assignee: nobody → andrei.br92
Commit pushed to master at
fix(tests): Revert #4289 head.js changes to turn off pref from browser.ini
Disable pref until bug 1481680
Severity: normal → enhancement
Blocks: 1482398
Ha oops. My simple grep accidentally found a commit message comment "Disable pref until bug 1481680" This is not fixed in 62.
No longer blocks: 1482398
Iteration: 63.4 - Aug 20 → 63.5 - Sep 3
Iteration: 63.5 - Sep 3 → 64.1 (Sep 14)
Commit pushed to master at
Fix Bug 1481680 - Add mochitests for search shortcuts and fix existing mochitests
Closed: 5 years ago
Resolution: --- → FIXED
Blocks: 1489962
Is this something we want to uplift to Beta. It's test-only, so it could be uplifted without approval.
Flags: needinfo?(andrei.br92)
This got backed out unfortunately, we're working on re-landing.
Flags: needinfo?(andrei.br92)
See Also: → 1489962
Backout by
Backed out changeset 8dde92f89a24 for browser_asrouter_cfr.js failures. a=backout

Ping RE: Beta uplift - worth it with a week left before the RC?
Flags: needinfo?(andrei.br92)
QA Contact: tspurway
I wouldn't uplift this since we already had the backout issue. Thanks.
Flags: needinfo?(andrei.br92)
QA Contact: tspurway
Blocks: 1493032
Blocks: 1492556
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.