Closed
Bug 1260399
Opened 8 years ago
Closed 6 years ago
Revert bug 1259169
Categories
(Core :: DOM: Core & HTML, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: baku, Assigned: baku)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug, Whiteboard: btpp-active)
Attachments
(5 files)
3.25 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
51.46 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
7.44 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
28.16 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
3.81 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Updated•8 years ago
|
Component: Networking: Cookies → DOM
Comment 1•8 years ago
|
||
We branch on Monday.
Assignee: nobody → amarchesini
Flags: needinfo?(amarchesini)
Whiteboard: btpp-active
Assignee | ||
Comment 2•8 years ago
|
||
I don't think we are going to do it... We will break too many addons doing it. We should wait a bit more.
Flags: needinfo?(amarchesini)
Blocks: post-57-api-changes
Updated•7 years ago
|
Summary: Revert bug 1259169 in FF47-FF48 → Revert bug 1259169
Updated•7 years ago
|
Keywords: good-first-bug
Assignee | ||
Updated•7 years ago
|
Assignee: amarchesini → nobody
Updated•6 years ago
|
Priority: -- → P3
Assignee | ||
Comment 3•6 years ago
|
||
Assignee: nobody → amarchesini
Attachment #9010920 -
Flags: review?(ehsan)
Assignee | ||
Comment 4•6 years ago
|
||
Attachment #9010921 -
Flags: review?(ehsan)
Assignee | ||
Comment 5•6 years ago
|
||
Attachment #9010922 -
Flags: review?(ehsan)
Assignee | ||
Comment 6•6 years ago
|
||
Here I changed the logic: instead of passing a cookie and considering just its host/path/name, I prefer to have a method with 3 separate fields.
Attachment #9010923 -
Flags: review?(ehsan)
Assignee | ||
Comment 7•6 years ago
|
||
Attachment #9010924 -
Flags: review?(ehsan)
Updated•6 years ago
|
Attachment #9010920 -
Flags: review?(ehsan) → review+
Updated•6 years ago
|
Attachment #9010921 -
Flags: review?(ehsan) → review+
Updated•6 years ago
|
Attachment #9010922 -
Flags: review?(ehsan) → review+
Updated•6 years ago
|
Attachment #9010923 -
Flags: review?(ehsan) → review+
Updated•6 years ago
|
Attachment #9010924 -
Flags: review?(ehsan) → review+
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/75c91b472821 Remove deprecated messages in nsICookieManager methods - part 0 - nsICookieManager.remove() should not have optional parameters, r=ehsan https://hg.mozilla.org/integration/mozilla-inbound/rev/ea67de243b02 Remove deprecated messages in nsICookieManager methods - part 1 - nsICookieManager.add(), r=ehsan https://hg.mozilla.org/integration/mozilla-inbound/rev/35fb4a8d10f7 Remove deprecated messages in nsICookieManager methods - part 2 - nsICookieManager.getCookiesFromHost(), r=ehsan https://hg.mozilla.org/integration/mozilla-inbound/rev/761834f331a7 Remove deprecated messages in nsICookieManager methods - part 3 - nsICookieManager.cookieExists, r=ehsan https://hg.mozilla.org/integration/mozilla-inbound/rev/4c1fb6fac9c4 Remove deprecated messages in nsICookieManager methods - part 4 - remove the deprecated message, r=ehsan
Comment 9•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/75c91b472821 https://hg.mozilla.org/mozilla-central/rev/ea67de243b02 https://hg.mozilla.org/mozilla-central/rev/35fb4a8d10f7 https://hg.mozilla.org/mozilla-central/rev/761834f331a7 https://hg.mozilla.org/mozilla-central/rev/4c1fb6fac9c4
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•