Closed
Bug 1950287
Opened 1 year ago
Closed 1 year ago
Consider merging FetchIconPerSpec() and FetchIconInfo()
Categories
(Toolkit :: Places, task, P3)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
141 Branch
| Tracking | Status | |
|---|---|---|
| firefox141 | --- | fixed |
People
(Reporter: daisuke, Assigned: daisuke)
References
Details
(Whiteboard: [sng-places-tech-debt])
Attachments
(1 file)
As suggested in phabricator, it may be wasteful to consider merging FetchIconPerSpec() and FetchIconInfo() as we might be able to get all favicon data from one query.
|
||
Updated•1 year ago
|
Severity: -- → N/A
Type: enhancement → task
Priority: -- → P3
Whiteboard: [sng-places-tech-debt]
|
||
Updated•1 year ago
|
|
Assignee | |
Updated•1 year ago
|
Assignee: nobody → daisuke
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•1 year ago
|
||
Depends on D244552
Pushed by dakatsuka.birchill@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/fd3b1149ae9c
https://hg.mozilla.org/integration/autoland/rev/b9f5012c482a
Merge FetchIconPerSpec() and FetchIconInfo() r=mak
|
||
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox141:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 141 Branch
|
||
Updated•1 year ago
|
QA Whiteboard: [qa-triage-done-c142/b141]
You need to log in
before you can comment on or make changes to this bug.
Description
•