Port 1574812 - Don't try...catch routeMessageToTarget, only the sendAsyncMessage. r=Mardak
Port 1574812 - Don't try...catch routeMessageToTarget, only the sendAsyncMessage. r=Mardak

As far as I can tell the try...catch was originally added around various sendAsyncMessage calls but then when some logic got moved to a helper, the try...catch didn't move along with it. This try...catch made it very annoying to debug an issue with bug 1570372 because it swallowed errors that were getting thrown. It doesn't seem like a good idea to swallow all errors for large pieces of code.

Differential Revision: https://phabricator.services.mozilla.com/D43863

Signed-off-by: Ed Lee <538d125fdeb565dcd3206241dcc3af47abb27687@mozilla.com>

mozilla/activity-streams,mozilla/activity-streams,mozilla/activity-stream,mozilla/activity-stream,mozilla/activity-stream,mozilla/activity-stream