Merge noThrow and catchException in WrapForJNI

NEW
Unassigned

Status

()

Firefox for Android
General
3 years ago
3 years ago

People

(Reporter: jchen, Unassigned)

Tracking

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(firefox43 affected)

Details

(Reporter)

Description

3 years ago
noThrow and catchException have mutually exclusive meanings in WrapForJNI. noThrow means the generated wrapper won't catch any exceptions, catchException means the generated wrapper will catch all exceptions. We should merge these two into an enum similar to [1].

For example, we can have an exceptionMode value with choices ABORT, IGNORE (noThrow), and NSRESULT (catchException).

[1] mxr.mozilla.org/mozilla-central/source/widget/android/jni/Refs.h?rev=7a19194812eb#41
You need to log in before you can comment on or make changes to this bug.