Closed Bug 744770 Opened 8 years ago Closed 8 years ago

Master Password dummy pref should not be saved

Categories

(Firefox for Android :: General, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 14
Tracking Status
blocking-fennec1.0 --- beta+

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
We use a dummy pref to make the MasterPassword code fit into the current Preference Model, but it's not a real pref and we should not allow it to be saved as a pref.
Attachment #614356 - Flags: review?(wjohnston)
Assignee: nobody → mark.finkle
The key change is adding the | return; | to leave the function early and not save the data.
Attachment #614356 - Flags: review?(wjohnston) → review+
https://hg.mozilla.org/mozilla-central/rev/a2d751d71d64
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
You need to log in before you can comment on or make changes to this bug.