csadilek
csadilek
Closing due to inactivity to code review. If there's interest please feel free to discuss and re-open.
Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1795412 Change performed by the [Move to Bugzilla add-on](https://addons.mozilla.org/en-US/firefox/addon/move-to-bugzilla/).
@jonalmeida What I was hoping re: interrupt and cancellation is that we can use `WorkManager` and the `setRequiresDeviceIdle(true)` (and perhaps also `setRequiresBatteryNotLow(true)` constraint(s) to kick off the `run_maintenance` call. However,...
We could land telemetry to record performance of the new call, and potentially decide that we don't need to interrupt. I would expect to see outliers though with very slow...
OK, that's good to know, thanks! I think we can start with your proposal. We have recently exposed the `interrupt` call of the `WriteConnection` via the new`cancelWrites` method on the...
Thought about two more issues worth considering: - I think we'd want to use a dedicated writer for this though, so we don't interrupt/cancel the wrong operation by mistake when...
> The call only removes 6 visits each time it is called, so doing it once per day seems unlikely to effectively keep a cap on the number of visits....
Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1802038 Change performed by the [Move to Bugzilla add-on](https://addons.mozilla.org/en-US/firefox/addon/move-to-bugzilla/).
Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1802028 Change performed by the [Move to Bugzilla add-on](https://addons.mozilla.org/en-US/firefox/addon/move-to-bugzilla/).
Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1801982 Change performed by the [Move to Bugzilla add-on](https://addons.mozilla.org/en-US/firefox/addon/move-to-bugzilla/).