Closed Bug 1722668 Opened 4 months ago Closed 3 months ago

Speed up temporary storage initialization by removing old origin directories

Categories

(Core :: Storage: Quota Manager, task, P1)

task

Tracking

()

RESOLVED FIXED
94 Branch
Tracking Status
firefox94 --- fixed

People

(Reporter: janv, Assigned: janv)

Details

Attachments

(5 files)

Some users have a lot of free space on disk so origins are never evicted which leads to large set of directories and files after some time (years). This can be a problem on slower hardware, so we should reduce number of origin directories if the initialization takes longer than X seconds. Origins which were used in last 13 months shouldn't be removed.

Attachment #9236966 - Attachment description: Bug 1722668 - Speed up temporary storage initialization by removing old origin directories; r=#dom-storage → Bug 1722668 - Speed up temporary storage initialization by archiving unaccessed origin directories; r=#dom-storage
Keywords: leave-open
Pushed by jvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a7179b67eb1a
Implement a workaround for the case when GetLastModifiedTime returns INT64_MIN; r=jari
https://hg.mozilla.org/integration/autoland/rev/c9f100cb3f86
Expose nsFileStream via the component manager; r=necko-reviewers,dragana,jari
https://hg.mozilla.org/integration/autoland/rev/db05822c64ff
Speed up temporary storage initialization by archiving unaccessed origin directories; r=jari

Backed out for failures on test_unsetLastAccessTime.js

[task 2021-09-04T21:08:20.666Z] 21:08:20     INFO -  TEST-START | dom/quota/test/xpcshell/test_unsetLastAccessTime.js
[task 2021-09-04T21:08:20.937Z] 21:08:20  WARNING -  TEST-UNEXPECTED-FAIL | dom/quota/test/xpcshell/test_unsetLastAccessTime.js | xpcshell return code: 0
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  TEST-INFO took 270ms
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  >>>>>>>
[task 2021-09-04T21:08:20.937Z] 21:08:20     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2021-09-04T21:08:20.938Z] 21:08:20     INFO -  TEST-PASS | dom/quota/test/xpcshell/test_unsetLastAccessTime.js | undefined assertion name - There should be a testSteps function - true == true
Flags: needinfo?(jvarga)
Pushed by jvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/48e005ee27b7
Implement a workaround for the case when GetLastModifiedTime returns INT64_MIN; r=jari
https://hg.mozilla.org/integration/autoland/rev/126032a9c310
Expose nsFileStream via the component manager; r=necko-reviewers,dragana,jari
https://hg.mozilla.org/integration/autoland/rev/144d07c1d155
Speed up temporary storage initialization by archiving unaccessed origin directories; r=jari
Flags: needinfo?(jvarga)
Keywords: leave-open
Pushed by jvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af59d0fac304
Implement nsIQuotaManagerService::getFullOriginMetadata; r=dom-storage-reviewers,jari
https://hg.mozilla.org/integration/autoland/rev/0d3c7f057415
Add test_unsetLastAccessTime.js for the case when quota information is cached; r=dom-storage-reviewers,jari
You need to log in before you can comment on or make changes to this bug.