Formatting MessageHandler

[imjacobclark]

Second contribution 👋!

No functional changes here, I simplified the check if we're a cordovaPlugin, javascriptError or anything else.

I also extracted the predicates that checks if the types (null, cordova, js error) to help make the postMessage a little easier to read.

Replaced new Runnable with a Lambda and the // Only eval the JS code if this is a valid callback id comment with a clear variable name.

Moved both public methods to the top of the file and the private methods at the bottom.

[imhoffd]

@imjacobclark Would you mind resolving the conflicts and updating this PR?

[imjacobclark]

@imjacobclark Would you mind resolving the conflicts and updating this PR?

@dwieeb yep, will fix this up tomorrow, thanks for taking a look!

[imjacobclark]

@dwieeb conflicts sorted!

[carlpoole]

Looks good! I just had a question about one of the branches of the if statement

[imjacobclark]

@carlpoole thanks, fixed the comment! What is the process to getting this merged?

[imhoffd]

We just merge it. 😉

Thanks for your contribution, @imjacobclark!