Initial integration of the Suggest Rust component into desktop
Categories
(Firefox :: Address Bar, task, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox120 | --- | fixed |
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
This bugs tracks initial integration of the Suggest Rust component into desktop urlbar.
| Assignee | ||
Comment 1•2 years ago
|
||
This builds on D188681 and adds a new BaseFeature called SuggestBackendRust.
When quickSuggestRustEnabled is true, UrlbarProviderQuickSuggest will use
SuggestBackendRust to fetch remote settings suggestions; otherwise it will use
SuggestBackendRemoteSettings.
The Rust component is already integrated into desktop Firefox (bug 1851256, bug
1851845), and it's exposed to JS via RustSuggest.sys.mjs. Currently it only
supports AMP (sponsored, a.k.a. adM) and Wikipedia (non-sponsored) suggestions,
so that's all that SuggestBackendRust supports too.
It's possible to configure the path of the Sqlite file created by the Rust
component. This path uses suggest.sqlite in the user's profile directory.
This is only the initial integration. I can think of a few follow-ups:
- Handle icons. In this patch, results from
SuggestBackendRustdon't have
icons at all. I have a WIP. - Handle ingestion better. "Ingest" here means Firefox must tell the Rust
component to re-fetch suggestions from remote settings and rebuild its Sqlite
database. Unfortunately the Rust component doesn't keep the data updated by
itself, so we'll need to periodically tell the component to ingest. This patch
performs ingestion every timeSuggestBackendRustis (re)enabled, which is a
good enough start. - Maybe handle tests better. For now I modified the main quick suggest unit
test, test_quicksuggest.js, so it tests both backends. Other tests should
maybe be updated too, I'm not sure yet. - Handle priority sponsored suggestions.
Depends on D188681
Updated•2 years ago
|
Comment 3•2 years ago
|
||
| bugherder | ||
| Assignee | ||
Updated•2 years ago
|
Description
•