[Remoting Android] Fix component build runtime error

[Remoting Android] Fix component build runtime error

Component build has been turned on by default but the generated Android client
doesn't work since the JNI shared library cannot be located. The issue is that
turning on the component build flag will add a ".cr" suffix to the name of the
library while we always load `remoting_client_jni` unconditionally.

This CL adds fallback for loading `remoting_client_jni.cr` if
`remoting_client_jni` can't be loaded.

BUG=634375
Committed: https://crrev.com/fa2204311b7b12bf707b1f670ca6ebe3b1e26250
Cr-Commit-Position: refs/heads/master@{#410550}

[Yuwei, 2016-08-09 00:40:54]

yuweih@chromium.org changed reviewers:
+ lambroslambrou@chromium.org

[Yuwei, 2016-08-09 00:40:54]

ptal

[Lambros, 2016-08-09 00:51:54]

lgtm

https://codereview.chromium.org/2228783002/diff/1/remoting/android/java/src/o...
File remoting/android/java/src/org/chromium/chromoting/jni/JniInterface.java
(right):

https://codereview.chromium.org/2228783002/diff/1/remoting/android/java/src/o...
remoting/android/java/src/org/chromium/chromoting/jni/JniInterface.java:26:
private static final String JNI_LIB_NAME = "remoting_client_jni";
Maybe just LIBRARY_NAME ?

[Yuwei, 2016-08-09 00:56:17]

https://codereview.chromium.org/2228783002/diff/1/remoting/android/java/src/o...
File remoting/android/java/src/org/chromium/chromoting/jni/JniInterface.java
(right):

https://codereview.chromium.org/2228783002/diff/1/remoting/android/java/src/o...
remoting/android/java/src/org/chromium/chromoting/jni/JniInterface.java:26:
private static final String JNI_LIB_NAME = "remoting_client_jni";
On 2016/08/09 00:51:54, Lambros wrote:
> Maybe just LIBRARY_NAME ?

Done.

[Yuwei, 2016-08-09 00:57:59]

The CQ bit was checked by yuweih@chromium.org

[Yuwei, 2016-08-09 00:57:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
lambroslambrou@chromium.org
Link to the patchset: https://codereview.chromium.org/2228783002/#ps20001
(title: "Reviewer's Feedback")

[commit-bot: I haz the power, 2016-08-09 00:58:31]

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-08-09 02:44:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-09 02:47:01]

Description was changed from

==========
[Remoting Android] Fix component build runtime error

Component build has been turned on by default but the generated Android client
doesn't work since the JNI shared library cannot be located. The issue is that
turning on the component build flag will add a ".cr" suffix to the name of the
library while we always load `remoting_client_jni` unconditionally.

This CL adds fallback for loading `remoting_client_jni.cr` if
`remoting_client_jni` can't be loaded.

BUG=634375
==========

to

==========
[Remoting Android] Fix component build runtime error

Component build has been turned on by default but the generated Android client
doesn't work since the JNI shared library cannot be located. The issue is that
turning on the component build flag will add a ".cr" suffix to the name of the
library while we always load `remoting_client_jni` unconditionally.

This CL adds fallback for loading `remoting_client_jni.cr` if
`remoting_client_jni` can't be loaded.

BUG=634375

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

[commit-bot: I haz the power, 2016-08-09 02:47:02]

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