Closed Bug 223617 Opened 22 years ago Closed 22 years ago

[cookies] remove support for network.cookies.strictDomains pref

Categories

(Core :: Networking: Cookies, defect)

Product:
Core
Component:
Networking: Cookies
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: dwitte, Assigned: dwitte)

Details

Attachments

(1 file)

vamoose
7.59 KB, patch
darin.moz
: review+
darin.moz
: superreview+
Details | Diff | Splinter Review
i don't think having this pref makes sense anymore... it's been around since the 1.1 days of nsCookies.cpp, and so is _probably_ just something we can do away with. patch coming up; there's a comment in there that explains what the pref is about...
OS: Windows 3.1 → All
Attached patch vamoose — — Splinter Review
i'm ok with this since it is not enabled by default... and, i presume that IE is also loose with this check.
Comment on attachment 134081 [details] [diff] [review] vamoose mvl whipped up a testcase for IE and confirmed it's loose with the check. thanks mvl! so, looks like this one is ready for review :)
Attachment #134081 - Flags: superreview?(darin)
Attachment #134081 - Flags: review?(darin)
Comment on attachment 134081 [details] [diff] [review] vamoose Please add a comment stating we don't do this check anymore. This does violate the spec...
sure, will do.
Comment on attachment 134081 [details] [diff] [review] vamoose r+sr=darin
Attachment #134081 - Flags: superreview?(darin)
Attachment #134081 - Flags: superreview+
Attachment #134081 - Flags: review?(darin)
Attachment #134081 - Flags: review+
fixed on trunk
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: