Closed Bug 337231 Opened 18 years ago Closed 17 years ago

Migrate Opera's "Accept Cookies Session Only" preference into network.cookie.lifetimePolicy instead of the unused network.cookie.enableForCurrentSessionOnly

Categories

(Firefox :: Migration, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jason.barnabe, Assigned: dwitte)

References

Details

Attachments

(1 file)

The Opera profile migrator is currently migrating the Opera preference "Accept Cookies Session Only" into the preference network.cookie.enableForCurrentSessionOnly. That preference is unused; it has been replaced by network.cookie.lifetimePolicy.

The migrator should set a lifetimePolicy value of 2 (Accept for session only) when Opera's pref is true and a value of 0 (use supplied value [the default]) when Opera's pref is false.
Blocks: 337232
Was this bug filed against the Firefox migrator, or something else??
Component: Profile: Migration → Migration
Product: Core → Firefox
QA Contact: profile-migration → migration
I didn't realize there was a Firefox-specific migrator...
Attached patch like soSplinter Review
pretty self-explanatory.
Assignee: nobody → dwitte
Status: NEW → ASSIGNED
Attachment #289113 - Flags: review?(gavin.sharp)
Attachment #289113 - Flags: review?(gavin.sharp) → review+
Comment on attachment 289113 [details] [diff] [review]
like so

fixes up a borken pref import.
Attachment #289113 - Flags: approval1.9?
Attachment #289113 - Flags: approval1.9? → approval1.9+
checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: