Release the camera when WebRTC is not visible in Android

content/browser/renderer_host/media/video_capture_controller.h

Index: content/browser/renderer_host/media/video_capture_controller.h
diff --git a/content/browser/renderer_host/media/video_capture_controller.h b/content/browser/renderer_host/media/video_capture_controller.h
index 2de5ff6bbf795cc898029dfd1afca9108015f281..efcbe0275e00926d5084f3efc6f961f1d2af9ae4 100644
--- a/content/browser/renderer_host/media/video_capture_controller.h
+++ b/content/browser/renderer_host/media/video_capture_controller.h
@@ -96,7 +96,15 @@ class CONTENT_EXPORT VideoCaptureController {
   int RemoveClient(const VideoCaptureControllerID& id,
                    VideoCaptureControllerEventHandler* event_handler);
 
-  int GetClientCount();
+  // Pause or resume the video capture for specified client.
+  void PauseOrResumeClient(const VideoCaptureControllerID& id,

[Tom Sepez, 2014/10/02 17:16:26]

Maybe split it into separate pause/resume methods to make the meaning clearer at
the call sites.

[michaelbai, 2014/10/02 18:55:39]

there will have a lot of duplicated code in implementation if split it.


On 2014/10/02 17:16:26, Tom Sepez wrote:

+                           VideoCaptureControllerEventHandler* event_handler,
+                           bool pause);
+
+  int GetClientCount() const;
+
+  // Return the number of client that isn't paused.

[Tom Sepez, 2014/10/02 17:16:26]

nit: // Return the number of clients that aren't paused.

[michaelbai, 2014/10/02 18:55:39]

Done.

+  int GetActiveClientCount() const;
 
   // API called directly by VideoCaptureManager in case the device is
   // prematurely closed.