Closed Bug 508471 Opened 12 years ago Closed 12 years ago
Password Manager should use set
Property instead of set Property As Interface for non-interface objects
The password manager currently assumes that if it calls nsIWritablePropertyBag2.setPropertyAsInterface("property", non_interface_object) and then it calls nsIPropertyBag.getProperty("property") the result will be the original object. Before the fix to bug 505988, that was the case; however, we now need to change that. Semantically, this assumption isn't correct, and there's even a semantically correct workaround that should make everybody happy (even with the fix in that bug): using nsIWritablePropertyBag.setProperty("property", blah) will do the Right Thing, even after the fix in the other bug.
Assignee: nobody → dolske
Attachment #392651 - Flags: review?
No additional tests needed, this is already extensively covered by test_basic_form_observer_*.html. Which passes with this patch, of course. Consumers of this interface will continue to work as before.
Attachment #392651 - Flags: review? → review+
(This blocks a P1 1.9.2 blocker)
Flags: blocking1.9.2? → blocking1.9.2+
Priority: -- → P1
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Mass change: adding fixed1.9.2 keyword (This bug was identified as a mozilla1.9.2 blocker which was fixed before the mozilla-1.9.2 repository was branched (August 13th, 2009) as per this query: http://is.gd/2ydcb - if this bug is not actually fixed on mozilla1.9.2, please remove the keyword. Apologies for the bugspam)
Justin, what about the remaining calls to setPropertyAsInterface for username, password, and selectedLogin. Don't they require the change too?
No, those are actually interfaces.
Thanks Blake. So marking as verified fixed on 1.9.2 based on check-in and no breaking tests.
Status: RESOLVED → VERIFIED
OS: Linux → All
Hardware: x86 → All
Version: unspecified → 1.9.2 Branch
Verified fixed on 1.9.1 based on check-in and no orangeness.
You need to log in before you can comment on or make changes to this bug.