fix(macos): do not ignore errors on sending HTTP responses

[rhysd]

All errors on sending HTTP responses are ignored on macOS.

wry/src/wkwebview/mod.rs

Line 345 in eafaadb

let _ = response(task, webview_id, url, sent_response);

This PR changes to check the errors. When some error happens, it now causes a panic.

[amrbashir]

I don't think our users would appreciate panics, we should instead just log::error the result

[rhysd]

Hmm, does it mean users can recover the error when it occurs? This is an internal error and users don't know why it happened. And there is no other way to receive a response via custom protocol. For unrecoverable errors, I believe panics are a correct way to handle them.

https://doc.rust-lang.org/beta/book/ch09-03-to-panic-or-not-to-panic.html

[amrbashir]

@lucasfernog what do you think?

[FabianLars]

I don't think the quote from the rust book applies well to gui applications. gui apps should really never crash like this, or at least not from inside a library (from someone else than the app dev), but of course the dev not having a way to react to those errors is quite problematic too, hmm 🤔