Extract VideoCaptureDeviceFactory out of VideoCaptureDevice and use for File and FakeVCD.

media/video/capture/fake_video_capture_device_factory.h

Index: media/video/capture/fake_video_capture_device_factory.h
diff --git a/media/video/capture/fake_video_capture_device_factory.h b/media/video/capture/fake_video_capture_device_factory.h
new file mode 100644
index 0000000000000000000000000000000000000000..2f1c43c293ac3bc75849a690fa5d8c90656bad38
--- /dev/null
+++ b/media/video/capture/fake_video_capture_device_factory.h
@@ -0,0 +1,40 @@
+// 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.
+
+// Implementation of a fake VideoCaptureDeviceFactory class.
+
+#ifndef MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_FACTORY_H_
+#define MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_FACTORY_H_
+
+#include "media/video/capture/video_capture_device_factory.h"
+
+namespace media {
+
+class MEDIA_EXPORT FakeVideoCaptureDeviceFactory :
+    public VideoCaptureDeviceFactory {
+ public:
+  FakeVideoCaptureDeviceFactory();
+  virtual ~FakeVideoCaptureDeviceFactory() {};

[tommi (sloooow) - chröme, 2014/04/23 14:33:59]

remove ;

[mcasas, 2014/04/23 15:13:30]

Done.

+
+  virtual scoped_ptr<VideoCaptureDevice> Create(
+      const VideoCaptureDevice::Name& device_name) OVERRIDE;
+  virtual void GetDeviceNames(VideoCaptureDevice::Names* device_names) OVERRIDE;
+  virtual void GetDeviceSupportedFormats(
+      const VideoCaptureDevice::Name& device,
+      VideoCaptureFormats* supported_formats) OVERRIDE;
+
+  void set_number_of_devices(size_t number_of_devices) {
+    number_of_devices_ = number_of_devices;
+  }
+  int32 number_of_devices() {

[tommi (sloooow) - chröme, 2014/04/23 14:33:59]

nit: fits in one line?  ... actually, would it make sense to DCHECK correct
threadness here and in the setter?

[mcasas, 2014/04/23 15:13:30]

Added DCHECK(thread_checker_.CalledOnValidThread());
(checks for Device Thread).

+    return number_of_devices_;
+  }
+
+ private:
+  int32 number_of_devices_;

[tommi (sloooow) - chröme, 2014/04/23 14:33:59]

why int32 and not int?

[mcasas, 2014/04/23 15:13:30]

Copied from previous implementation.

+};
+
+}  // namespace media
+
+#endif  // MEDIA_VIDEO_CAPTURE_FAKE_VIDEO_CAPTURE_DEVICE_FACTORY_H_