Closed Bug 994742 Opened 10 years ago Closed 10 years ago

Check if calendar.setProperty('xyz', null); will reset the pref or if it fails

Categories

(Calendar :: Internal Components, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Preferences.jsm might think that null is an object and iterate through it. Fix this or verify it works.
Attached patch Fix - v1 — — Splinter Review
Attachment #8404953 - Flags: review?(mohit.kanwal)
Comment on attachment 8404953 [details] [diff] [review]
Fix - v1

Review of attachment 8404953 [details] [diff] [review]:
-----------------------------------------------------------------

Looks Good To me!
Attachment #8404953 - Flags: review?(mohit.kanwal) → review+
https://hg.mozilla.org/comm-central/rev/1b9edf24461c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 3.3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: