Hook up EncodedVideoSource on the browser side

media/video/capture/fake_video_capture_device.h

 // Copyright (c) 2012 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.
 
 // Implementation of a fake VideoCaptureDevice class. Used for testing other
 // video capture classes when no real hardware is available.
 
 #ifndef MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_H_
 #define MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread.h"
 #include "media/video/capture/video_capture_device.h"
 
 namespace media {
 
 class MEDIA_EXPORT FakeVideoCaptureDevice : public VideoCaptureDevice {
  public:
-  static VideoCaptureDevice* Create(const Name& device_name);
+  static VideoCaptureDevice* Create(const Name& device_name,
+                                    bool encoded_capture);

[Ami GONE FROM CHROMIUM, 2013/06/18 18:35:55]

indent is off here and elsewhere in the CL.  Please make a pass.

[sheu, 2013/08/22 22:40:31]

Not sure what you mean here?

   virtual ~FakeVideoCaptureDevice();
   // Used for testing. This will make sure the next call to Create will
   // return NULL;
   static void SetFailNextCreate();
 
-  static void GetDeviceNames(Names* device_names);
+  static void GetDeviceNames(Names* device_names, bool encoded_capture);

[Ami GONE FROM CHROMIUM, 2013/06/18 18:35:55]

Why pass this in if not to inspect it in Create() (and thus not have to pass it
to Create)?

[sheu, 2013/08/22 22:40:31]

Not quite following here.  GetDeviceNames() constructs a different name if it's
encoding, and Create() creates an encoding fake device if it's encoding.

 
   // VideoCaptureDevice implementation.
+  virtual VideoEncodingCapabilities GetEncodingCapabilities() OVERRIDE;
+  virtual void TryConfigureEncodedBitstream(
+      const RuntimeVideoEncodingParameters& params) OVERRIDE;
   virtual void Allocate(int width,
                         int height,
                         int frame_rate,
                         VideoCaptureDevice::EventHandler* observer) OVERRIDE;
   virtual void Start() OVERRIDE;
   virtual void Stop() OVERRIDE;
   virtual void DeAllocate() OVERRIDE;
   virtual const Name& device_name() OVERRIDE;
 
  private:
   // Flag indicating the internal state.
   enum InternalState {
     kIdle,
     kAllocated,
     kCapturing,
     kError
   };
-  explicit FakeVideoCaptureDevice(const Name& device_name);
+  explicit FakeVideoCaptureDevice(const Name& device_name,
+                                  bool encoded_capture);
 
   // Called on the capture_thread_.
   void OnCaptureTask();
 
   Name device_name_;
+  bool encoded_capture_;
   VideoCaptureDevice::EventHandler* observer_;
   InternalState state_;
   base::Thread capture_thread_;
   int frame_size_;
   scoped_ptr<uint8[]> fake_frame_;
+  scoped_refptr<media::EncodedBitstreamBuffer> fake_encoded_bitstream_buffer_;

[Ami GONE FROM CHROMIUM, 2013/06/18 18:35:55]

unused

[sheu, 2013/08/22 22:40:31]

Done.

   int frame_count_;
   int frame_width_;
   int frame_height_;
 
   static bool fail_next_create_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(FakeVideoCaptureDevice);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_H_