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

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

Issue 2378943002: Let clients interact with VideoCaptureDeviceClient instead of VideoCaptureDevice (Closed)
Patch Set: mcasas comments Created 4 years, 2 months 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 "services/video_capture/public/interfaces/video_capture_device_proxy.mo jom.h" 10 #include "services/video_capture/public/interfaces/video_capture_device_proxy.mo jom.h"
10 11
11 namespace video_capture { 12 namespace video_capture {
12 13
13 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance 14 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance
14 // of media::VideoCaptureDevice. 15 // of media::VideoCaptureDevice.
15 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy { 16 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy {
16 public: 17 public:
17 VideoCaptureDeviceProxyImpl( 18 VideoCaptureDeviceProxyImpl(std::unique_ptr<media::VideoCaptureDevice> device,
18 std::unique_ptr<media::VideoCaptureDevice> device); 19 const media::VideoCaptureJpegDecoderFactoryCB&
20 jpeg_decoder_factory_callback);
19 ~VideoCaptureDeviceProxyImpl() override; 21 ~VideoCaptureDeviceProxyImpl() override;
20 22
21 // mojom::VideoCaptureDeviceProxy: 23 // mojom::VideoCaptureDeviceProxy:
22 void Start(mojom::VideoCaptureFormatPtr requested_format, 24 void Start(mojom::VideoCaptureFormatPtr requested_format,
23 mojom::ResolutionChangePolicy resolution_change_policy, 25 mojom::ResolutionChangePolicy resolution_change_policy,
24 mojom::PowerLineFrequency power_line_frequency, 26 mojom::PowerLineFrequency power_line_frequency,
25 mojom::VideoCaptureDeviceClientPtr client) override; 27 mojom::VideoFrameReceiverPtr receiver) override;
26 28
27 void Stop(); 29 void Stop();
28 30
29 void OnClientConnectionErrorOrClose(); 31 void OnClientConnectionErrorOrClose();
30 32
31 private: 33 private:
32 std::unique_ptr<media::VideoCaptureDevice> device_; 34 const std::unique_ptr<media::VideoCaptureDevice> device_;
33 bool device_running_ = false; 35 media::VideoCaptureJpegDecoderFactoryCB jpeg_decoder_factory_callback_;
36 bool device_running_;
34 }; 37 };
35 38
36 } // namespace video_capture 39 } // namespace video_capture
37 40
38 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_ 41 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698