Closed
Bug 1589626
Opened 6 years ago
Closed 6 years ago
LSNG: Add a check for null quota object
Categories
(Core :: Storage: localStorage & sessionStorage, defect, P1)
Core
Storage: localStorage & sessionStorage
Tracking
()
RESOLVED
FIXED
mozilla71
People
(Reporter: janv, Assigned: janv)
References
(Blocks 1 open bug)
Details
(Keywords: crash)
Crash Data
Attachments
(1 file)
mozilla::dom::quota::QuotaObject::LockedMaybeUpdateSize started showing up in the crash stats. I believe it's related to a mismatch between base domain computed by MozURL and nsIPrincipal. I think we should rather fail datastore preparation instead of crashing if this happens, at least for now.
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Updated•6 years ago
|
Type: defect → task
Summary: LSNG: Check for null quota object → LSNG: Add a check for null quota object
|
|
Assignee | |
Comment 2•6 years ago
|
||
Pushed by jvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c36aa74864ff
LSNG: Add a check for null quota object; r=ttung
|
||
Comment 4•6 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
|
|
Assignee | |
Updated•6 years ago
|
status-firefox67:
--- → disabled
status-firefox68:
--- → disabled
status-firefox69:
--- → disabled
status-firefox70:
--- → affected
status-firefox-esr60:
--- → unaffected
status-firefox-esr68:
--- → disabled
|
||
Updated•6 years ago
|
Type: task → defect
|
||
Comment 5•6 years ago
|
||
In early crash results from 70 release, this is in the top 10 crash signatures.
|
|
||
Comment 6•6 years ago
|
||
Tracking to see what the volume looks like after a bit more time on release.
tracking-firefox70:
--- → +
|
||
Comment 7•6 years ago
|
||
I guess this bug is moot for Fx70 now given bug 1592136.
|
|
||
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•