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

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

Issue 2544523002: Pass Connector* into MojoMediaClient::Initialize(). (Closed)
Patch Set: Rebase. Created 4 years 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/android_mojo_media_client.cc
diff --git a/media/mojo/services/android_mojo_media_client.cc b/media/mojo/services/android_mojo_media_client.cc
index 318429904300bf8c6c3cf29aaf98cfaef1d4ad76..d0dd421a3270de53d55a45ac207a7d1cdb2c5e6e 100644
--- a/media/mojo/services/android_mojo_media_client.cc
+++ b/media/mojo/services/android_mojo_media_client.cc
@@ -40,6 +40,7 @@ std::unique_ptr<AudioDecoder> AndroidMojoMediaClient::CreateAudioDecoder(
std::unique_ptr<CdmFactory> AndroidMojoMediaClient::CreateCdmFactory(
service_manager::mojom::InterfaceProvider* interface_provider) {
+ DCHECK(interface_provider) << "InterfaceProvider is not bound!";
xhwang 2016/11/30 18:52:50 Given how flexible people can use mojo media, I im
slan 2016/12/01 00:47:22 To be honest, if I were a user of this interface,
return base::MakeUnique<AndroidCdmFactory>(
base::Bind(&CreateProvisionFetcher, interface_provider));
}

Powered by Google App Engine
This is Rietveld 408576698