Cast: Flow hw encoder initialization error to extensions API

media/cast/sender/video_sender.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_CAST_SENDER_VIDEO_SENDER_H_
 #define MEDIA_CAST_SENDER_VIDEO_SENDER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 19 matching lines @@
 // packets, congestion control, video encoder, parsing and sending of
 // RTCP packets.
 // Additionally it posts a bunch of delayed tasks to the main thread for various
 // timeouts.
 class VideoSender : public FrameSender,
                     public base::NonThreadSafe,
                     public base::SupportsWeakPtr<VideoSender> {
  public:
   VideoSender(scoped_refptr<CastEnvironment> cast_environment,
               const VideoSenderConfig& video_config,
+              const CastInitializationCallback& initialization_cb,
               const CreateVideoEncodeAcceleratorCallback& create_vea_cb,
               const CreateVideoEncodeMemoryCallback& create_video_encode_mem_cb,
               CastTransportSender* const transport_sender);
 
   virtual ~VideoSender();
 
-  CastInitializationStatus InitializationResult() const {
-    return cast_initialization_status_;

[hubbe, 2014/09/09 19:11:03]

Can we get rid of this function on the audio side too?
And maybe the variable too?

[Alpha Left Google, 2014/09/09 19:41:18]

Audio initialization status actually is in use, it checks the configuration to
make sure it's valid.

-  }
-
   // Note: It is not guaranteed that |video_frame| will actually be encoded and
   // sent, if VideoSender detects too many frames in flight.  Therefore, clients
   // should be careful about the rate at which this method is called.
   //
   // Note: It is invalid to call this method if InitializationResult() returns
   // anything but STATUS_VIDEO_INITIALIZED.
   void InsertRawVideoFrame(const scoped_refptr<media::VideoFrame>& video_frame,
                            const base::TimeTicks& capture_time);
 
  protected:
   // Protected for testability.
   void OnReceivedCastFeedback(const RtcpCastMessage& cast_feedback);
 
  private:
   // Returns true if there are too many frames in flight, or if the media
   // duration of the frames in flight would be too high by sending the next
   // frame.  The latter metric is determined from the given |capture_time|
   // for the next frame to be encoded and sent.
   bool ShouldDropNextFrame(base::TimeTicks capture_time) const;
 
   // Called by the |video_encoder_| with the next EncodeFrame to send.
   void SendEncodedVideoFrame(int requested_bitrate_before_encode,
                              scoped_ptr<EncodedFrame> encoded_frame);
+
+  // Called when the encoder is initialized or has failed to initialize.
+  void OnEncoderInitialized(
+      const CastInitializationCallback& initialization_cb,
+      CastInitializationStatus status);
+
   // If this value is non zero then a fixed value is used for bitrate.
   // If external video encoder is used then bitrate will be fixed to
   // (min_bitrate + max_bitrate) / 2.
   const size_t fixed_bitrate_;
 
   // Encodes media::VideoFrame images into EncodedFrames.  Per configuration,
   // this will point to either the internal software-based encoder or a proxy to
   // a hardware-based encoder.
   scoped_ptr<VideoEncoder> video_encoder_;
 
   // The number of frames currently being processed in |video_encoder_|.
   int frames_in_encoder_;
 
   // When we get close to the max number of un-acked frames, we set lower
   // the bitrate drastically to ensure that we catch up. Without this we
   // risk getting stuck in a catch-up state forever.
   CongestionControl congestion_control_;
 
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<VideoSender> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoSender);
 };
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_SENDER_VIDEO_SENDER_H_