Update Firefox for Android web app manifest features

[jonalmeida]

Summary

Updating the state of web manifest properties that are used by Firefox for Android.

Members that are parsed by Gecko, but not used by Firefox for Android:

• description
• dir
• lang
• related_applications
• prefer_related_applications
• share_target

Members that are parsed by Gecko, but not used in Firefox and Firefox for Android:

• id

Test results and supporting details

• Properties parsed by Gecko and Firefox for Android: https://searchfox.org/mozilla-central/rev/ece43b04e7baa4680dac46a06d5ad42b27b124f4/mobile/android/android-components/components/concept/engine/src/main/java/mozilla/components/concept/engine/manifest/WebAppManifestParser.kt#64-78
• Properties parsed only by Gecko: https://searchfox.org/mozilla-central/rev/ece43b04e7baa4680dac46a06d5ad42b27b124f4/dom/manifest/ManifestProcessor.sys.mjs#118

Related Firefox for Android bugs:

• Install a PWA to my home screen mozilla-mobile/fenix#778
• As a User I want websites that I have saved to my phone's home screen to look like native applications mozilla-mobile/fenix#772
• Open installed PWAs in fullscreen / standalone mode mozilla-mobile/fenix#770
• Use PWAs while offline mozilla-mobile/fenix#769
• Refresh the currently displayed page in a PWA mozilla-mobile/fenix#779

[jonalmeida]

@Elchi3 could you advise if the updates below are correct by:

• Taking inspiration from Updated Safari support for Web App Manifests #23640, we have marked only Firefox for Android as false for properties that are parsed by Gecko but we do not implement the feature.
• Leaving Firefox as null even though they share the same engine that parses the manifest values.

[Elchi3]

Thanks so much @jonalmeida! 👍

I would say that we should set firefox desktop to false everywhere then. Do you want to do that in this PR or do you want me to do that in a follow-up?

[dipikabh]

@jonalmeida, in an offline discussion, Florian has suggested that we could use the following approach for manifest members that are parsed but do not currently cause any visible change for users. This could help convey the complete picture about the present "support" of the member.

browser-compat-data/css/properties/d.json

Line 27 in 40f2d89

"notes": "The property parses, but has no effect."

[Elchi3]

I can't add suggestions for the following three members, but firefox_android also needs to have the same note there:

dir
lang
share_target

[github-actions]

This pull request has merge conflicts that must be resolved before it can be merged.

[jonalmeida]

@Elchi3 Sorry about the slow turn around time. I've update the PR with most of the recommendations. I do have one more question below:

I can't add suggestions for the following three members, but firefox_android also needs to have the same note there:

dir
lang

I noticed that there aren't any files for dir and lang at the moment (probably because of the note on MDN here that says they are not implemented in any engine). Should I create those files and mark firefox as false and firefox_android with the parsing note? If yes, should I do that in this PR or would it be better to make a separate one for clarity?

share_target

Done.

[Elchi3]

I noticed that there aren't any files for dir and lang at the moment (probably because of the note on MDN here that says they are not implemented in any engine). Should I create those files and mark firefox as false and firefox_android with the parsing note? If yes, should I do that in this PR or would it be better to make a separate one for clarity?

Oh sorry, my bad, I was just going through your list above. It's fine to not add these files for the moment. We only add features to this repo when it sees at least one implementation somewhere and that is not given for these two. So that's all good as is. Thank you!

[Elchi3]

Fantastic work! Thank you so much @jonalmeida and @dipikabh 👍

[dipikabh]

Thanks a lot, @jonalmeida, for these updates and your diligence! 🙌
Thanks for reviewing, @Elchi3!