Open Bug 1793183 Opened 3 years ago Updated 1 year ago

TabCollection(s) in BrowserState

Categories

(Firefox for Android :: Collections, defect)

All
Android
defect

Tracking

()

People

(Reporter: csadilek, Unassigned)

Details

From github: https://github.com/mozilla-mobile/android-components/issues/11088.

This issue is about investigating whether we could have TabCollections inside BrowserState.

Currently the app accesses TabCollectionStorage directly.

However for new code we start to follow this pattern:

  • All state is in BrowserState
  • A Middleware is responsible for interacting with the storage
  • The Middleware loads the initial state
  • The Middleware persists state changes in the storage

Advantages of this pattern are:

  • It's easy to observe the state and related changes
  • Other code/components can interact with the state too and react to specific actions from a middleware (Adding tabs to collections, Restoring tabs, ...)

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

The severity field is not set for this bug.
:cpeterson, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(cpeterson)
Severity: -- → S3
Flags: needinfo?(cpeterson)
Component: General → Collections
You need to log in before you can comment on or make changes to this bug.