Closed Bug 384990 Opened 15 years ago Closed 15 years ago

remove nsICookieServiceInternal

Categories

(Core :: Networking: Cookies, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dwitte, Assigned: dwitte)

Details

Attachments

(1 file)

this interface (and corresponding nsCookieService refactoring) was done to provide dom cookie support (i.e. allowing xul to set cookies). however, only the cookie backend changes were landed before dom storage came along, and the work was obsoleted. in particular, see:
https://bugzilla.mozilla.org/show_bug.cgi?id=295994#c52 and https://bugzilla.mozilla.org/show_bug.cgi?id=295994#c53

i don't see any use for this stuff otherwise, and it's cluttering things, so i'd like to back it out.
Attached patch v1Splinter Review
all code movement, no functional changes whatsoever. i'll remove nsICookieServiceInternal.idl when this goes in.
Assignee: nobody → dwitte
Status: NEW → ASSIGNED
Attachment #268920 - Flags: superreview?(cbiesinger)
Attachment #268920 - Flags: review?(cbiesinger)
Attachment #268920 - Flags: superreview?(cbiesinger)
Attachment #268920 - Flags: superreview+
Attachment #268920 - Flags: review?(cbiesinger)
Attachment #268920 - Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
checked in to trunk.
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.