VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl() before sending

VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl() before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897
Committed: https://crrev.com/f81183123c82618825cd5734c5ace89fe1723d3c
Cr-Commit-Position: refs/heads/master@{#423907}

[mcasas, 2016-10-07 16:26:08]

Description was changed from

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

BUG=653806
==========

to

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806
==========

[mcasas, 2016-10-07 16:33:22]

mcasas@chromium.org changed reviewers:
+ emircan@chromium.org

[mcasas, 2016-10-07 16:33:23]

emircan@ PTAL (easy CL)

[emircan, 2016-10-07 17:09:53]

lgtm

[mcasas, 2016-10-07 17:13:31]

Description was changed from

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806
==========

to

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897
==========

[mcasas, 2016-10-07 17:13:36]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-10-07 17:13:58]

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-10-07 17:59:38]

Description was changed from

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897
==========

to

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897
==========

[commit-bot: I haz the power, 2016-10-07 17:59:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-07 18:01:16]

Description was changed from

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897
==========

to

==========
VideoCaptureImpl: check that there's still a channel in ~VideoCaptureImpl()
before sending

This was not hit when using IPC due to the channel ignoring
the late message, but now we need to protect against a
disappeared VideoCaptureHost explicitly.

Also, this CL  the unused |suspended_| variable
and simplified RemoveClient() method -- I know, I
know, I shouldn't bundle modifications in a CL...
but these were just too easy to let them pass.

BUG=653806, 651897

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

[commit-bot: I haz the power, 2016-10-07 18:01:17]

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