1101240 - Expose an interface for bulk-adding history via GlobalHistory

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Data Providers, defect)

Description

[Richard Newman [:rnewman]]

Perf optimization from Bug 949104, related to Bug 949541.

We'd like to be able to use the bulkInsert API to BrowserProvider to insert multiple history URIs in a single transaction.

(Note that it's not just inserting URIs: it's also incrementing visit counts and bumping timestamps. This isn't as simple as a fire-and-forget INSERT.)

Perhaps the easiest way to do this is to expose an insertion URI in BrowserProvider itself that will buffer and flush, so callers don't need to worry about the niceties of timestamps or INSERT OR UPDATE, but still get the benefit of a single SQL transaction.

This still pays the overhead of multiple ContentResolver hits, so a little timing with synthetic tests would be worthwhile -- trigger a bunch of page loads, with some delayed history events like following links, and then see how much time we might be able to save.

Comment 1

[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.