media: Fix lifetime of InterfaceFactoryImpl and created impls

media/mojo/services/interface_factory_impl.h

 // Copyright 2015 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.
 
 #ifndef MEDIA_MOJO_SERVICES_INTERFACE_FACTORY_IMPL_H_
 #define MEDIA_MOJO_SERVICES_INTERFACE_FACTORY_IMPL_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "media/mojo/interfaces/interface_factory.mojom.h"
 #include "media/mojo/services/mojo_cdm_service_context.h"
+#include "media/mojo/services/strong_binding_set.h"
 #include "services/service_manager/public/cpp/connector.h"
 #include "services/service_manager/public/cpp/service_context_ref.h"
 
 namespace service_manager {
 namespace mojom {
 class InterfaceProvider;
 }
 }
 
 namespace media {
@@ skipping 23 matching lines @@
 #if defined(ENABLE_MOJO_RENDERER)
   RendererFactory* GetRendererFactory();
 #endif  // defined(ENABLE_MOJO_RENDERER)
 
 #if defined(ENABLE_MOJO_CDM)
   CdmFactory* GetCdmFactory();
 #endif  // defined(ENABLE_MOJO_CDM)
 
   MojoCdmServiceContext cdm_service_context_;
 
+#if defined(ENABLE_MOJO_AUDIO_DECODER)
+  StrongBindingSet<mojom::AudioDecoder> audio_decoder_bindings_;
+#endif  // defined(ENABLE_MOJO_AUDIO_DECODER)
+
+#if defined(ENABLE_MOJO_AUDIO_DECODER)

[Julien Isorce Samsung, 2016/11/30 16:30:44]

Should be ENABLE_MOJO_VIDEO_DECODER

[xhwang, 2016/11/30 17:38:49]

oops, thanks!

I'll fix it now.

OOC, are you using mojo video decoder?

+  StrongBindingSet<mojom::VideoDecoder> video_decoder_bindings_;
+#endif  // defined(ENABLE_MOJO_AUDIO_DECODER)
+
 #if defined(ENABLE_MOJO_RENDERER)
   std::unique_ptr<RendererFactory> renderer_factory_;
+  StrongBindingSet<mojom::Renderer> renderer_bindings_;
 #endif  // defined(ENABLE_MOJO_RENDERER)
 
 #if defined(ENABLE_MOJO_CDM)
   std::unique_ptr<CdmFactory> cdm_factory_;
   service_manager::mojom::InterfaceProviderPtr interfaces_;
+  StrongBindingSet<mojom::ContentDecryptionModule> cdm_bindings_;
 #endif  // defined(ENABLE_MOJO_CDM)
 
   scoped_refptr<MediaLog> media_log_;
   std::unique_ptr<service_manager::ServiceContextRef> connection_ref_;
   MojoMediaClient* mojo_media_client_;
 
   DISALLOW_COPY_AND_ASSIGN(InterfaceFactoryImpl);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_MOJO_SERVICES_INTERFACE_FACTORY_IMPL_H_