[WebRTC] Improve documentation (#6347)
authorHaesu Gwon <haesu.gwon@samsung.com>
Wed, 25 Sep 2024 04:52:57 +0000 (13:52 +0900)
committerGitHub <noreply@github.com>
Wed, 25 Sep 2024 04:52:57 +0000 (13:52 +0900)
16 files changed:
src/Tizen.Multimedia.Remoting/WebRTC/MediaCameraSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaCustomSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaFileSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaMicrophoneSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaNullSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaPacketBufferStatusChangedEventArgs.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaPacketSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaPacketSourceConfiguration.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaScreenSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaStreamTrack.cs
src/Tizen.Multimedia.Remoting/WebRTC/MediaTestSource.cs
src/Tizen.Multimedia.Remoting/WebRTC/WebRTC.cs
src/Tizen.Multimedia.Remoting/WebRTC/WebRTCDataChannelErrorOccurredEventArgs.cs
src/Tizen.Multimedia.Remoting/WebRTC/WebRTCEnums.cs
src/Tizen.Multimedia.Remoting/WebRTC/WebRTCFrameEncodedEventArgs.cs
src/Tizen.Multimedia.Remoting/WebRTC/WebRTCTrackAddedEventArgs.cs

index 50a34111a76487581facbc89d9b973378e2b25d0..2efc12409cb67b2c760b7801d69abd057c5c0ff1 100755 (executable)
@@ -21,7 +21,7 @@ using NativeWebRTC = Interop.NativeWebRTC;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a camera source.
+    /// Represents a media source that makes video data using camera device internally.
     /// </summary>
     /// <remarks>The camera privilege(http://tizen.org/privilege/camera) is required.</remarks>
     /// <seealso cref="WebRTC.AddSource"/>
@@ -36,7 +36,7 @@ namespace Tizen.Multimedia.Remoting
         public MediaCameraSource() : base(MediaType.Video) {}
 
         /// <summary>
-        /// Gets or sets the camera device id.
+        /// Gets or sets the camera device id that identifies each camera device.
         /// </summary>
         /// <value>A value that specifies the camera device id.</value>
         /// <exception cref="InvalidOperationException">MediaSource is not attached yet.</exception>
index 4f42fd930f172010de3ec4f07dd2023ee7e952e1..112beb5a5d3e6dbde18f6836be9f86dd87351584 100755 (executable)
@@ -22,8 +22,11 @@ using NativeWebRTC = Interop.NativeWebRTC;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a audio, video custom source.
+    /// Represents a media source that makes audio, video data internally.
     /// </summary>
+    /// <remarks>
+    /// This supports the product infrastructure and is not intended to be used directly from application code.
+    /// </remarks>
     /// <seealso cref="WebRTC.AddSource"/>
     /// <seealso cref="WebRTC.AddSources"/>
     /// <since_tizen> 9 </since_tizen>
index 40e6ea36f0825e3a2f6736062df634b9683fa74c..4cc04bb1b7bf7ce7b00f48bb6374e74a9c1159bc 100755 (executable)
@@ -21,7 +21,7 @@ using NativeWebRTC = Interop.NativeWebRTC;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a media source with contents read from a file.
+    /// Represents a media source that reads media data from a file.
     /// </summary>
     /// <remarks>
     /// Depending on where the source file is located either the media storage privilege (http://tizen.org/privilege/mediastorage) is required or<br/>
index 8823ef89ef19af082b09923b58f5edd8c17cb761..7c0cf06f7ef399e046f94526b92ba5d72490e2a5 100755 (executable)
@@ -21,7 +21,7 @@ using NativeWebRTC = Interop.NativeWebRTC;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a microphone source.
+    /// Represents a media source that makes audio data using microphone.
     /// </summary>
     /// <remarks>The recorder privilege(http://tizen.org/privilege/recorder) is required.</remarks>
     /// <seealso cref="WebRTC.AddSource"/>
index cdd1d34c45ff3ecce29ba65dec0401f32aa20cf4..586aede9c94e4ea29713c3519f9aec6c2fa8df7f 100755 (executable)
@@ -21,7 +21,9 @@ using NativeWebRTC = Interop.NativeWebRTC;
 
 namespace Tizen.Multimedia.Remoting
 {
-    /// <summary>Represents a null source.</summary>
+    /// <summary>
+    /// Represents a media source that only receives media streams from peer.
+    /// </summary>
     /// <remarks>
     /// If you add this source, WebRTC only receives media stream.<br/>
     /// <see cref="TransceiverDirection"/> is set <see cref="TransceiverDirection.RecvOnly"/> by default.
@@ -84,7 +86,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Retrieves the supported transceiver codecs.
+        /// Retrieves the supported transceiver codecs for receiving media stream.
         /// </summary>
         /// <param name="type">The media type.</param>
         /// <returns>The supported transceiver codecs.</returns>
index 2f293e478b7e22442defb5c2c69dd9f6a99c67f1..50a93a2cf959ca7d27540a9d138f5f07dce1f005 100755 (executable)
@@ -31,14 +31,14 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the source ID.
+        /// Gets the source ID of media packet.
         /// </summary>
         /// <value>The source ID.</value>
         /// <since_tizen> 9 </since_tizen>
         public uint SourceId { get; }
 
         /// <summary>
-        /// Gets the media packet buffer status.
+        /// Gets the media packet buffer status that indicates underrun or overflow.
         /// </summary>
         /// <value>The media packet buffer status.</value>
         /// <since_tizen> 9 </since_tizen>
index 09fea0058edea5044b3150607dcf3e289c778c2d..8f31f186b2cf1d2f982341b652b7c777de79b2a0 100755 (executable)
@@ -23,7 +23,7 @@ using static Interop;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a media packet source.
+    /// Represents a media source that wraps media data into MediaPacket.
     /// </summary>
     /// <seealso cref="WebRTC.AddSource"/>
     /// <seealso cref="WebRTC.AddSources"/>
index d411a01fad1b23d8cc6cb049e46fcd639fde9108..3e0edbded23351d264f870802dc04c9e96dd4382 100755 (executable)
@@ -20,7 +20,7 @@ using static Interop;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Provides means to configure properties and handle events for <see cref="MediaPacketSource"/>.
+    /// Provides means to configure properties and handle event for <see cref="MediaPacketSource"/>.
     /// </summary>
     /// <seealso cref="MediaPacketSource"/>
     /// <since_tizen> 9 </since_tizen>
@@ -35,7 +35,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Occurs when the buffer underruns or overflows.
+        /// Occurs when the status of buffer is underruns or overflows.
         /// </summary>
         /// <remarks>The event handler will be executed on an internal thread.</remarks>
         /// <since_tizen> 9 </since_tizen>
index 391a13a5aa35c0a473bde9a755ce935cb776e6a0..ad51dfb8fe04d824ec65273499242d2b69c43c4c 100755 (executable)
@@ -21,7 +21,7 @@ using NativeWebRTC = Interop.NativeWebRTC;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents a screen source.
+    /// Represents a media source to share screen data.
     /// </summary>
     /// <remarks>
     /// MediaScreenSource is not allowed to be used by third-party applications due to the security reasons. (Since Tizen 7.0)
index be62bebf2361ba0df3b9b9bff9935c028cba081d..fcd12909121c6ece04003b8af78b94eea954cea9 100755 (executable)
@@ -21,7 +21,7 @@ using static Interop;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Provides the ability to control audio/video track.
+    /// Provides the ability to control audio or video track from peer.
     /// </summary>
     /// <since_tizen> 9 </since_tizen>
     public sealed class MediaStreamTrack : IDisplayable<WebRTCErrorCode>
@@ -38,7 +38,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the the of media stream track.
+        /// Gets the type of media stream track.
         /// </summary>
         /// <value><see cref="MediaType"/></value>
         /// <since_tizen> 9 </since_tizen>
@@ -55,7 +55,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets or sets the display to show remote video.
+        /// Gets or sets the display to show video data from peer.
         /// </summary>
         /// <value>A <see cref="Multimedia.Display"/> that specifies the display.</value>
         /// <remarks>
index d3226b83c7dd625ce386a9763dacca9d98690617..a7c514770db7bb029eca64e80207c308276ad7c9 100755 (executable)
@@ -21,7 +21,7 @@ using static Interop;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Represents an audio or a video test source.
+    /// Represents a media source that makes test audio or video stream.
     /// </summary>
     /// <seealso cref="WebRTC.AddSource"/>
     /// <seealso cref="WebRTC.AddSources"/>
index 0590d0f0faa2cddaba275b2b4519cf3fc7b2ad38..a6d3ce2857e5e7f042f3730417b1907d3a0eae78 100755 (executable)
@@ -129,7 +129,7 @@ namespace Tizen.Multimedia.Remoting
         #endregion
 
         /// <summary>
-        /// Starts the WebRTC.
+        /// Starts the WebRTC with specific media source.
         /// </summary>
         /// <remarks>
         /// The WebRTC must be in the <see cref="WebRTCState.Idle"/> state.<br/>
@@ -150,7 +150,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Starts the WebRTC asynchronously.
+        /// Starts the WebRTC asynchronously with specific media source.
         /// </summary>
         /// <remarks>
         /// The WebRTC must be in the <see cref="WebRTCState.Idle"/> state.<br/>
@@ -228,7 +228,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Creates SDP offer asynchronously to start a new WebRTC connection to a remote peer.
+        /// Creates SDP(Session Description Protocol) offer asynchronously to start a new WebRTC connection to a remote peer.
         /// </summary>
         /// <remarks>
         /// The WebRTC must be in the <see cref="WebRTCState.Negotiating"/> or <see cref="WebRTCState.Playing"/>(Since API Level 12)
@@ -267,7 +267,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Creates SDP answer asynchronously with option to an offer received from a remote peer.
+        /// Creates SDP(Session Description Protocol) answer asynchronously with option to an offer received from a remote peer.
         /// </summary>
         /// <remarks>
         /// The WebRTC must be in the <see cref="WebRTCState.Negotiating"/> or <see cref="WebRTCState.Playing"/>(Since API Level 12)
@@ -354,7 +354,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Sets the session description of the remote peer's current offer or answer.
+        /// Sets the offer or answer session description from the current remote peer.
         /// </summary>
         /// <remarks>
         /// The WebRTC must be in the <see cref="WebRTCState.Negotiating"/> or <see cref="WebRTCState.Playing"/>(Since API Level 12)
@@ -378,7 +378,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the session description of the remote peer's current offer or answer.
+        /// Gets the offer or answer session description from the current remote peer.
         /// </summary>
         /// <value>The remote session description string</value>
         /// <exception cref="ObjectDisposedException">The WebRTC has already been disposed.</exception>
@@ -445,7 +445,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Adds media source.
+        /// Adds media source to the current WebRTC.
         /// </summary>
         /// <remarks>
         /// This method does not throw state exception anymore(Since API Level 12). It can be called in any state.<br/>
@@ -491,7 +491,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Adds media sources.
+        /// Adds media sources from the current WebRTC.
         /// </summary>
         /// <remarks>
         /// This method does not throw state exception anymore(Since API Level 12). It can be called in any state.<br/>
@@ -538,7 +538,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Removes media source.
+        /// Removes media source from the current WebRTC.
         /// </summary>
         /// <remarks>
         /// This method does not throw state exception anymore(Since API Level 12). It can be called in any state.<br/>
@@ -571,7 +571,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Removes media sources.
+        /// Removes media sources from the current WebRTC.
         /// </summary>
         /// <remarks>
         /// This method does not throw state exception anymore(Since API Level 12). It can be called in any state.<br/>
@@ -598,7 +598,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Sets a turn server.
+        /// Sets a turn server for signalling with remote peer which cannot be connected directly.
         /// </summary>
         /// <exception cref="ArgumentNullException">The <paramref name="turnServer"/> is null.</exception>
         /// <exception cref="ObjectDisposedException">The WebRTC has already been disposed.</exception>
@@ -617,7 +617,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Sets turn servers.
+        /// Sets turn servers for signalling with remote peer which cannot be connected directly.
         /// </summary>
         /// <exception cref="ArgumentNullException">The one of <paramref name="turnServers"/> is null.</exception>
         /// <exception cref="ObjectDisposedException">The WebRTC has already been disposed.</exception>
index ddae8b4d3e01ae28296ed0c948aebc37bcdb0df6..8963ea7c93f074225d47fb0630c7c24e5c65aa34 100755 (executable)
@@ -30,7 +30,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the error.
+        /// Gets the error of data channel.
         /// </summary>
         /// <value>The error.</value>
         /// <since_tizen> 9 </since_tizen>
index 24796c35b57a8f3ca1ff9c13a0972be377950be9..61cafb38f2a7744c5f89eb48d84ddde1044b27e8 100755 (executable)
@@ -20,7 +20,7 @@ using System.ComponentModel;
 namespace Tizen.Multimedia.Remoting
 {
     /// <summary>
-    /// Specifies errors.
+    /// Specifies WebRTC errors.
     /// </summary>
     /// <seealso cref="WebRTC.ErrorOccurred"/>
     /// <seealso cref="WebRTCErrorOccurredEventArgs"/>
@@ -361,7 +361,7 @@ namespace Tizen.Multimedia.Remoting
     }
 
     /// <summary>
-    /// Specifies the display type.
+    /// Specifies the display mode that indicates the way of displaying video data.
     /// </summary>
     /// <since_tizen> 9 </since_tizen>
     public enum WebRTCDisplayMode
@@ -383,7 +383,8 @@ namespace Tizen.Multimedia.Remoting
     }
 
     /// <summary>
-    /// Specifies the bundle policy.
+    /// Specifies the bundle policy that affects which media tracks are negotiated if the remote endpoint is not bundle-aware,
+    /// and what ICE candidates are gathered.
     /// </summary>
     /// <remarks>
     /// The details of bundle policy enum is described in https://www.w3.org/TR/webrtc/#rtcbundlepolicy-enum.
index 455009c51547e71b86ec5d98ea1962025bd79bef..aea9583b98060159e8814f4ded52582d8186aa60 100755 (executable)
@@ -31,7 +31,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the track information.
+        /// Gets the information of media stream track from remote peer.
         /// </summary>
         /// <value>The media type.</value>
         /// <since_tizen> 9 </since_tizen>
index e95c70eb1138fdcbd2be8c1c6fc0ed83e118093d..dce812cca786d36b538f11893c58d9c392e31102 100755 (executable)
@@ -30,7 +30,7 @@ namespace Tizen.Multimedia.Remoting
         }
 
         /// <summary>
-        /// Gets the media type.
+        /// Gets the media stream track from reemote peer.
         /// </summary>
         /// <value>The media type.</value>
         /// <since_tizen> 9 </since_tizen>