Closed Bug 636569 Opened 11 years ago Closed 11 years ago

'this.notifyStatus is not a function' when a crash report fails to submit

Categories

(Toolkit :: Crash Reporting, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jdm, Assigned: jdm)

References

Details

Attachments

(1 file)

I've got crash reports failing to submit in Fennec, and I see the subject appear in the error console.  The problem line is here:

>354     let self = this;
>355     function loadHandler() {
>356       if (iframe.contentWindow.location == "about:blank")
>357         return;
>358       iframe.removeEventListener("load", loadHandler, true);
>359       if (!self.submitForm()) {
>360         this.notifyStatus(FAILED);
>361         self.cleanup();
>362       }
>363     }

The |this| looks out of place when everything else is using |self|.
Attachment #514896 - Flags: review?(dtownsend)
Attachment #514896 - Flags: review+
Attachment #514896 - Flags: approval2.0+
Keywords: checkin-needed
Assignee: nobody → josh
http://hg.mozilla.org/mozilla-central/rev/2b4767713e82
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.