Closed Bug 255142 Opened 21 years ago Closed 21 years ago

The radio groups "Cookie Acceptance Policy" and "Cookie Lifetime Policy" are not lockable

Categories

(Core :: Networking: Cookies, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: eagle.lu, Unassigned)

Details

Attachments

(1 file)

Lock the preferences "network.cookie.cookieBehavior" and "network.cookie.lifetimePolicy" in autoconfig.jsc. But the corresponding radio groups in Privacy & Security-->Cookies are not disabled.
Summary: Privacy And The radio groups "Cookie Acceptance Policy" and "Cookie Lifetime Policy" are not lockable → The radio groups "Cookie Acceptance Policy" and "Cookie Lifetime Policy" are not lockable
Attached patch patch v0Splinter Review
Comment on attachment 155780 [details] [diff] [review] patch v0 can you give r? Thanks
Attachment #155780 - Flags: review?(neil.parkwaycc.co.uk)
this belongs in the cookie component. mconnor, can you please take a look at this few-liner?
Component: Preferences → Networking: Cookies
Boying, I tested both of these and they work fine on a recent trunk build. We should only need to do manual locking if we're also doing manual enabling/disabling of the element. The prefwindow API takes care of this itself.
Mike, Would you please tell me the detailed steps you test. Thanks
I created a mozilla.cfg with the appropriate lockPref entry, added the appropriate pref as follows // silly comment on the first line lockPref("network.cookie.cookieBehavior", 2); I then added the following pref to all.js: pref("general.config.filename", "mozilla.cfg"); also, so as to not require constantly converting the file, I turned off byteshifting by setting general.config.obscure_value to 0. It works fine, based on that. Really, the mechanics of how to lock the pref are irrelevant, as long as about:config shows the pref as locked. From there, the prefwindow API will lock the UI unless we futz with disabling/enabling it ourself. The bits at http://lxr.mozilla.org/mozilla/source/xpfe/components/prefwindow/resources/content/nsPrefWindow.js#388 are what I'm talking about.
Attachment #155780 - Flags: review?(neil.parkwaycc.co.uk) → review-
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: