[Mojo Video Capture] Introduce abstraction VideoCaptureSystem

media/capture/video/video_capture_device_info.cc

Index: media/capture/video/video_capture_device_info.cc
diff --git a/media/capture/video/video_capture_device_info.cc b/media/capture/video/video_capture_device_info.cc
new file mode 100644
index 0000000000000000000000000000000000000000..50f999ba20df9d9244418ae140cbfcb8f0c7c0fd
--- /dev/null
+++ b/media/capture/video/video_capture_device_info.cc
@@ -0,0 +1,23 @@
+// Copyright 2017 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.
+
+#include "media/capture/video/video_capture_device_info.h"
+
+namespace media {
+
+VideoCaptureDeviceInfo::VideoCaptureDeviceInfo() = default;
+
+VideoCaptureDeviceInfo::VideoCaptureDeviceInfo(
+    media::VideoCaptureDeviceDescriptor descriptor)
+    : descriptor(descriptor) {}
+
+VideoCaptureDeviceInfo::VideoCaptureDeviceInfo(
+    const VideoCaptureDeviceInfo& other) = default;
+
+VideoCaptureDeviceInfo::~VideoCaptureDeviceInfo() = default;
+
+VideoCaptureDeviceInfo& VideoCaptureDeviceInfo::operator=(
+    const VideoCaptureDeviceInfo& other) = default;

[mcasas, 2017/04/04 18:38:41]

Since all this methods are =default, why not
moving them to the header file and nuking this
file?

[chfremer, 2017/04/04 19:05:33]

Because this struct is considered "complex" (because of its members) our build
requires non-inline constructor, destructor, copy constructor and assignment
operator.

I don't know any way to do this without a .cc file, but would love it if there
was.

+
+}  // namespace media