Closed Bug 1595085 Opened 4 years ago Closed 4 years ago

Implement use of preloaded elements with IDBCursor.continuePrimaryKey

Categories

(Core :: Storage: IndexedDB, task, P3)

Product:
Core
Component:
Storage: IndexedDB
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla72
Tracking Flags:
Tracking Status
firefox72 --- fixed

People

(Reporter: sg, Assigned: ssengupta)

Details

Attachments

(1 file)

Bug 1595085 - BackgroundCursorChild::SendContinueInternal discards cached responses conditionally also for ContinuePrimaryKey r=sg
47 bytes, text/x-phabricator-request
Details | Review

Currently, at https://searchfox.org/mozilla-central/rev/524bed6dfbc5ae21c62632d83b7573448b29e0ac/dom/indexedDB/ActorsChild.cpp#3364, the cached cursor entries are invalidated when IDBCursor.continuePrimaryKey is called. This behaviour should be improved. We need to implement the logic from ActorsParent.cpp to skip over the intermediate entries based on the params.get_ContinuePrimaryKeyParams() here.

Priority: -- → P3
Assignee: nobody → ssengupta
Pushed by btara@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7aff50c10a47
BackgroundCursorChild::SendContinueInternal discards cached responses conditionally also for ContinuePrimaryKey r=sg

https://hg.mozilla.org/mozilla-central/rev/7aff50c10a47

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox72: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.