Closed Bug 1474981 Opened 6 years ago Closed 6 years ago

Search engine can't be loaded from AMO (favicon problem)

Categories

(Firefox :: Search, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 63
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- unaffected
firefox61 --- unaffected
firefox62 --- unaffected
firefox63 + verified
firefox64 --- verified

People

(Reporter: mkaply, Assigned: mossop)

References

Details

(Keywords: regression)

Attachments

(2 files)

If you go to a search engine on AMO:

https://addons.mozilla.org/en-US/firefox/addon/tv-tropes-94107/?src=search

and try to add it, you get:

TypeError: tabbrowser.shouldLoadFavIcon is not a function[Learn More] MainProcessSingleton.js:26:45 

on the console.

Tracked this down to a regression from bug 1453751
Flags: needinfo?(dtownsend)
Component: Tabbed Browser → Search
Assignee: nobody → dtownsend
Flags: needinfo?(dtownsend)
Priority: -- → P1
Comment on attachment 8991941 [details]
Bug 1474981: Allow newly added search engines to use the tab icon if it is a data uri.

Mike de Boer [:mikedeboer] has approved the revision.

https://phabricator.services.mozilla.com/D2123
Attachment #8991941 - Flags: review+
Pushed by dtownsend@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/977302799d70
Allow newly added search engines to use the tab icon if it is a data uri. r=mikedeboer
https://hg.mozilla.org/mozilla-central/rev/977302799d70
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Attached image Bug1474981.png
I was able to reproduce this issue on Firefox 63.0a1(20180711100118) under Win 7 64-bit and  Mac OS X 10.13.3.

This issue is verified as fixed on Firefox 64.0a1(20180916220033) and Firefox 63.0b6(20180913141435) under Win 7 64-bit and Mac OS X 10.13.3.

Please see the attached screenshot.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: