1215139 - Handle stale native calls after nsWindow destruction

Status: RESOLVED FIXED

(Core Graveyard :: Widget: Android, defect)

Description

[(inactive) Jim Chen [:jchen] [:darchons]]

I'm noticing some test failures related to native calls arriving from the UI thread after the Gecko nsWindow has been destroyed, when the test is shutting down. We should discard these stale calls instead of trying to complete them, which leads to a null pointer exception.

Comment 1

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Make NativeImpl::DisposeNative a static member (v1)

It's helpful to have a static DisposeNative when disposing of instances
that are still associated with a native object through an null weak
pointer. In that case, we can't call instance methods because the weak
pointer is null, but we can still call static methods.

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Separate out nsWindow closing from disposeNative (v1)

This patch adds a separate close() call to nsWindow, and let the
GeckoView decide whether to make that call or not. This lets us use the
static version of disposeNative. If nsWindow is destroyed in the
meantime, we still want to call disposeNative, which would only be
possible using the static version of disposeNative.

Comment 3

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Discard stale native calls after nsWindow destruction (v1)

In the period after nsWindow is destroyed, but before disposeNative is
called, we should discard any pending native calls. These calls would
result in exceptions anyways because the nsWindow weak pointer is now
null. After disposeNative is called, any native calls will still result
in exceptions, because in that case, it's Java code's responsibility to
ensure that calls are not made after disposeNative is called.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/adc70fa9aa35
https://hg.mozilla.org/integration/mozilla-inbound/rev/3fde465d5226
https://hg.mozilla.org/integration/mozilla-inbound/rev/cc90b297f32f

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/adc70fa9aa35
https://hg.mozilla.org/mozilla-central/rev/3fde465d5226
https://hg.mozilla.org/mozilla-central/rev/cc90b297f32f