Closed Bug 1413648 Opened 4 years ago Closed 4 years ago

default icon no longer appears for page actions


(WebExtensions :: Frontend, defect, P2)

49 Branch


(firefox-esr52 unaffected, firefox56 unaffected, firefox57 unaffected, firefox58 verified, firefox59 verified)

Tracking Status
firefox-esr52 --- unaffected
firefox56 --- unaffected
firefox57 --- unaffected
firefox58 --- verified
firefox59 --- verified


(Reporter: mixedpuppy, Assigned: mixedpuppy)



(Keywords: regression)


(2 files)

Using a manifest that does not define any icon results in a default extension icon being used.  After bug 1395387 this no longer happens for page actions.
Priority: -- → P2
Keywords: regression
This re-introduces getIconData which was removed in bug 1395387.  The trick here is that getPreferredIcon actually is where we add the default extension icon if the manifest does not contain the icon.  We also limited to 16/32 for icons at this point.  getIconData is different than before in that it returns an object usable by PageActions rather than the css we used to return.
Comment on attachment 8924285 [details]
Bug 1413648 fix default extension icon use,

lgtm ftw!  I should have done it this way. :-)  Here's the additional review you asked for, but take it with a grain of salt, or as an f+, since I'm not a webextensions peer or anything.
Attachment #8924285 - Flags: review+
Comment on attachment 8924285 [details]
Bug 1413648 fix default extension icon use,

How about adding a test too?
Attachment #8924285 - Flags: review?(aswan) → review+
Pushed by
fix default extension icon use, r=aswan
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Attached file
I can reproduce this issue on Firefox 58.0a1 (20171101104430) under Wind 7.

As you can see in the screenshot, the icon is not displayed for the default_icon.

On Firefox 59.0a1 (20171115220414) under Wind 7 the icon is displayed if you have the default_icon removed from the manifest.json file but in case that default_icon has an empty string, the icon is not displayed.

Is this expected? Or I should log another bug for this issue?
Flags: needinfo?(mixedpuppy)
See Also: → 1419109
bug 1419109 for the empty icon.  This is not caused by the changes in this bug (or the related bugs), it happened prior.
Flags: needinfo?(mixedpuppy)
Assignee: nobody → mixedpuppy
Based on comment 8 and comment 9, this issue is verified as fixed on Firefox 59.0a1 (20171120222519) and Firefox 58.0b4 (20171115114231) under Wind 7 64-bit and Mac OS X 10.13.
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.