1606736 - Remove redundant IsValid() check in SystemIsBeingShutDown()

Status: RESOLVED FIXED

(Core :: XPConnect, task, P3)

Description

[Andrew McCreight [:mccr8]]

XPCWrappedNativeScope::SystemIsBeingShutDown() does this:
if (wrapper->IsValid()) {
wrapper->SystemIsBeingShutDown();
}

...but XPCWrappedNative::SystemIsBeingShutDown() just returns immediately if the wrapper isn't valid, so this check is not needed. (I'm clearing out my old git branches in preparation for a new laptop...)

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1606736 - Remove redundant IsValid() check in SystemIsBeingShutDown().

PCWrappedNative::SystemIsBeingShutDown() returns immediately if the
XPCWN isn't valid, so this check is not needed.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e33851f3f935
Remove redundant IsValid() check in SystemIsBeingShutDown(). r=bzbarsky

Comment 3

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/e33851f3f935