media: Add GpuMojoMediaClient.

media/mojo/services/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//media/media_options.gni")
 import("//services/service_manager/public/cpp/service.gni")
 import("//services/service_manager/public/service_manifest.gni")
 import("//testing/test.gni")
 
 config("mojo_media_config") {
@@ skipping 34 matching lines @@
     }
   }
 }
 
 component("services") {
   output_name = "media_mojo_services"
 
   sources = [
     "demuxer_stream_provider_shim.cc",
     "demuxer_stream_provider_shim.h",
+    "gpu_mojo_media_client.cc",
+    "gpu_mojo_media_client.h",

[sandersd (OOO until July 31), 2016/11/22 23:12:16]

Main deps concern is here, since the GpuMojoMediaClient may some day take on a
variety of deps to implement its factory. The main alternative is to move this
to //content/gpu, but then //content/gpu would start growing media deps.

[xhwang, 2016/11/23 06:58:03]

Thanks for bringing this up. Either way it looks good to me. I guess this is
totally up to you.

Also, content depends on media and media/mojo. I don't see anything wrong to
have content/gpu depending on media stuff :)

That being said, it won't hurt to add a content/gpu owner to confirm :)

[sandersd (OOO until July 31), 2016/11/23 21:21:02]

In that case I'll just keep going, and we can worry about this when there are
specific deps worth discussing ;-)

     "interface_factory_impl.cc",
     "interface_factory_impl.h",
     "media_mojo_export.h",
     "media_service.cc",
     "media_service.h",
     "media_service_factory.cc",
     "media_service_factory.h",
     "mojo_audio_decoder_service.cc",
     "mojo_audio_decoder_service.h",
     "mojo_cdm_allocator.cc",
@@ skipping 117 matching lines @@
   data_deps = [
     ":media",
     ":pipeline_test_manifest",
   ]
 }
 
 service_manifest("pipeline_test_manifest") {
   name = "media_pipeline_integration_unittests"
   source = "pipeline_test_manifest.json"
 }