media: Support MojoMediaApplication in GPU process.

content/gpu/gpu_process_control_impl.cc

Index: content/gpu/gpu_process_control_impl.cc
diff --git a/content/gpu/gpu_process_control_impl.cc b/content/gpu/gpu_process_control_impl.cc
index c087c8319fe582fed151d6990f4552324924d49a..cb3175b8bfc0bbd0272744fa8e289428f946b451 100644
--- a/content/gpu/gpu_process_control_impl.cc
+++ b/content/gpu/gpu_process_control_impl.cc
@@ -4,7 +4,12 @@
 
 #include "content/gpu/gpu_process_control_impl.h"
 
-#include "base/logging.h"
+#if defined(ENABLE_MOJO_MEDIA_IN_GPU_PROCESS)
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "media/mojo/services/mojo_media_application.h"
+#include "mojo/shell/static_application_loader.h"
+#endif
 
 namespace content {
 
@@ -14,9 +19,12 @@ GpuProcessControlImpl::~GpuProcessControlImpl() {}
 
 void GpuProcessControlImpl::RegisterApplicationLoaders(
     URLToLoaderMap* url_to_loader_map) {
-  // TODO(xhwang): Support MojoMediaApplication here.
-  // See http://crbug.com/521755
-  NOTIMPLEMENTED();
+#if defined(ENABLE_MOJO_MEDIA_IN_GPU_PROCESS)
+  (*url_to_loader_map)[GURL("mojo:media")] =

[piman, 2015/10/10 00:33:08]

nit: I see "mojo:media" is used in a lot of places, it sounds like it's worth
making it a constant somewhere?

[xhwang, 2015/10/12 18:05:02]

That is what I did in the old code. But it seems it's introducing some
unnecessary dependencies. I don't have a good solution now and it seems all
other mojo apps are using the string directly as well...

rockot: Any good recommendations from mandoline's POV?

+      new mojo::shell::StaticApplicationLoader(
+          base::Bind(&media::MojoMediaApplication::CreateApp),
+          base::Bind(&base::DoNothing));
+#endif
 }
 
 }  // namespace content