Closed
Bug 1457170
Opened 7 years ago
Closed 6 years ago
Consider to remove nsICookieService::ACCEPT_FOR_N_DAYS
Categories
(Core :: Networking: Cookies, enhancement, P3)
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: baku, Assigned: baku)
References
Details
(Whiteboard: [necko-triaged][killthem])
Attachments
(1 file)
(deleted),
patch
|
valentin
:
review+
|
Details | Diff | Splinter Review |
This feature is not exposed and it can be removed.
We just support ACCEPT_NORMALLY and ACCEPT_SESSION via about:preferences.
Updated•7 years ago
|
Priority: -- → P3
Whiteboard: [necko-triaged]
Updated•7 years ago
|
OS: Unspecified → All
Hardware: Unspecified → All
See Also: → https://bugzilla.mozilla.org/show_bug.cgi?id=232744,
https://bugzilla.mozilla.org/show_bug.cgi?id=1231319
Whiteboard: [necko-triaged] → [necko-triaged][killthem]
Assignee | ||
Comment 1•6 years ago
|
||
Assignee: nobody → amarchesini
Attachment #8996619 -
Flags: review?(valentin.gosu)
Comment 2•6 years ago
|
||
Comment on attachment 8996619 [details] [diff] [review]
accept_for_n_days.patch
Review of attachment 8996619 [details] [diff] [review]:
-----------------------------------------------------------------
This really sounds like a good feature in theory, but since it's not that easy to find and you can easily rebuild the functionality in webextensions, this means it should go away.
Attachment #8996619 -
Flags: review?(valentin.gosu) → review+
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b6bb656084bc
Get rid of nsICookieService::ACCEPT_FOR_N_DAYS, r=valentin
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•