VideoCapturerAndroid: Check that camera is still running in every *OnCameraThread method

VideoCapturerAndroid: Check that camera is still running in every *OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
R=sakal@webrtc.org

Committed: https://crrev.com/2e646c8729f9ca6a07663d9e0ffa82a380742f5d
Cr-Commit-Position: refs/heads/master@{#12981}

[magjed1, 2016-05-31 14:21:02]

Description was changed from

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods before continuing.

BUG=b/29015569
==========

to

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
==========

[magjed_webrtc, 2016-05-31 14:24:06]

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

[magjed_webrtc, 2016-05-31 14:24:19]

Sami - Please take a look.

[sakal, 2016-05-31 15:10:56]

Description was changed from

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
==========

to

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=webrtc:29015569
==========

[sakal, 2016-05-31 15:11:29]

Description was changed from

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=webrtc:29015569
==========

to

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
==========

[sakal, 2016-05-31 15:20:32]

Ugly but lgtm so we don't crash.

[magjed_webrtc, 2016-05-31 15:23:10]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2016-05-31 15:23:18]

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

[commit-bot: I haz the power, 2016-05-31 16:29:01]

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

[commit-bot: I haz the power, 2016-05-31 16:29:05]

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

[magjed_webrtc, 2016-06-01 07:45:37]

Description was changed from

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
==========

to

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
R=sakal@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/2e646c8729f9ca6a07663d9e0...
==========

[commit-bot: I haz the power, 2016-06-01 07:45:37]

Description was changed from

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
==========

to

==========
VideoCapturerAndroid: Check that camera is still running in every
*OnCameraThread method

removeCallbacksAndMessages() is called on the camera thread handler
before setting it to null to remove all pending runnables. The purpose
is to make sure *OnCameraThread methods are not executed when the
camera is stopped, but this does not seem to work reliably. This CL
resorts to a belt and braces approach and checks that the the handler is
still alive in all *OnCameraThread methods.

BUG=b/29015569
R=sakal@webrtc.org

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

[commit-bot: I haz the power, 2016-06-01 07:45:38]

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

[magjed_webrtc, 2016-06-01 07:45:38]

Committed patchset #1 (id:1) manually as
2e646c8729f9ca6a07663d9e0ffa82a380742f5d (presubmit successful).