Revert of Refactor Chromoting JNI code to use jni/Client (Java changes only).

Revert of Refactor Chromoting JNI code to use jni/Client (Java changes only). (patchset #7 id:120001 of https://codereview.chromium.org/1537183002/ )

Reason for revert:
This broke ToT: http://crbug.com/585799

Original issue's description:
> Refactor Chromoting JNI code to use jni/Client (Java changes only).
>
> This removes globals from jni/JniInterface in favor of passing a
> jni/Client instance to code that needs to access the Chromoting
> client connection.
>
> The C++ code is unchanged - JniInterface class is still used to
> marshall Java <-> C++ calls. JniInterface still holds a singleton
> Client connection, but all other globals have been removed (including
> the singleton CapabilityManager instance).
>
> Each Activity's onCreate() method gets the singleton Client instance,
> and then passes it to objects that need it (for input injection and
> so on). This should make it easier to use a fake Client instance for
> unit-instrumentation tests.
>
> BUG=526336
>
> Committed: https://crrev.com/22ee9b8426839b0c2c841fbc919cd4a79b1456d0
> Cr-Commit-Position: refs/heads/master@{#374548}

TBR=sergeyu@chromium.org,joedow@chromium.org,lambroslambrou@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=526336
Committed: https://crrev.com/6fd49ae9090db2678c07f0479a96e7dbd6651dff
Cr-Commit-Position: refs/heads/master@{#374660}

[hartmanng, 2016-02-10 14:59:09]

Created Revert of Refactor Chromoting JNI code to use jni/Client (Java changes
only).

[commit-bot: I haz the power, 2016-02-10 14:59:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1687873002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1687873002/1

[commit-bot: I haz the power, 2016-02-10 15:01:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-10 15:03:28]

Description was changed from

==========
Revert of Refactor Chromoting JNI code to use jni/Client (Java changes only).
(patchset #7 id:120001 of https://codereview.chromium.org/1537183002/ )

Reason for revert:
This broke ToT: http://crbug.com/585799

Original issue's description:
> Refactor Chromoting JNI code to use jni/Client (Java changes only).
> 
> This removes globals from jni/JniInterface in favor of passing a
> jni/Client instance to code that needs to access the Chromoting
> client connection.
> 
> The C++ code is unchanged - JniInterface class is still used to
> marshall Java <-> C++ calls. JniInterface still holds a singleton
> Client connection, but all other globals have been removed (including
> the singleton CapabilityManager instance).
> 
> Each Activity's onCreate() method gets the singleton Client instance,
> and then passes it to objects that need it (for input injection and
> so on). This should make it easier to use a fake Client instance for
> unit-instrumentation tests.
> 
> BUG=526336
> 
> Committed: https://crrev.com/22ee9b8426839b0c2c841fbc919cd4a79b1456d0
> Cr-Commit-Position: refs/heads/master@{#374548}

TBR=sergeyu@chromium.org,joedow@chromium.org,lambroslambrou@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=526336
==========

to

==========
Revert of Refactor Chromoting JNI code to use jni/Client (Java changes only).
(patchset #7 id:120001 of https://codereview.chromium.org/1537183002/ )

Reason for revert:
This broke ToT: http://crbug.com/585799

Original issue's description:
> Refactor Chromoting JNI code to use jni/Client (Java changes only).
>
> This removes globals from jni/JniInterface in favor of passing a
> jni/Client instance to code that needs to access the Chromoting
> client connection.
>
> The C++ code is unchanged - JniInterface class is still used to
> marshall Java <-> C++ calls. JniInterface still holds a singleton
> Client connection, but all other globals have been removed (including
> the singleton CapabilityManager instance).
>
> Each Activity's onCreate() method gets the singleton Client instance,
> and then passes it to objects that need it (for input injection and
> so on). This should make it easier to use a fake Client instance for
> unit-instrumentation tests.
>
> BUG=526336
>
> Committed: https://crrev.com/22ee9b8426839b0c2c841fbc919cd4a79b1456d0
> Cr-Commit-Position: refs/heads/master@{#374548}

TBR=sergeyu@chromium.org,joedow@chromium.org,lambroslambrou@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=526336

Committed: https://crrev.com/6fd49ae9090db2678c07f0479a96e7dbd6651dff
Cr-Commit-Position: refs/heads/master@{#374660}
==========

[commit-bot: I haz the power, 2016-02-10 15:03:29]

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