This is based on bug 82805 which autually locks the offline preference through xpcom js approach for User Prefs "mail.server.server1.offline_download" "mail.server.server1.download_bodies_on_get_new_mail" Bug 79305 will provide generic approach to make all preferenc lockable through adding prefstring, severkey, identitykey. After 79305 ( eddyk) being checked in, we need to add the prefstrings for the above preference to am-offline.xul, see attachement 38993 in bug 82805
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.3
I'd like to request a change; at line 109, you have three pref attributes, but no prefstring. It should be an all or nothing use. Can that be fixed in this bug as well?
my area...QA over to me.
QA Contact: nbaca → rvelasco
Summary: Make Account Manager Offline Preference Locable → Make Account Manager Offline Preference Lockable
Eddy, your question will be fixed in another bug Disk Space bug 85335.
Request R & SR, Eddy, Bhuvan, Seth?
looks right. r=bhuvan. Please run test scenarios with prefs locked. It possible you may come across bugs like 86911 (bug 86911), which will be important to know to ensure the correctness of locking feature.
Assignee: dianesun → eddyk
Status: ASSIGNED → NEW
set milestone to 0.9.4
eddyk, do you still need a review on that last patch?
I'm moving TFV to future and putting a dependancy on bug 79305. I'm doing this because the offline/disk panel elements are populated and saved via am-offline.js and not AccountManager.js. Adding the xul attributes in the patch would involve the code in AccountManager and seems a bit dangerous at this point at worst and redundant at best. Once 79305 is implemented, I'll have to come back and resolve any conflicts between the differing methods. From my point of view, it seems possible that most or all of the code in am-offline.js would need to be dropped once 79305 is done.
Depends on: 79305
Target Milestone: mozilla0.9.4 → Future
Hi, Putterman: Any chance we get this one in by 0.9.9? Steven, Jeff - would you provide some inputs on this? Private email will do, too. thx
Assignee: eddyk → putterman
reassigning to racham. I've marked the bug it's dependent on as 1.2. How immportant are these sets of bugs and is there anyone on your team that can help out, Tao?
Assignee: putterman → racham
Keywords: nsbeta1 → nsbeta1+
Moving to 1.0, but it's likely these will be moved out further.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
Keywords: nsbeta1+ → nsbeta1-
Target Milestone: mozilla1.0 → mozilla1.2
this bug is fixed due to bug 144563. fixed on the trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
verified on trunk...see bug 144563 for more details
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.