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

Unified Diff: media/mojo/services/mojo_decryptor_service.cc

Issue 2908303003: media: Create Mojo StructTraits for VideoFrame (Closed)
Patch Set: Eagerly initialize the context object. Created 3 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: media/mojo/services/mojo_decryptor_service.cc
diff --git a/media/mojo/services/mojo_decryptor_service.cc b/media/mojo/services/mojo_decryptor_service.cc
index 592bcb6ed873ecfe439ea7f5b4791c7a6366ac81..0e140b79de70e85119c5dcf95c81849e0514f82b 100644
--- a/media/mojo/services/mojo_decryptor_service.cc
+++ b/media/mojo/services/mojo_decryptor_service.cc
@@ -238,15 +238,13 @@ void MojoDecryptorService::OnVideoDecoded(
// If |frame| has shared memory that will be passed back, keep the reference
// to it until the other side is done with the memory.
- mojom::VideoFramePtr mojo_frame = mojom::VideoFrame::From(frame);
mojom::FrameResourceReleaserPtr releaser;
if (frame->storage_type() == VideoFrame::STORAGE_MOJO_SHARED_BUFFER) {
- mojo::MakeStrongBinding(
- base::MakeUnique<FrameResourceReleaserImpl>(std::move(frame)),
- mojo::MakeRequest(&releaser));
+ mojo::MakeStrongBinding(base::MakeUnique<FrameResourceReleaserImpl>(frame),
+ mojo::MakeRequest(&releaser));
}
- callback.Run(status, std::move(mojo_frame), std::move(releaser));
+ callback.Run(status, std::move(frame), std::move(releaser));
}
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698