[meta] Remove Fennec code
Categories
(Firefox for Android :: General, enhancement, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr68 | --- | wontfix |
People
(Reporter: snorp, Unassigned)
References
(Depends on 11 open bugs)
Details
(Keywords: meta)
Attachments
(4 obsolete files)
Out with the old, in with the new.
|
Reporter | |
Updated•3 months ago
|
|
||
Comment 1•2 months ago
|
||
One thing to note is that we'll want to keep the l10n config files for fennec around as long as we ship that from ESR. The reason is that the cross-channel code can merge l10n files across branches, but not the config files, so we're taking those from m-c.
|
|
Reporter | |
Comment 2•2 months ago
|
||
(In reply to Axel Hecht [:Pike] from comment #1)
One thing to note is that we'll want to keep the l10n config files for fennec around as long as we ship that from ESR. The reason is that the cross-channel code can merge l10n files across branches, but not the config files, so we're taking those from m-c.
OK that's good info, thanks. I don't think I actually know which files you're referring to, though. Could you share a link?
|
|
||
Comment 3•2 months ago
|
||
Sure,
mobile/locales/l10n.ini
mobile/locales/filter.py
mobile/android/locales/l10n.ini
mobile/android/locales/maemo-locales
mobile/android/locales/all-locales
mobile/android/locales/filter.py
mobile/android/locales/l10n.toml
(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) (he/him) from comment #2)
(In reply to Axel Hecht [:Pike] from comment #1)
One thing to note is that we'll want to keep the l10n config files for fennec around as long as we ship that from ESR. The reason is that the cross-channel code can merge l10n files across branches, but not the config files, so we're taking those from m-c.
OK that's good info, thanks. I don't think I actually know which files you're referring to, though. Could you share a link?
Just FYI, I've taken @snorp's advice and just torn Fennec out entirely. My current patches (see https://treeherder.mozilla.org/#/jobs?repo=try&revision=4b337cdb7e0bd2fba187bbbb99525a6339660086) don't touch l10n in any way, for this and related reasons. Tearing out Fennec omnijar pieces, and simplifying the l10n process, will be follow-up.
|
||
Updated•2 months ago
|
|
||
Comment 5•2 months ago
|
||
|
|
||
Comment 6•2 months ago
|
||
Some of this may be able to be cleaned up further.
|
|
||
Comment 7•2 months ago
|
||
Didn't want to remove all the WebAuthn code so I left it there in case it's
useful.
|
|
||
Comment 8•2 months ago
|
||
While we're at it...
|
||
Comment 9•2 months ago
|
||
Comment on attachment 9101824 [details]
Bug 1582218 - Remove Fennec history implementation.
Revision D49586 was moved to bug 1589327. Setting attachment 9101824 [details] to obsolete.
|
|
||
Comment 10•2 months ago
|
||
Whoops, that was meant to be in bug 1589327.
|
|
||
Comment 11•2 months ago
|
||
Comment on attachment 9101825 [details]
Bug 1582218 - Remove some other various fennec-dependent code.
Revision D49587 was moved to bug 1589327. Setting attachment 9101825 [details] to obsolete.
|
|
||
Comment 12•2 months ago
|
||
Comment on attachment 9101826 [details]
Bug 1582218 - Remove more dead code conditional on jni::IsFennec() or similar.
Revision D49588 was moved to bug 1589327. Setting attachment 9101826 [details] to obsolete.
|
|
||
Comment 13•2 months ago
|
||
Comment on attachment 9101827 [details]
Bug 1582218 - Remove GeckoViewNavigationChild.
Revision D49589 was moved to bug 1589327. Setting attachment 9101827 [details] to obsolete.
|
|
||
Updated•Last month
|
|
||
Comment 14•Last month
|
||
Sorry Chris but I updated the alias to make it more explicit ;)
Description
•