1414498 - Support tabs.favIconUrl on Android

Status: NEW

(Firefox for Android :: WebExtensions, defect, P3)

Description

[Baptiste Thémine]

Attachment: Test.zip

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0
Build ID: 20171024165158

Steps to reproduce:

I tryed to get favIconUrl from tabs on Firefox Android with the following code :

//Tab open
browser.tabs.onCreated.addListener(function(tab){
	console.log(tab.favIconUrl);
});

//Tab update
browser.tabs.onUpdated.addListener(function(tabid,info,tab){
	console.log(tab.favIconUrl);
});

Related Topic : https://discourse.mozilla.org/t/webextensions-get-tab-faviconurl-on-firefox-android/20958


Actual results:

The code above is working on Firefox Desktop but the favIconUrl is always undefined on Firefox Android.
The attached file contains Test.xpi and screenshots of both result in Firefox Desktop and Firefox Android.
Tested on Firefox 56 and Firefox Android Nightly 58.




Expected results:

The favIconUrl property should be defined in Firefox Android.

Comment 1

[Kris Maglione [:kmag]]

This is easier said than done. The tab favicon is available on the Java side, but there's no simple way to reflect it into JS.

Comment 2

[Baptiste Thémine]

I don't understand. Is tabs.favIconUrl not already supposed to be supported on Android since version 54?
See compatibility table on https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/tabs/Tab

Comment 3

[Kris Maglione [:kmag]]

(In reply to Baptiste Thémine from comment #2)
> Is tabs.favIconUrl not already supposed to be supported on Android since
> version 54?

No, favIconUrl support was never implemented for Android.

Comment 4

[Baptiste Thémine]

Ok, it was surely a mistake. I will create a pull request to https://github.com/mdn/browser-compat-data for this issue.