Open Bug 1915362 Opened 2 months ago Updated 11 days ago

Enable partial history collection for better performance on history state updates

Categories

(GeckoView :: General, enhancement, P1)

All
Android
enhancement

Tracking

(Not tracked)

People

(Reporter: kaya, Assigned: kaya)

References

Details

Attachments

(1 file)

Currently on Android, we are not exercising partial history collection and we do a full collection all the time. This is not optimal, perf-wise.
While moving on with SHIP on Android, we can consider enabling the partial history collection as well.

Linking a crash ticket as well, which was caused by the lack of partial history collection. The current SHIP code behaves as if there's partial history change updates on the client end (GeckoSession.SessionState cache), but indeed, there is not. That's why when SHIP is enabled, the history items are fully overriden when it was supposed to be a partial update which reduces the history items size and keeps the index same, and as a result ends up with an IndexOutOfBounds exception.
No-SHIP codepath always collects the history fully, that's why it is not an issue there.

Assignee: nobody → kkaya
Priority: -- → P1
Attachment #9422597 - Attachment description: WIP: Bug 1915362 - Enable partial history collection for better performance on history state updates for SHIP on Android → WIP: Bug 1915362 - Enable partial history collection for better performance on history state updates for SHIP on Android.
Summary: Enable partial history collection for better performance on history state updates on SHIP → Enable partial history collection for better performance on history state updates
Attachment #9422597 - Attachment description: WIP: Bug 1915362 - Enable partial history collection for better performance on history state updates for SHIP on Android. → Bug 1915362 - Enable partial history collection for better performance on history state updates for SHIP on Android.
Attachment #9422597 - Attachment description: Bug 1915362 - Enable partial history collection for better performance on history state updates for SHIP on Android. → Bug 1915362 - Enable partial history collection for better performance on history state updates on Android.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: