#1053 Drop support for Android and remove Fennec

Open
opened 1 year ago by wolfbeast · 6 comments
wolfbeast commented 1 year ago (Migrated from github.com)

There are no plans whatsoever to build a Fennec client from UXP or any other app for that matter that targets Android. Any potential client would also only be interesting for users if it was a client that could sync in a Weave fashion, which is currently also non-existent in the /mobile client in the tree as it was ripped out in favor of FxA.

Doing this will allow us to greatly simplify code throughout the tree; any future Android client should be built on native Android engines for the moving target of Google “Standards” anyway to be useful, and would best be built from the ground up, and be a native Android developed app.

There are no plans whatsoever to build a Fennec client from UXP or any other app for that matter that targets Android. Any potential client would also only be interesting for users if it was a client that could sync in a Weave fashion, which is currently also non-existent in the `/mobile` client in the tree as it was ripped out in favor of FxA. Doing this will allow us to greatly simplify code throughout the tree; any future Android client should be built on native Android engines for the moving target of Google "Standards" anyway to be useful, and would best be built from the ground up, and be a native Android developed app. - [x] Part 1: Remove Fennec. (Sorry little guy, but you have to find your own home.) - [x] 1a: Remove `mobile/` - [x] 1b: Remove `MOZ_FENNEC` - [x] 1c: Remove references and cases of `mobile/android` in the build system - [x] Part 2a: Remove Android support in our layout engine - [x] Part 2b: Clean up layout reftests for the many special cases/skips/fuzzies for Android - [ ] Part 3: Remove Android support from the renderer (gfx) - [x] 3a: Remove conditional defines and similar logic - [x] 3b: Remove AndroidSurfaceTexture (tree-wide) - [ ] 3c: Remove EGL renderer extensions support for Android (optional) - [ ] Remove Android only libs - [x] Remove Android support in our renderer/widgets - [x] Remove Android support in XPCOM - [ ] Remove Android support in the rest of our tree - [ ] Remove Android support in the build system
mattatobin commented 1 year ago (Migrated from github.com)
Owner

Just in case anyone asks.. This shall not remove ARM architecture support.

Just in case anyone asks.. This shall not remove ARM architecture support.
JustOff commented 11 months ago (Migrated from github.com)
Owner

There are some leftovers in UserAgentUpdates.jsm that were missed during removal of android support from netwerk. I noticed this while working on improving SSUAO, and now I’m going to submit a PR to remove them.

There are some leftovers in `UserAgentUpdates.jsm` that were missed during [removal of android support from netwerk](https://github.com/MoonchildProductions/UXP/commit/bdad2357a1f0d2aca7f1106f3195d1fd929dc1d1#diff-888228427807bd9ad3e953ef1df43f43). I noticed this while working on improving SSUAO, and now I'm going to submit a PR to remove them.
mattatobin commented 11 months ago (Migrated from github.com)
Owner

Such as?

Such as?
JustOff commented 11 months ago (Migrated from github.com)
Owner

These are function readChannel(url) and lazy module getters for NetUtil.jsm and Promise.jsm, see 78eb87174f.

These are `function readChannel(url)` and lazy module getters for `NetUtil.jsm` and `Promise.jsm`, see https://github.com/mozilla/gecko-dev/commit/78eb87174f28b941e2fd6c098e9e4d50c206e377.
mattatobin commented 11 months ago (Migrated from github.com)
Owner

Oh ok.. Go for it then. 👍

Oh ok.. Go for it then. :+1:
Moonchild added the
Good first issue
label 2 months ago
Moonchild commented 2 months ago
Owner

FTR, I’m just doing this between the holidays because nobody else has. If you feel like axing this drop a note and i’ll let you have at it ;-)

FTR, I'm just doing this between the holidays because nobody else has. If you feel like axing this drop a note and i'll let you have at it ;-)
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.