Closed Bug 1791343 Opened 2 years ago Closed 2 years ago

Avoid parent query when file entryId is known

Categories

(Core :: DOM: File, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: jjalkanen, Assigned: jjalkanen)

Details

Attachments

(3 files)

File system entries can be queried

  • with entry's name and its parent's entryId, usually from a directory handle when the caller doesn't yet have a handle to the child item, or
  • directly with the entryId when the caller has already obtained a handle and wants to act on the item pointed by the handle.

By providing an overload of DoesDirectory/FileExist functions to cover the second case, a query of the parent entryId can be avoided in the move-related calls, and further down the road, in quota usage related functions.

Pushed by jjalkanen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/68253907138d
Have rootId constant available in file system database manager. r=dom-storage-reviewers,jesup,janv
https://hg.mozilla.org/integration/autoland/rev/328a3402a06d
Return error from FileSystemDatabaseManager on request to list deleted folders. r=dom-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/bf28018e2e2a
Avoid parent queries for known entries. r=dom-storage-reviewers,jesup

Some kind of conflict with D157643 was feared: it had begun landing on the same day.

Flags: needinfo?(jjalkanen)
Pushed by jjalkanen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/773a2b7c52da
Have rootId constant available in file system database manager. r=dom-storage-reviewers,jesup,janv
https://hg.mozilla.org/integration/autoland/rev/482fa41be849
Return error from FileSystemDatabaseManager on request to list deleted folders. r=dom-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/4c7e266ad367
Avoid parent queries for known entries. r=dom-storage-reviewers,jesup
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: