[Extensions Bindings] Move safeCallbackApply to exception handler

[Extensions Bindings] Move safeCallbackApply to exception handler

Move the safeCallbackApply function to the uncaught exception handler
module, rather than the sendRequest module.
Two main reasons:
- It makes more sense, since the purpose of the function is to run a
  supplied function and notify the exception handler on failure,
- Moving to native bindings, we should be able to avoid using anything
  from the sendRequest module.

BUG=653596
Review-Url: https://codereview.chromium.org/2794043002
Cr-Commit-Position: refs/heads/master@{#461538}
Committed: https://chromium.googlesource.com/chromium/src/+/b026fbdecb915942a1dcff6fd664d664586dc56f

[Devlin, 2017-04-03 17:20:08]

The CQ bit was checked by rdevlin.cronin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-03 17:20:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 18:09:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-03 18:09:27]

Dry run: This issue passed the CQ dry run.

[Devlin, 2017-04-03 20:21:56]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Devlin, 2017-04-03 20:21:57]

Just movin' stuff around (I'm hoping to be able to blacklist send_request.js,
event.js, binding.js, and maybe a few others when we turn on native bindings).

jbroman@ fyi.

[lazyboy, 2017-04-03 20:55:08]

lgtm

[Devlin, 2017-04-03 20:55:48]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2017-04-03 20:56:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 21:09:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491252948298050, "parent_rev":
"151bd31ea040cb745cdc07e59d4b097186ec3ab3", "commit_rev":
"b026fbdecb915942a1dcff6fd664d664586dc56f"}

[commit-bot: I haz the power, 2017-04-03 21:09:57]

Description was changed from

==========
[Extensions Bindings] Move safeCallbackApply to exception handler

Move the safeCallbackApply function to the uncaught exception handler
module, rather than the sendRequest module.
Two main reasons:
- It makes more sense, since the purpose of the function is to run a
  supplied function and notify the exception handler on failure,
- Moving to native bindings, we should be able to avoid using anything
  from the sendRequest module.

BUG=653596
==========

to

==========
[Extensions Bindings] Move safeCallbackApply to exception handler

Move the safeCallbackApply function to the uncaught exception handler
module, rather than the sendRequest module.
Two main reasons:
- It makes more sense, since the purpose of the function is to run a
  supplied function and notify the exception handler on failure,
- Moving to native bindings, we should be able to avoid using anything
  from the sendRequest module.

BUG=653596

Review-Url: https://codereview.chromium.org/2794043002
Cr-Commit-Position: refs/heads/master@{#461538}
Committed:
https://chromium.googlesource.com/chromium/src/+/b026fbdecb915942a1dcff6fd664...
==========

[commit-bot: I haz the power, 2017-04-03 21:09:58]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b026fbdecb915942a1dcff6fd664...