Open Bug 1915774 Opened 2 months ago Updated 21 days ago

Stop using setFaviconForPage of head_common.js

Categories

(Toolkit :: Places, task, P3)

task

Tracking

()

People

(Reporter: daisuke, Unassigned)

References

Details

(Whiteboard: [favicons-2024][sng])

Many tests are using setFaviconForPage of head_common.js. But, as we stop using nsIFaviconService.setAndFetchFaviconForPage() in bug 1894633, the function came to simulate old behavior of setAndFetchFaviconForPage. As it hides API logic, we stop using this function, and then will do the old behavior in each tests.

The main concern here is many tests don't need to fetch from the database, and even less from the network.
So each test should just do what they effectively need.

Whiteboard: [favicons-2024][sng]
Severity: -- → N/A
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.