condition on the unbind handler, otherwise double complete call, part of figuring out issue #589

resources/static/dialog/controllers/dialog_controller.js

@@ -118,8 +118,12 @@
         self.doCheckAuth();
 
         dom.bindEvent(win, "unload", function() {
-          bid.Storage.setStagedOnBehalfOf("");
-          self.doCancel();
+          // do this only if something else hasn't
+          // declared success
+          if (!self.success) {
+            bid.Storage.setStagedOnBehalfOf("");
+            self.doCancel();
+          }
         });
       },
 
@@ -277,6 +281,7 @@
         // calls window.close, which would trigger the onerror callback if we
         // tried this afterwards.
         self.onerror = null;
+        self.success = true;
         self.onsuccess(assertion);
       },