Allow disabling the cookie database in the profile
Categories
(Core :: Networking: Cookies, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox100 | --- | fixed |
People
(Reporter: mossop, Assigned: nalexander)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged][fidedi-ope])
Attachments
(2 files, 2 obsolete files)
The background update agent will use an ephemeral profile, it will be deleted when the agent exits. Even so we would like to reduce the amount of file I/O the agent generates by disabling writing things to the profile where possible.
The cookie service always creates a cookies database in the profile regardless of whether any networking occurs but it also has a private mode that functions without the database so this bug will add a way to always use that mode and not create the database at all when running in the background.
Updated•4 years ago
|
Reporter | ||
Comment 1•4 years ago
|
||
Background tasks run with a temporary profile and we want to reduce disk I/O by
stopping things from writing to the profile where doing so is straightforward.
Since the cookie service already supports in-memory storage for private browsing
we can easily disable the persistent storage when running a background task.
Note that this depends on the patch in https://phabricator.services.mozilla.com/D96481.
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Background tasks run with a temporary profile and we want to reduce disk I/O by
stopping things from writing to the profile where doing so is straightforward.
Since the cookie service already supports in-memory storage for private browsing
we can easily disable the persistent storage when running a background task.
Updated•4 years ago
|
Reporter | ||
Comment 3•4 years ago
|
||
I'm not working on this, Nick is this still needed or can it just be closed?
Assignee | ||
Comment 4•4 years ago
|
||
(In reply to Dave Townsend [:mossop] (Out May 3-14) from comment #3)
I'm not working on this, Nick is this still needed or can it just be closed?
It is still a nice to have; it'll show up more as we get to Bug 1679443.
Assignee | ||
Comment 5•3 years ago
|
||
Firefox background tasks use a new temporary profile directory for
every invocation. We would like to write as little data as possible
into the temporary profile directory in this mode. This patch allows
to make the persistent cookie storage discard cookies just like
private cookie storage.
This approach simply uses that the base CookieStorage
class is close
to sufficient already. There may be a performance impact with more
virtual invocations, but none of these functions seem likely to be
hot.
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 6•3 years ago
|
||
NI to baku to answer https://phabricator.services.mozilla.com/D139910#inline-773464 and suggest a docstring.
Updated•3 years ago
|
Assignee | ||
Comment 7•3 years ago
|
||
Depends on D139910
Comment 9•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/09bb73dac836
https://hg.mozilla.org/mozilla-central/rev/51c22199741b
Description
•