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

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: Remove method AsClientBuffer() 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/base/video_capture_types.h" 8 #include "media/base/video_capture_types.h"
9 #include "media/capture/video/video_capture_device.h" 9 #include "media/capture/video/video_capture_device.h"
10 #include "media/capture/video/video_capture_device_client.h"
10 #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"
11 12
12 namespace video_capture { 13 namespace video_capture {
13 14
14 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance 15 // Implementation of mojom::VideoCaptureDeviceProxy backed by a given instance
15 // of media::VideoCaptureDevice. 16 // of media::VideoCaptureDevice.
16 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy { 17 class VideoCaptureDeviceProxyImpl : public mojom::VideoCaptureDeviceProxy {
17 public: 18 public:
18 VideoCaptureDeviceProxyImpl( 19 VideoCaptureDeviceProxyImpl(std::unique_ptr<media::VideoCaptureDevice> device,
19 std::unique_ptr<media::VideoCaptureDevice> device); 20 const media::VideoCaptureJpegDecoderFactoryCB&
21 jpeg_decoder_factory_callback);
20 ~VideoCaptureDeviceProxyImpl() override; 22 ~VideoCaptureDeviceProxyImpl() override;
21 23
22 // mojom::VideoCaptureDeviceProxy: 24 // mojom::VideoCaptureDeviceProxy:
23 void Start(const media::VideoCaptureFormat& requested_format, 25 void Start(const media::VideoCaptureFormat& requested_format,
24 media::ResolutionChangePolicy resolution_change_policy, 26 media::ResolutionChangePolicy resolution_change_policy,
25 media::PowerLineFrequency power_line_frequency, 27 media::PowerLineFrequency power_line_frequency,
26 mojom::VideoCaptureDeviceClientPtr client) override; 28 mojom::VideoFrameReceiverPtr receiver) override;
27 29
28 void Stop(); 30 void Stop();
29 31
30 void OnClientConnectionErrorOrClose(); 32 void OnClientConnectionErrorOrClose();
31 33
32 private: 34 private:
33 std::unique_ptr<media::VideoCaptureDevice> device_; 35 const std::unique_ptr<media::VideoCaptureDevice> device_;
34 bool device_running_ = false; 36 media::VideoCaptureJpegDecoderFactoryCB jpeg_decoder_factory_callback_;
37 bool device_running_;
35 }; 38 };
36 39
37 } // namespace video_capture 40 } // namespace video_capture
38 41
39 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_ 42 #endif // SERVICES_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_IMPL_H_
OLDNEW
« no previous file with comments | « services/video_capture/video_capture_device_factory_impl.cc ('k') | services/video_capture/video_capture_device_proxy_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698