Remove LooperExecutor and replace it with built-in HandlerThread.

Remove LooperExecutor and replace it with built-in HandlerThread.

Committed: https://crrev.com/5d4d59c069628396548ce36c5b7c4d95fba34a5b
Cr-Commit-Position: refs/heads/master@{#13688}

[sakal, 2016-08-02 09:07:00]

sakal@webrtc.org changed reviewers:
+ magjed@webrtc.org

[sakal, 2016-08-02 09:07:35]

Magnus, PTAL when you get back from vacation.

[sakal, 2016-08-02 13:45:22]

The CQ bit was checked by sakal@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 13:45:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-08-02 13:55:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-02 13:55:39]

Dry run: Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

[sakal, 2016-08-04 07:31:53]

Description was changed from

==========
Modify LooperExecutor to use HandlerThread.
==========

to

==========
Remove LooperExecutor and replace it with built-in HandlerThread.
==========

[magjed_webrtc, 2016-08-08 09:37:31]

https://codereview.webrtc.org/2206473003/diff/60001/webrtc/examples/androidap...
File webrtc/examples/androidapp/src/org/appspot/apprtc/WebSocketRTCClient.java
(right):

https://codereview.webrtc.org/2206473003/diff/60001/webrtc/examples/androidap...
webrtc/examples/androidapp/src/org/appspot/apprtc/WebSocketRTCClient.java:92:
handlerThread.quitSafely();
quitSafely was added in API level 18 so I don't think you can use it. Is it safe
to just quit() instead? Or you can post a quit() call to emulate quitSafely().

Also, you don't need to store handlerThread, you can quit the looper in
handler.getLooper() instead, it's equivalent.

[sakal, 2016-08-08 09:52:52]

https://codereview.webrtc.org/2206473003/diff/60001/webrtc/examples/androidap...
File webrtc/examples/androidapp/src/org/appspot/apprtc/WebSocketRTCClient.java
(right):

https://codereview.webrtc.org/2206473003/diff/60001/webrtc/examples/androidap...
webrtc/examples/androidapp/src/org/appspot/apprtc/WebSocketRTCClient.java:92:
handlerThread.quitSafely();
On 2016/08/08 09:37:31, magjed_webrtc wrote:
> quitSafely was added in API level 18 so I don't think you can use it. Is it
safe
> to just quit() instead? Or you can post a quit() call to emulate quitSafely().
> 
> Also, you don't need to store handlerThread, you can quit the looper in
> handler.getLooper() instead, it's equivalent.

Done.

[magjed_webrtc, 2016-08-08 12:01:20]

lgtm

[sakal, 2016-08-08 12:02:52]

The CQ bit was checked by sakal@webrtc.org

[commit-bot: I haz the power, 2016-08-08 12:02:55]

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

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

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-08 12:15:27]

Try jobs failed on following builders:
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_clang_dbg/build...)

[sakal, 2016-08-09 08:58:02]

The CQ bit was checked by sakal@webrtc.org

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

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

[commit-bot: I haz the power, 2016-08-09 09:03:44]

Description was changed from

==========
Remove LooperExecutor and replace it with built-in HandlerThread.
==========

to

==========
Remove LooperExecutor and replace it with built-in HandlerThread.

Committed: https://crrev.com/5d4d59c069628396548ce36c5b7c4d95fba34a5b
Cr-Commit-Position: refs/heads/master@{#13688}
==========

[commit-bot: I haz the power, 2016-08-09 09:03:44]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-09 09:03:45]

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