aw: Try to turn crash into java exception

aw: Try to turn crash into java exception

Since I can't work out why things are null in native code anymore.

BUG=618807
Committed: https://crrev.com/c731f40f3bfbcdd77bab19a0a45cd2f23e86bd6e
Cr-Commit-Position: refs/heads/master@{#400690}

[boliu, 2016-06-16 18:42:56]

boliu@chromium.org changed reviewers:
+ torne@chromium.org

[boliu, 2016-06-16 18:42:57]

thoughts here?

maybe get a second pair of eyes to check that all production callsites to
loadUrl checks destroy first

and if you can't figure out what's going on here like me, then since we have
java crashes now, maybe this might help?..

[boliu, 2016-06-17 15:08:14]

On 2016/06/16 18:42:57, boliu wrote:
> thoughts here?
> 
> maybe get a second pair of eyes to check that all production callsites to
> loadUrl checks destroy first
> 
> and if you can't figure out what's going on here like me, then since we have
> java crashes now, maybe this might help?..

Ping. How about this one?

[Torne, 2016-06-20 14:15:05]

lgtm

[boliu, 2016-06-20 15:09:45]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2016-06-20 15:09:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2076633002/1

[commit-bot: I haz the power, 2016-06-20 15:55:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-20 15:58:30]

Description was changed from

==========
aw: Try to turn crash into java exception

Since I can't work out why things are null in native code anymore.

BUG=618807
==========

to

==========
aw: Try to turn crash into java exception

Since I can't work out why things are null in native code anymore.

BUG=618807

Committed: https://crrev.com/c731f40f3bfbcdd77bab19a0a45cd2f23e86bd6e
Cr-Commit-Position: refs/heads/master@{#400690}
==========

[commit-bot: I haz the power, 2016-06-20 15:58:31]

Patchset 1 (id:??) landed as
https://crrev.com/c731f40f3bfbcdd77bab19a0a45cd2f23e86bd6e
Cr-Commit-Position: refs/heads/master@{#400690}

[boliu, 2016-06-27 03:01:02]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2099133002/ by boliu@chromium.org.

The reason for reverting is: Native crash still happens, which suggests
mNativeAwContents being null is not the issue..