[mq]: weakRefForms
authorMike Connor <mconnor@mozilla.com>
Wed, 04 Aug 2010 10:42:09 -0400
changeset 49593 a6f137a4e2ea48b9eeff06f8364369074353b9bb
parent 49592 c1c2c2a17739ec3aa2bf2808e506b22af7a6be0a
child 49594 0915690b9080dfb7d5f1e7a7fdceaf8cdf9297a0
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
[mq]: weakRefForms
services/sync/modules/engines/forms.js
--- a/services/sync/modules/engines/forms.js
+++ b/services/sync/modules/engines/forms.js
@@ -202,24 +202,25 @@ FormStore.prototype = {
 
 function FormTracker(name) {
   Tracker.call(this, name);
   Svc.Obs.add("form-notifier", this);
 
   // nsHTMLFormElement doesn't use the normal observer/observe pattern and looks
   // up nsIFormSubmitObservers to .notify() them so add manually to observers
   Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService).
-    addObserver(this, "earlyformsubmit", false);
+    addObserver(this, "earlyformsubmit", true);
 }
 FormTracker.prototype = {
   __proto__: Tracker.prototype,
 
   QueryInterface: XPCOMUtils.generateQI([
     Ci.nsIFormSubmitObserver,
-    Ci.nsIObserver]),
+    Ci.nsIObserver,
+    Ci.nsISupportsWeakReference]),
 
   trackEntry: function trackEntry(name, value) {
     this.addChangedID(FormWrapper.getGUID(name, value));
     this.score += 10;
   },
 
   observe: function observe(subject, topic, data) {
     let name, value;