Cast: Refactoring Cast API's

media/cast/cast_sender.h

 // Copyright 2013 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.
 //
 // This is the main interface for the cast sender. All configuration are done
 // at creation.
 //
 // The FrameInput and PacketReciever interfaces should normally be accessed from
 // the IO thread. However they are allowed to be called from any thread.

[Ami GONE FROM CHROMIUM, 2014/02/14 18:23:54]

no longer true

[mikhal1, 2014/02/18 19:20:43]

Code refactored.
On 2014/02/14 18:23:54, Ami Fischman (OOO Feb 19-23) wrote:

 
 #ifndef MEDIA_CAST_CAST_SENDER_H_
 #define MEDIA_CAST_CAST_SENDER_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/tick_clock.h"
 #include "base/time/time.h"
 #include "media/cast/cast_config.h"
 #include "media/cast/cast_environment.h"
 #include "media/cast/transport/cast_transport_sender.h"
 #include "media/filters/gpu_video_accelerator_factories.h"
 
 namespace media {
 class AudioBus;
+class GpuVideoAcceleratorFactories;
 class VideoFrame;
-}
 
-namespace media {
 namespace cast {
+class AudioSender;
+class VideoSender;
 
-// This Class is thread safe.
 class FrameInput : public base::RefCountedThreadSafe<FrameInput> {
  public:
   // The video_frame must be valid until the callback is called.
   // The callback is called from the main cast thread as soon as
   // the encoder is done with the frame; it does not mean that the encoded frame
   // has been sent out.
   virtual void InsertRawVideoFrame(
       const scoped_refptr<media::VideoFrame>& video_frame,
       const base::TimeTicks& capture_time) = 0;
 
   // The |audio_bus| must be valid until the |done_callback| is called.
   // The callback is called from the main cast thread as soon as the encoder is
   // done with |audio_bus|; it does not mean that the encoded data has been
   // sent out.
   virtual void InsertAudio(const AudioBus* audio_bus,
                            const base::TimeTicks& recorded_time,
                            const base::Closure& done_callback) = 0;
 
+  // Set the audio sender on which audio frames will be inserted to Cast. Must
+  // be called on the main thread.
+  // This function MUST be called before raw frames are inserted.
+  virtual void SetAudioSender(base::WeakPtr<AudioSender> audio_sender) = 0;
+
+  // Set the video sender on which video frames will be inserted to Cast. Must
+  // be called on the main thread.
+  // This function must be called before raw frames are inserted.
+  virtual void SetVideoSender(base::WeakPtr<VideoSender> video_sender) = 0;
+
  protected:
   virtual ~FrameInput() {}
 
  private:
   friend class base::RefCountedThreadSafe<FrameInput>;
 };
 
 // This Class is thread safe.
 // The provided CastTransportSender object will always be called from the main
 // cast thread.
-//  At least one of AudioSenderConfig and VideoSenderConfig have to be provided.
 class CastSender {
  public:
-  static CastSender* CreateCastSender(
+  static CastSender* Create(

[Ami GONE FROM CHROMIUM, 2014/02/14 18:23:54]

I think you missed this comment:

This doesn't doco ownership semantics of the returned object.
I believe this should instead return a scoped_ptr<CastSender> which would
obviate the need for a comment.

[mikhal1, 2014/02/18 19:20:43]

Done, and matched receiver and transport.
On 2014/02/14 18:23:54, Ami Fischman (OOO Feb 19-23) wrote:

       scoped_refptr<CastEnvironment> cast_environment,
-      const AudioSenderConfig* audio_config,
-      const VideoSenderConfig* video_config,
-      const scoped_refptr<GpuVideoAcceleratorFactories>& gpu_factories,
-      const CastInitializationCallback& cast_initialization,
+      const CastInitializationCallback& cast_initialization_cb,
       transport::CastTransportSender* const transport_sender);
 
   virtual ~CastSender() {}
 
   // All audio and video frames for the session should be inserted to this
   // object.
   // Can be called from any thread.
   virtual scoped_refptr<FrameInput> frame_input() = 0;
 
   // All RTCP packets for the session should be inserted to this object.
   // Can be called from any thread.
   virtual transport::PacketReceiverCallback packet_receiver() = 0;
+
+  // Initialize the audio stack. Must be called in order to send audio frames.
+  // This function MUST be called before raw frames are inserted.
+  virtual void InitializeAudio(const AudioSenderConfig& audio_config) = 0;
+
+  // Initialize the video stack. Must be called in order to send video frames.
+  // This function must be called before raw frames are inserted.
+  virtual void InitializeVideo(
+      const VideoSenderConfig& video_config,
+      const scoped_refptr<GpuVideoAcceleratorFactories>& gpu_factories) = 0;
 };
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_CAST_SENDER_H_