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

Side by Side Diff: media/mojo/services/mojo_video_decoder_service.cc

Issue 2640153004: Add mailbox-based Mojo VideoFrame variant. (Closed)
Patch Set: Switch to an interface method. Created 3 years, 10 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 #include "media/mojo/services/mojo_video_decoder_service.h" 5 #include "media/mojo/services/mojo_video_decoder_service.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/threading/thread_task_runner_handle.h" 10 #include "base/threading/thread_task_runner_handle.h"
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 126
127 void MojoVideoDecoderService::OnDecoderReset(const ResetCallback& callback) { 127 void MojoVideoDecoderService::OnDecoderReset(const ResetCallback& callback) {
128 DVLOG(1) << __func__; 128 DVLOG(1) << __func__;
129 callback.Run(); 129 callback.Run();
130 } 130 }
131 131
132 void MojoVideoDecoderService::OnDecoderOutput( 132 void MojoVideoDecoderService::OnDecoderOutput(
133 const scoped_refptr<VideoFrame>& frame) { 133 const scoped_refptr<VideoFrame>& frame) {
134 DVLOG(2) << __func__; 134 DVLOG(2) << __func__;
135 DCHECK(client_); 135 DCHECK(client_);
136 client_->OnVideoFrameDecoded(mojom::VideoFrame::From(frame)); 136 client_->OnVideoFrameDecoded(mojom::VideoFrame::From(frame), base::nullopt);
nasko 2017/02/16 21:37:29 You probably want base/optional.h included here in
sandersd (OOO until July 31) 2017/02/25 01:19:38 Done.
137 }
138
139 void MojoVideoDecoderService::OnReleaseMailbox(
140 const base::UnguessableToken& release_token,
141 const gpu::SyncToken& release_sync_token) {
142 DVLOG(2) << __func__;
137 } 143 }
138 144
139 } // namespace media 145 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698