1676411 - Fix usage calculation for file:// (and potentially other) URIs

Status: RESOLVED FIXED

(Core :: Storage: localStorage & sessionStorage, defect, P2)

Comment 1

[Jan Varga [:janv]]

Attachment: Bug 1676411 - Fix usage calculation for file:// (and potentially other) URIs; r=#dom-workers-and-storage-reviewers,asuth

This patch handles the case when GetBaseDomain returns an empty string in
BasePrincipal::GetLocalStorageQuotaKey. The method would return a very generic
scope otherwise, causing the usage calculation to return usage for all origins.
LocalStorageCache::Init falls back to the origin scope for usage calculation
when the domain scope returned by BasePrincipal::GetLocalStorageQuotaKey is
empty or the method fails.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9a9e9411d5e3
Fix usage calculation for file:// (and potentially other) URIs; r=asuth,dom-workers-and-storage-reviewers

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9a9e9411d5e3