1966240 - Refactor PlacesQuery.sys.mjs to have different types of CachedHistory

Status: RESOLVED FIXED

(Toolkit :: Places, task, P3)

Description

[James Teow [:jteow]]

CachedHistory is three separate types and Typescript will enforce checking that any access to a variable does proper type checking before attempting to use method associated with it. To make this easier, we should either use one data type or split them into separate ones.

Comment 1

[James Teow [:jteow]]

Attachment: Bug 1966240 - Refactor PlacesQuery cache management logic into a separate class - r?#places-reviewers

Extracted cache logic from PlacesQuery into separate classes to have
clear separation of concerns and allow future specialization of
sorting strategies.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/f0f1d5f5e7d1
https://hg.mozilla.org/integration/autoland/rev/566436e0445b
Refactor PlacesQuery cache management logic into a separate class - r=places-reviewers,Standard8

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/566436e0445b