Differential D195509

Bug 1866402 - Replace some nsIQuotaManagerService::ResetStoragesForPrincipal calls with nsIQuotaManagerService::ResetStoragesForClient; r=#dom-storage
AcceptedPublic
Actions

Authored by janv on Dec 5 2023, 2:37 AM.

Details

Reviewers
jari
Group Reviewers
dom-storage-reviewers
Bugzilla Bug ID
1866402
Summary

There's now a dedicated nsIQuotaManagerService::ResetStoragesForClient method
for resetting storages by client type. All callers of
nsIQuotaManagerService::ResetStoragesForPrincipal which want to reset storages
for a specific client only can be now converted to call the new method.

Test Plan

https://treeherder.mozilla.org/jobs?repo=try&revision=e40c393eab1f0b3592eb4d9bbd9e4f2f2fa41c4a

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Branch
default
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

janv created this revision.Dec 5 2023, 2:37 AM
Herald added a project: secure-revision. · View Herald TranscriptDec 5 2023, 2:37 AM
Harbormaster completed remote builds in B603759: Diff 795086.Dec 5 2023, 2:37 AM
phab-bot published this revision for review.Dec 5 2023, 2:37 AM
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
janv edited the test plan for this revision. (Show Details)Dec 5 2023, 2:39 AM
janv added a child revision: D195510: Bug 1866402 - Change nsIQuotaManagerService::ResetStoragesForPrincipal to support resetting of entire origins only; r=#dom-storage.Dec 5 2023, 2:50 AM
janv updated this revision to Diff 801036.Dec 17 2023, 8:39 PM
janv edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B608934: Diff 801036.Dec 17 2023, 8:39 PM
janv updated this revision to Diff 806479.Jan 7 2024, 4:45 PM
Harbormaster completed remote builds in B613637: Diff 806479.Jan 7 2024, 4:45 PM
janv updated this revision to Diff 806480.Jan 7 2024, 4:51 PM
Harbormaster completed remote builds in B613638: Diff 806480.Jan 7 2024, 4:51 PM
janv edited the test plan for this revision. (Show Details)Jan 7 2024, 4:54 PM
janv updated this revision to Diff 807471.Jan 10 2024, 9:31 AM
Harbormaster completed remote builds in B614456: Diff 807471.Jan 10 2024, 9:31 AM
janv edited the test plan for this revision. (Show Details)Jan 11 2024, 8:39 AM
janv updated this revision to Diff 808100.Jan 11 2024, 4:21 PM
Harbormaster completed remote builds in B614974: Diff 808100.Jan 11 2024, 4:21 PM
janv edited the test plan for this revision. (Show Details)Jan 11 2024, 4:26 PM
janv updated this revision to Diff 814871.Jan 29 2024, 11:21 AM
Harbormaster completed remote builds in B620907: Diff 814871.Jan 29 2024, 11:21 AM
janv updated this revision to Diff 818954.Feb 6 2024, 3:26 PM
janv edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B624377: Diff 818954.Feb 6 2024, 3:26 PM
janv edited the test plan for this revision. (Show Details)Feb 6 2024, 3:30 PM
jari accepted this revision.Feb 12 2024, 12:34 PM
jari added a project: testing-approved.
This revision is now accepted and ready to land.Feb 12 2024, 12:34 PM
jari mentioned this in D195508: Bug 1866402 - Add a dedicated method for resetting storages of a specific client; r=#dom-storage.Feb 12 2024, 12:35 PM
jari mentioned this in D195507: Bug 1866402 - Rework QuotaManagerService::ResetStoragesForPrincipal to use an async message with an async response; r=#dom-storage.

Revision Contents
Changeset List

PathSize
dom/
cache/
test/
marionette/
2 lines
localstorage/
test/
unit/
8 lines
4 lines
2 lines
2 lines
2 lines
2 lines
2 lines
8 lines
quota/
test/
marionette/
9 lines
xpcshell/
common/
6 lines
tests/
mochitest/
localstorage/
4 lines

Diff 818954

View Options

dom/cache/test/marionette/test_cacheapi_encryption_PBM.py

Loading...
View Options

dom/localstorage/test/unit/head.js

Loading...
View Options

dom/localstorage/test/unit/test_corruptedDatabase.js

Loading...
View Options

dom/localstorage/test/unit/test_eviction.js

Loading...
View Options

dom/localstorage/test/unit/test_stringLength.js

Loading...
View Options

dom/localstorage/test/unit/test_stringLength2.js

Loading...
View Options

dom/localstorage/test/unit/test_unicodeCharacters.js

Loading...
View Options

dom/localstorage/test/unit/test_uri_encoding_edge_cases.js

Loading...
View Options

dom/localstorage/test/unit/test_usage.js

Loading...
View Options

dom/quota/test/marionette/quota_test_case.py

Loading...
View Options

dom/quota/test/xpcshell/common/head.js

Loading...
View Options

dom/tests/mochitest/localstorage/localStorageCommon.js

Loading...
Privacy · Cookies · Legal