Introduce MojoRendererFactory::ServiceProvider.

media/mojo/services/mojo_renderer_factory.h

 // 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.
 
 #ifndef MEDIA_MOJO_SERVICES_MOJO_RENDERER_FACTORY_H_
 #define MEDIA_MOJO_SERVICES_MOJO_RENDERER_FACTORY_H_
 
 #include "media/base/media_export.h"
 #include "media/base/renderer_factory.h"
-#include "mojo/public/interfaces/application/service_provider.mojom.h"
+#include "media/mojo/interfaces/media_renderer.mojom.h"
+#include "mojo/public/cpp/bindings/interface_ptr.h"
 
 namespace media {
 
 // The default factory class for creating MojoRendererImpl.
 class MEDIA_EXPORT MojoRendererFactory : public RendererFactory {
  public:
-  MojoRendererFactory(mojo::ServiceProviderPtr renderer_service_provider);
+  class ServiceProvider {

[jamesr, 2014/12/16 01:47:37]

comment?

[xhwang, 2014/12/16 05:05:11]

Done.

+   public:
+    ServiceProvider(){};

[jamesr, 2014/12/16 01:47:37]

no ;, spacing seems odd. did you run 'git cl format'?

[xhwang, 2014/12/16 05:05:11]

Yep, done by git cl format. Fixed manually.

+    virtual ~ServiceProvider(){};
+    virtual void ConnectToService(
+        mojo::InterfacePtr<mojo::MediaRenderer>* media_renderer_ptr) = 0;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(ServiceProvider);
+  };
+
+  MojoRendererFactory(scoped_ptr<ServiceProvider> service_provider);

[jamesr, 2014/12/16 01:47:37]

explicit

[xhwang, 2014/12/16 05:05:11]

Done.

   ~MojoRendererFactory() final;
 
   scoped_ptr<Renderer> CreateRenderer(
       const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
       AudioRendererSink* audio_renderer_sink) final;
 
  private:
-  mojo::ServiceProviderPtr renderer_service_provider_;
+  scoped_ptr<ServiceProvider> service_provider_;
 
   DISALLOW_COPY_AND_ASSIGN(MojoRendererFactory);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_MOJO_SERVICES_MOJO_RENDERER_FACTORY_H_