Open
Bug 1915774
Opened 2 months ago
Updated 21 days ago
Stop using setFaviconForPage of head_common.js
Categories
(Toolkit :: Places, task, P3)
Toolkit
Places
Tracking
()
NEW
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.
Comment 1•2 months ago
|
||
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]
Updated•2 months ago
|
Updated•21 days ago
|
Severity: -- → N/A
Priority: -- → P3
You need to log in
before you can comment on or make changes to this bug.
Description
•