Adding error signalling from device to VideocaptureManager to relay up to MediaStream and WebKit.

content/common/media/media_stream_options.cc

Index: content/common/media/media_stream_options.cc
===================================================================
--- content/common/media/media_stream_options.cc	(revision 0)
+++ content/common/media/media_stream_options.cc	(revision 0)
@@ -0,0 +1,27 @@
+// Copyright (c) 2011 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 "content/common/media/media_stream_options.h"
+
+namespace media_stream {
+
+StreamDeviceInfo::StreamDeviceInfo()
+    : stream_type(kNoService),
+      name(),

[John Knottenbelt, 2011/06/16 15:20:43]

is it necessary to list name and device_id here (they will be invoked with
default constructors anyway)

[mflodman1, 2011/06/20 19:48:03]

Done.
I'm used to do it that way to indicate it's not forgotten and I didn't see any
style guide about it. It doesn't seem to be the Chrome style though, so changed.

+      device_id(),
+      in_use(false),
+      session_id(kNoId) {}
+
+StreamDeviceInfo::StreamDeviceInfo(MediaStreamType service_param,
+                                   const std::string name_param,
+                                   const std::string device_param,
+                                   bool opened)
+    : stream_type(service_param),
+      name(name_param),
+      device_id(device_param),
+      in_use(opened),
+      session_id(kNoId) {}
+
+}  // namespace media_stream
+

[scherkus (not reviewing), 2011/06/17 03:03:52]

nit: get rid of blank line

[mflodman1, 2011/06/20 19:48:03]

Done.