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

Side by Side Diff: services/video_capture/receiver_mojo_to_media_adapter.cc

Issue 2583603002: [Mojo Video Capture] Split OnIncomingCapturedVideoFrame() to OnNewBuffer() + OnFrameReadyInBuffer() (Closed)
Patch Set: Improve naming and fix Android background issue Created 3 years, 11 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
« no previous file with comments | « services/video_capture/receiver_mojo_to_media_adapter.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "services/video_capture/receiver_mojo_to_media_adapter.h" 5 #include "services/video_capture/receiver_mojo_to_media_adapter.h"
6 6
7 #include "media/mojo/common/media_type_converters.h"
8 #include "media/mojo/common/mojo_shared_buffer_video_frame.h"
9
10 namespace video_capture { 7 namespace video_capture {
11 8
12 ReceiverMojoToMediaAdapter::ReceiverMojoToMediaAdapter( 9 ReceiverMojoToMediaAdapter::ReceiverMojoToMediaAdapter(
13 mojom::ReceiverPtr receiver) 10 mojom::ReceiverPtr receiver)
14 : receiver_(std::move(receiver)) {} 11 : receiver_(std::move(receiver)) {}
15 12
16 ReceiverMojoToMediaAdapter::~ReceiverMojoToMediaAdapter() = default; 13 ReceiverMojoToMediaAdapter::~ReceiverMojoToMediaAdapter() = default;
17 14
18 void ReceiverMojoToMediaAdapter::OnIncomingCapturedVideoFrame( 15 void ReceiverMojoToMediaAdapter::OnNewBufferHandle(
19 media::VideoCaptureDevice::Client::Buffer buffer, 16 int buffer_id,
20 scoped_refptr<media::VideoFrame> frame) { 17 std::unique_ptr<media::BufferHandleProvider> handle_provider) {
21 NOTIMPLEMENTED(); 18 NOTIMPLEMENTED();
22 } 19 }
23 20
21 void ReceiverMojoToMediaAdapter::OnFrameReadyInBuffer(
22 int buffer_id,
23 int frame_feedback_id,
24 std::unique_ptr<media::Ownership> buffer_usage_reservation,
25 media::mojom::VideoFrameInfoPtr frame_info) {
26 NOTIMPLEMENTED();
27 }
28
24 void ReceiverMojoToMediaAdapter::OnError() { 29 void ReceiverMojoToMediaAdapter::OnError() {
25 receiver_->OnError(); 30 receiver_->OnError();
26 } 31 }
27 32
28 void ReceiverMojoToMediaAdapter::OnLog(const std::string& message) { 33 void ReceiverMojoToMediaAdapter::OnLog(const std::string& message) {
29 receiver_->OnLog(message); 34 receiver_->OnLog(message);
30 } 35 }
31 36
32 void ReceiverMojoToMediaAdapter::OnBufferDestroyed(int buffer_id_to_drop) { 37 void ReceiverMojoToMediaAdapter::OnBufferRetired(int buffer_id) {
33 // Nothing to do here. 38 NOTIMPLEMENTED();
34 // This call is only needed for clients who need to explicitly share buffers
35 // with other processes and keep track of which buffers the buffer pool is
36 // no longer going to reuse/resurrect.
37 // In the world of Mojo, we do not need to share buffers explicitly.
38 } 39 }
39 40
40 } // namespace video_capture 41 } // namespace video_capture
OLDNEW
« no previous file with comments | « services/video_capture/receiver_mojo_to_media_adapter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698