[Mojo Video Capture] Hook up connection lost events

[Mojo Video Capture] Hook up connection lost events

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL26

Pass a callback into ServiceLaunchedVideoCaptureDevice that gets invoked when
the connection to the video capture service is lost. The connection lost signal
is passed through the VideoCaptureController to the VideoCaptureManager.

Additional change:
Uninitialize VideoCaptureProvider when the last client disconnects. This is to
allow the service to shut down when it is no longer in use.

BUG=584797
TEST=
  content_unittests --gtest_filter="*Video*"
  content_browsertests --gtest_filter="VideoCaptureBrowserTest.*"

[1] https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxsxWslJ_X-zpOhijvI8/edit?usp=sharing

Review-Url: https://codereview.chromium.org/2902203002
Cr-Commit-Position: refs/heads/master@{#476469}
Committed: https://chromium.googlesource.com/chromium/src/+/4186535f976a6ce2d1785d87e1f200419928164b

[chfremer, 2017-05-24 17:02:06]

The CQ bit was checked by chfremer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-24 17:02:42]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chfremer, 2017-05-24 17:02:45]

chfremer@chromium.org changed reviewers:
+ emircan@chromium.org, mcasas@chromium.org, miu@chromium.org

[chfremer, 2017-05-24 17:02:45]

emircan@: PTAL
mcasas@, miu@: FYI

[commit-bot: I haz the power, 2017-05-24 18:50:30]

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

[commit-bot: I haz the power, 2017-05-24 18:50:31]

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

[emircan, 2017-06-01 18:13:29]

lgtm % nits.

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
File
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc
(right):

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc:82:
base::MockCallback<base::OnceClosure> connection_lost_cb;
s/connection_lost_cb/connection_lost_cb_

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc:83:
base::MockCallback<base::OnceClosure> done_cb;
s/done_cb/done_cb_

[chfremer, 2017-06-01 19:25:56]

Thanks for the review!

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
File
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc
(right):

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc:82:
base::MockCallback<base::OnceClosure> connection_lost_cb;
On 2017/06/01 18:13:29, emircan wrote:
> s/connection_lost_cb/connection_lost_cb_

Done.

https://codereview.chromium.org/2902203002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/media/service_video_capture_device_launcher_unittest.cc:83:
base::MockCallback<base::OnceClosure> done_cb;
On 2017/06/01 18:13:29, emircan wrote:
> s/done_cb/done_cb_

Done.

[chfremer, 2017-06-01 19:26:02]

The CQ bit was checked by chfremer@chromium.org

[chfremer, 2017-06-01 19:26:02]

The patchset sent to the CQ was uploaded after l-g-t-m from emircan@chromium.org
Link to the patchset: https://codereview.chromium.org/2902203002/#ps20001
(title: "Incorporate suggestions from PS1")

[commit-bot: I haz the power, 2017-06-01 19:26:48]

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, 2017-06-01 20:04:48]

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

[commit-bot: I haz the power, 2017-06-01 20:04:50]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[chfremer, 2017-06-01 21:14:12]

The CQ bit was checked by chfremer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-01 21:15:21]

Dry run: 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, 2017-06-01 22:48:48]

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

[commit-bot: I haz the power, 2017-06-01 22:48:49]

Dry run: This issue passed the CQ dry run.

[chfremer, 2017-06-01 22:59:53]

The CQ bit was checked by chfremer@chromium.org

[chfremer, 2017-06-01 22:59:53]

The patchset sent to the CQ was uploaded after l-g-t-m from emircan@chromium.org
Link to the patchset: https://codereview.chromium.org/2902203002/#ps40001
(title: "Fix duplicate member introduced during merging")

[commit-bot: I haz the power, 2017-06-01 23:00:39]

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, 2017-06-01 23:04:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496357993172520,
"parent_rev": "309bf274ee09d077847e184b5addf5f935a852e0", "commit_rev":
"4186535f976a6ce2d1785d87e1f200419928164b"}

[commit-bot: I haz the power, 2017-06-01 23:04:46]

Description was changed from

==========
[Mojo Video Capture] Hook up connection lost events

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL26

Pass a callback into ServiceLaunchedVideoCaptureDevice that gets invoked when
the connection to the video capture service is lost. The connection lost signal
is passed through the VideoCaptureController to the VideoCaptureManager.

Additional change:
Uninitialize VideoCaptureProvider when the last client disconnects. This is to
allow the service to shut down when it is no longer in use.

BUG=584797
TEST=
  content_unittests --gtest_filter="*Video*"
  content_browsertests --gtest_filter="VideoCaptureBrowserTest.*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
[Mojo Video Capture] Hook up connection lost events

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL26

Pass a callback into ServiceLaunchedVideoCaptureDevice that gets invoked when
the connection to the video capture service is lost. The connection lost signal
is passed through the VideoCaptureController to the VideoCaptureManager.

Additional change:
Uninitialize VideoCaptureProvider when the last client disconnects. This is to
allow the service to shut down when it is no longer in use.

BUG=584797
TEST=
  content_unittests --gtest_filter="*Video*"
  content_browsertests --gtest_filter="VideoCaptureBrowserTest.*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...

Review-Url: https://codereview.chromium.org/2902203002
Cr-Commit-Position: refs/heads/master@{#476469}
Committed:
https://chromium.googlesource.com/chromium/src/+/4186535f976a6ce2d1785d87e1f2...
==========

[commit-bot: I haz the power, 2017-06-01 23:04:47]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4186535f976a6ce2d1785d87e1f2...