Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate the feasibility of newer Mozilla Android components for sessions #1121

Open
HollowMan6 opened this issue Nov 27, 2023 · 0 comments
Labels
bug Something isn't working gecko Issues related to the gecko backend

Comments

@HollowMan6
Copy link
Collaborator

Configuration

Wolvic version: main (gecko)
Wolvic build ID: main

Hardware: Any

Related issues

Current Behavior

Chromium doesn't have these similar issues, and those are mostly not reproducible in Firefox for Android

Expected Behavior

All those issues should be resolved.

Possible Solution

Nothing weird can be spotted when debugging our codebase. So those can be caused because we handle the gecko session differently than Fenix (Firefox for Android). We started to notice the difference between Wolvic and Fenix when working on the find-in-page feature #929 (comment), as Fenix heavily depends on browserStore and EngineView, while we use the lower level API to interact with the session. It can be that, by the time FxR was born the only available API was the GeckoView interface. Then Mozilla built some other components on top.

So later, if we find those new Mozilla Android components still compatible to integrate with another backend (i.e Chromium), we can consider refactoring our codebase and moving everything there. This can be a root fix as I guess Gecko may not expect people to use these lower-level APIs now, so they can be very buggy (e.g. caches didn't get cleared correctly when reloading / all media sessions' sound gets muted when we close a tab) if we continue using them.

Context

#1100

Error Logs and Stack Traces

@HollowMan6 HollowMan6 added bug Something isn't working gecko Issues related to the gecko backend labels Nov 27, 2023
@HollowMan6 HollowMan6 self-assigned this Nov 27, 2023
@HollowMan6 HollowMan6 removed their assignment Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gecko Issues related to the gecko backend
Projects
None yet
Development

No branches or pull requests

1 participant