Closed Bug 1460525 Opened 6 years ago Closed 6 years ago

Behavior regarding errors in RemoteSettings event handlers

Categories

(Firefox :: Remote Settings Client, enhancement)

57 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 62
Tracking Status
firefox62 --- fixed

People

(Reporter: leplatrem, Assigned: leplatrem)

Details

Attachments

(1 file)

Currently if one event handler fails for a particular client, the others are not executed.

This can seem odd, are we sure that we want this?

Currently in our use-cases, only one event handler is registered :)
Comment on attachment 8981150 [details]
Bug 1460525 - Execute every Remote Settings event listeners when one fails

https://reviewboard.mozilla.org/r/247248/#review253594

Looks good to me.
Attachment #8981150 - Flags: review?(mgoodwin) → review+
Assignee: nobody → mathieu
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/09582e9a6593
Execute every Remote Settings event listeners when one fails r=mgoodwin
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/09582e9a6593
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: