976155 - Implement 'schema upgrades' for HomeConfig

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

[Lucas Rocha (:lucasr)]

We'll need to be able to migrate a HomeConfig state to new versions of the schema. At the very least (in the short-term), we should ensure that a saved HomeConfig in Fx29 will not cause crashes in Fx30.

The reason we need migrations for HomeConfig is simple: as we add new features to HomeConfig, we'll have to enforce certain invariants that might not be present in previous versions. For example: we enforce item handler in ViewConfigs in Fx30 but there's no such thing in Fx29. In this case, we have to migrate any Fx29 saved state to be a valid one in Fx30 e.g. include the default item handler value in the configuration.

Comment 1

[Lucas Rocha (:lucasr)]

Ok, tested upgrades from Fx29 to Fx30 in different scenarios with a saved HomeConfig state. Works fine as in Fx29 we only save built-in panels on which we don't do any validation on layouts, views, etc.

So, this is not relevant for Fx30. We should only care about performing schema migrations in HomeConfig if we ever add a feature (and a matching PanelConfig/ViewConfig validation) in Fx31 that is not available in Fx30.

Downgrading the priority for now.

Comment 2

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.