1522464 - Enhance the OriginParser for obsolete origins

Status: NEW

(Core :: Storage: Quota Manager, enhancement, P2)

Description

[Tom Tung [:tt, :ttung]]

See:
https://bugzilla.mozilla.org/show_bug.cgi?id=1423917#c73
https://bugzilla.mozilla.org/show_bug.cgi?id=1423917#c88

Before bug 1423917, obsolete origins were supposed to generate Spec string after parsing. P3 there make this be not guaranteed anymore. Although, the current code won't touch spec if it's an obsolete origin
The possible solutions are:

• Parse obsolete origins till the end so that we would get the spec for the given directory.
• Have a wrapper to protect the spec and have an assertion to avoid from accessing if the caller gets these kinds of information when it's an obsolete origin.
• Use mfbt/Variant.h or similar stuff to ensure spec is not available when it's an obsolete origin.

Note that: it's actually only blocked by the P3 in the bug 1423917

Comment 1

[Jan Varga [:janv]]

Maybe we can get rid of OriginParser in the end.