OLD | NEW |
---|---|
1 // Copyright 2017 The Chromium Authors. All rights reserved. | 1 // Copyright 2017 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DEVIC E_H_ | 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DEVIC E_H_ |
6 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DEVIC E_H_ | 6 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DEVIC E_H_ |
7 | 7 |
8 #include "content/browser/renderer_host/media/video_capture_provider.h" | 8 #include "content/browser/renderer_host/media/video_capture_provider.h" |
9 #include "services/video_capture/public/interfaces/device.mojom.h" | 9 #include "services/video_capture/public/interfaces/device.mojom.h" |
10 | 10 |
(...skipping 19 matching lines...) Expand all Loading... | |
30 void MaybeSuspendDevice() override; | 30 void MaybeSuspendDevice() override; |
31 void ResumeDevice() override; | 31 void ResumeDevice() override; |
32 void RequestRefreshFrame() override; | 32 void RequestRefreshFrame() override; |
33 | 33 |
34 void SetDesktopCaptureWindowIdAsync(gfx::NativeViewId window_id, | 34 void SetDesktopCaptureWindowIdAsync(gfx::NativeViewId window_id, |
35 base::OnceClosure done_cb) override; | 35 base::OnceClosure done_cb) override; |
36 | 36 |
37 void OnUtilizationReport(int frame_feedback_id, double utilization) override; | 37 void OnUtilizationReport(int frame_feedback_id, double utilization) override; |
38 | 38 |
39 private: | 39 private: |
40 const video_capture::mojom::DevicePtr device_; | 40 void OnLostConnectionToDevice(); |
41 | |
42 video_capture::mojom::DevicePtr device_; | |
43 base::ThreadChecker thread_checker_; | |
miu
2017/05/08 20:21:11
Is this necessary? Seems like this class is single
chfremer
2017/05/08 20:51:46
I was not aware of that discussion and wasn't able
chfremer
2017/05/09 18:41:33
Removed ThreadChecker here and in ServiceVideoCapt
miu
2017/05/09 19:34:19
Here: https://groups.google.com/a/chromium.org/for
chfremer
2017/05/09 20:49:01
After reading the e-mail thread, my conclusion is
| |
41 }; | 44 }; |
42 | 45 |
43 } // namespace content | 46 } // namespace content |
44 | 47 |
45 #endif // CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DE VICE_H_ | 48 #endif // CONTENT_BROWSER_RENDERER_HOST_MEDIA_SERVICE_LAUNCHED_VIDEO_CAPTURE_DE VICE_H_ |
OLD | NEW |