Closed
Bug 1457170
Opened 6 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)
|
17.03 KB,
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•6 years ago
|
Priority: -- → P3
Whiteboard: [necko-triaged]
|
||
Updated•6 years ago
|
|
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 | ||
https://hg.mozilla.org/mozilla-central/rev/b6bb656084bc
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
•