Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1644)

Side by Side Diff: services/video_capture/video_capture_device_proxy_impl.h

Issue 2457003002: [Mojo Video Capture] Simplify API and some cleanups (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_ 5 #ifndef SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_
6 #define SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_ 6 #define SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_
7 7
8 #include "media/capture/video/video_capture_device.h" 8 #include "media/capture/video/video_capture_device.h"
9 #include "media/capture/video/video_capture_device_client.h" 9 #include "media/capture/video/video_capture_device_client.h"
10 #include "media/capture/video_capture_types.h" 10 #include "media/capture/video_capture_types.h"
11 #include "services/video_capture/public/interfaces/video_capture_device_proxy.mo jom.h" 11 #include "services/video_capture/public/interfaces/video_capture_device_proxy.mo jom.h"
12 12
13 namespace video_capture { 13 namespace video_capture {
14 14
15 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance 15 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance
16 // of media::VideoCaptureDevice. 16 // of media::VideoCaptureDevice.
17 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy { 17 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy {
18 public: 18 public:
19 VideoCaptureDeviceProxyImpl(std::unique_ptr<media::VideoCaptureDevice> device, 19 VideoCaptureDeviceProxyImpl(std::unique_ptr<media::VideoCaptureDevice> device,
20 const media::VideoCaptureJpegDecoderFactoryCB& 20 const media::VideoCaptureJpegDecoderFactoryCB&
21 jpeg_decoder_factory_callback); 21 jpeg_decoder_factory_callback);
22 ~VideoCaptureDeviceProxyImpl() override; 22 ~VideoCaptureDeviceProxyImpl() override;
23 23
24 // mojom::VideoCaptureDeviceProxy: 24 // mojom::VideoCaptureDeviceProxy:
25 void Start(const media::VideoCaptureFormat& requested_format, 25 void Start(const VideoCaptureSettings& requested_settings,
26 media::ResolutionChangePolicy resolution_change_policy,
27 media::PowerLineFrequency power_line_frequency,
28 mojom::VideoFrameReceiverPtr receiver) override; 26 mojom::VideoFrameReceiverPtr receiver) override;
29 27
30 void Stop(); 28 void Stop();
31 29
32 void OnClientConnectionErrorOrClose(); 30 void OnClientConnectionErrorOrClose();
33 31
34 private: 32 private:
35 const std::unique_ptr<media::VideoCaptureDevice> device_; 33 const std::unique_ptr<media::VideoCaptureDevice> device_;
36 media::VideoCaptureJpegDecoderFactoryCB jpeg_decoder_factory_callback_; 34 media::VideoCaptureJpegDecoderFactoryCB jpeg_decoder_factory_callback_;
37 bool device_running_; 35 bool device_running_;
38 }; 36 };
39 37
40 } // namespace video_capture 38 } // namespace video_capture
41 39
42 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_ 40 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698