From 1d6b73a5d75572ab6b70cd049032c4a665856e46 Mon Sep 17 00:00:00 2001
From: Dan Mills <thunder@mozilla.com>
Date: Thu, 26 Mar 2009 10:58:54 -0700
Subject: [PATCH] Bug 484848: try #3

---
 services/sync/modules/engines/passwords.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/services/sync/modules/engines/passwords.js b/services/sync/modules/engines/passwords.js
index dbf8eca342f..8bfaee5add5 100644
--- a/services/sync/modules/engines/passwords.js
+++ b/services/sync/modules/engines/passwords.js
@@ -240,14 +240,15 @@ PasswordTracker.prototype = {
 
     switch (aData) {
     case 'modifyLogin':
-      aSubject = aSubject.queryElementAt(1, Ci.nsILoginMetaInfo);
+      aSubject = aSubject.QueryInterface(Ci.nsIArray).
+        queryElementAt(1, Ci.nsILoginMetaInfo);
     case 'addLogin':
-    case 'removeLogin': {
-      let metaInfo = aSubject.QueryInterface(Ci.nsILoginMetaInfo);
+    case 'removeLogin':
+      aSubject.QueryInterface(Ci.nsILoginMetaInfo);
       this._score += 15;
-      this._log.debug(aData + ": " + metaInfo.guid);
-      this.addChangedID(metaInfo.guid);
-    } break;
+      this._log.debug(aData + ": " + aSubject.guid);
+      this.addChangedID(aSubject.guid);
+      break;
     case 'removeAllLogins':
       this._log.debug(aData);
       this._score += 50;