1844518 - GeckoView Implement a GeckoServiceTranslationProcess

Status: RESOLVED WORKSFORME

(GeckoView :: General, task, P2)

Description

[Jonathan Almeida [:jonalmeida]]

We probably need to have a separate process where we load the translation engine and the pair of language models into it.

Requires GeckoView API design.

Comment 1

[Olivia Hall [:olivia]]

Summarizing some additional background for this bug:

• Translations currently uses a translations engine per-process, which can cause bigger memory footprints.

  • For Android, this could make the Android tab reloading issue happen even more with translations in use.
  • More data on how memory looks with translations on Android would be beneficial (bug 1815340 and bug 1826718)
  • The translations team is looking at moving translations into a dedicated process in bug 1815339

• For a dedicated process, then this bug 1844518 is the next step for Android after bug 1815339

  • We can add the new translation process GeckoProcessType.java in the AndroidManifest.xml#47-88
  • We will need to determine what behavior we should manage for the new process. We need to ensure it properly unloads and/or is selected by Android as one of the first processes to de-prioritize and also how to gracefully handle the loss of the process.
  • gpu, utility, and IPDL test support are examples of setting up processes this way

Comment 2

[Olivia Hall [:olivia]]

Resolving as works for me because bug 1815339 will move translations into a dedicated process and will also manage the process lifecycle. GeckoView managing this process is not needed at this time, but it might be something to consider for future optimizations.