Always install the microdump signal handler on N+ devices.

Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
Committed: https://crrev.com/2c944c9e76acc4e22a67bff89e33afd2342db0a1
Cr-Commit-Position: refs/heads/master@{#430713}

[Tobias Sargeant, 2016-11-08 13:12:13]

Description was changed from

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
==========

to

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
==========

[Tobias Sargeant, 2016-11-08 13:12:13]

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

[Torne, 2016-11-08 14:36:13]

lgtm

[Tobias Sargeant, 2016-11-08 20:04:41]

The CQ bit was checked by tobiasjs@chromium.org

[commit-bot: I haz the power, 2016-11-08 20:05:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-08 21:10:38]

Description was changed from

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
==========

to

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
==========

[commit-bot: I haz the power, 2016-11-08 21:10:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 21:15:25]

Description was changed from

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853
==========

to

==========
Always install the microdump signal handler on N+ devices.

N+ shared library namespacing means that the current detection code that
determines whether it is safe to install the microdump signal handler
fails, because it is impossible to dlopen libnativebridge (because it
isn't in the NDK).

Because the libnativebridge on N+ devices is sufficiently recent as to
not need the test, we simply short-circuit on those devices.

BUG=662853

Committed: https://crrev.com/2c944c9e76acc4e22a67bff89e33afd2342db0a1
Cr-Commit-Position: refs/heads/master@{#430713}
==========

[commit-bot: I haz the power, 2016-11-08 21:15:26]

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