Closed Bug 1460570 Opened 3 years ago Closed 3 years ago

Merge mozIAsyncFavicons into nsIFaviconService

Categories

(Toolkit :: Places, enhancement, P3)

Product:
Toolkit
Component:
Places
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: mak, Assigned: mak)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

Bug 1460570 - Merge mozIAsyncFavicons into nsIFaviconService.
59 bytes, text/x-review-board-request
standard8
: review+
Details 
Now that icons async-ification is complete, there's no point in having 2 separate interfaces for icons, so we can simplify this.
Blocks: 1460575
Comment on attachment 8974670 [details]
Bug 1460570 - Merge mozIAsyncFavicons into nsIFaviconService.

https://reviewboard.mozilla.org/r/243054/#review248874
Attachment #8974670 - Flags: review?(standard8) → review+
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/e241b7fa7f9a
Merge mozIAsyncFavicons into nsIFaviconService. r=standard8
https://hg.mozilla.org/mozilla-central/rev/e241b7fa7f9a
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.