From 82ca531885e3a6913984862cc676d5a59f5da586 Mon Sep 17 00:00:00 2001
From: Haesu Gwon <haesu.gwon@samsung.com>
Date: Wed, 2 Oct 2024 18:38:05 +0900
Subject: [PATCH] [MediaPlayer] Improve documentation (#6357)

* [MediaPlayer] Improve documentation
---
 .../Player/AdaptiveVariants.cs                |  6 ++--
 .../Player/AudioDataDecodedEventArgs.cs       |  2 +-
 .../Player/AudioEffect.cs                     |  4 +--
 .../Player/AudioOffload.cs                    |  4 +--
 .../Player/EqualizerBand.cs                   |  4 +--
 .../Player/MediaBufferSource.cs               |  7 ++--
 ...MediaStreamBufferStatusChangedEventArgs.cs |  2 +-
 .../Player/MediaStreamConfiguration.cs        |  8 ++---
 .../MediaStreamSeekingOccurredEventArgs.cs    |  8 ++++-
 .../Player/MediaStreamSource.cs               |  6 ++--
 .../Player/MediaUriSource.cs                  |  8 ++---
 .../Player/PlaybackInterruptedEventArgs.cs    |  2 +-
 .../Player/Player.Properties.cs               | 35 +++++++++----------
 .../Player/Player.cs                          | 28 +++++++--------
 .../Player/PlayerDisplaySettings.cs           |  4 +--
 .../Player/PlayerEnums.cs                     | 14 ++++----
 .../Player/SubtitleUpdatedEventArgs.cs        |  2 +-
 .../Player/VideoFrameDecodedEventArgs.cs      |  2 +-
 18 files changed, 75 insertions(+), 71 deletions(-)

diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AdaptiveVariants.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AdaptiveVariants.cs
index b24188e5788..aa66f1f0fa8 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AdaptiveVariants.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AdaptiveVariants.cs
@@ -93,7 +93,7 @@ internal AdaptiveVariants(Player player)
         }
 
         /// <summary>
-        /// Sets the maximum limit of the streaming variant.
+        /// Sets the maximum limit of adaptive streaming variants.
         /// </summary>
         /// <exception cref="ObjectDisposedException">The <see cref="Player"/> has already been disposed of.</exception>
         /// <exception cref="ArgumentOutOfRangeException">
@@ -119,7 +119,7 @@ public void SetMaxLimit(int bandwidth, int width = -1, int height = -1)
         }
 
         /// <summary>
-        /// Gets the maximum limit of the streaming variant.
+        /// Gets the maximum limit of the adaptive streaming variant.
         /// </summary>
         /// <returns>The <see cref="VariantInfo"/> containing the variant information.</returns>
         /// <exception cref="ObjectDisposedException">The <see cref="Player"/> has already been disposed of.</exception>
@@ -136,7 +136,7 @@ public VariantInfo GetMaxLimit()
         }
 
         /// <summary>
-        /// Retrieves all the available adaptive variants.
+        /// Retrieves all the available adaptive streaming variants.
         /// </summary>
         /// <returns>
         /// It returns a list contained all the available adaptive variants.
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AudioDataDecodedEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AudioDataDecodedEventArgs.cs
index 17a00850c7e..1f0ac187777 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AudioDataDecodedEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AudioDataDecodedEventArgs.cs
@@ -32,7 +32,7 @@ internal AudioDataDecodedEventArgs(MediaPacket packet)
         }
 
         /// <summary>
-        /// Gets the packet containing the decoded frame.
+        /// Gets the media packet containing the audio decoded frame.
         /// </summary>
         /// <since_tizen> 6 </since_tizen>
         public MediaPacket Packet { get; }
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
index 9815506a3b1..66b670207e4 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
@@ -98,7 +98,7 @@ public void Clear()
         }
 
         /// <summary>
-        /// Gets the number of items.
+        /// Gets the number of equalizer bands available.
         /// </summary>
         /// <exception cref="NotAvailableException">If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
         ///     -or-<br/>
@@ -148,7 +148,7 @@ public Range BandLevelRange
         }
 
         /// <summary>
-        /// Gets the value whether the AudioEffect is available or not.
+        /// Gets the value indicating whether the AudioEffect is available or not.
         /// </summary>
         /// <remarks>This function returns the availability of the <see cref="AudioEffect"/>.
         /// It could be unavailable depending on the platform capabilities.<br/>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
index e25c20ea83a..cf474668a61 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
@@ -50,7 +50,7 @@ internal void CheckDisabled()
         }
 
         /// <summary>
-        /// Enables or disables the audio offload.
+        /// Gets or sets whether the audio offload is enabled.
         /// </summary>
         /// <value>The value indicating whether or not audio offload is enabled. The default value is false.</value>
         /// <remarks><para>The player lets the hardware decode and render the sound if the audio offload is enabled.
@@ -103,7 +103,7 @@ public bool IsEnabled
         }
 
         /// <summary>
-        /// Get a state whether or not the audio offload is activated.
+        /// Get a value indication whether or not the audio offload is activated.
         /// </summary>
         /// <value>The value indicating whether or not AudioOffload is activated.</value>
         /// <remarks>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs b/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
index d5c7e5406bb..b359aa05f49 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
@@ -85,7 +85,7 @@ public int Level
 
 
         /// <summary>
-        /// Gets the frequency in dB.
+        /// Gets the frequency in dB of the equalizer band.
         /// </summary>
         /// <exception cref="NotAvailableException">
         ///     If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
@@ -105,7 +105,7 @@ public int Frequency
         }
 
         /// <summary>
-        /// Gets the frequency range in dB.
+        /// Gets the frequency range in dB of the equalizer band.
         /// </summary>
         /// <exception cref="NotAvailableException">
         ///     If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaBufferSource.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaBufferSource.cs
index 0ad90e3ff76..c2d47dafdcb 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaBufferSource.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaBufferSource.cs
@@ -33,7 +33,7 @@ public sealed class MediaBufferSource : MediaSource
         private byte[] _buffer;
 
         /// <summary>
-        /// Initializes a new instance of the MediaBufferSource class with an allocated buffer.
+        /// Initializes a new instance of the MediaBufferSource class with the specified buffer length.
         /// </summary>
         /// <param name="length">The value indicating the size of the buffer.</param>
         /// <exception cref="ArgumentOutOfRangeException">
@@ -54,7 +54,7 @@ public MediaBufferSource(int length)
         }
 
         /// <summary>
-        /// Initializes a new instance of the MediaBufferSource class from the buffer.
+        /// Initializes a new instance of the MediaBufferSource class with the specified buffer.
         /// </summary>
         /// <param name="buffer">The source array to be copied into the buffer.</param>
         /// <exception cref="ArgumentNullException"><paramref name="buffer"/> is null.</exception>
@@ -65,7 +65,7 @@ public MediaBufferSource(byte[] buffer) : this(buffer, buffer == null ? 0 : buff
 
         //TODO remove default parameter.
         /// <summary>
-        /// Initializes a new instance of the MediaBufferSource class from the buffer
+        /// Initializes a new instance of the MediaBufferSource class with buffer, length, and optional offset.
         /// with the specified length and the specified offset.
         /// </summary>
         /// <param name="buffer">The source array to be copied into the buffer.</param>
@@ -141,4 +141,3 @@ internal override void OnAttached(Player player)
         }
     }
 }
-
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamBufferStatusChangedEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamBufferStatusChangedEventArgs.cs
index dd48e886b80..468a41dfb67 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamBufferStatusChangedEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamBufferStatusChangedEventArgs.cs
@@ -36,7 +36,7 @@ public MediaStreamBufferStatusChangedEventArgs(MediaStreamBufferStatus status)
         }
 
         /// <summary>
-        /// Gets the status.
+        /// Gets the current buffer status.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public MediaStreamBufferStatus Status { get; }
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamConfiguration.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamConfiguration.cs
index c8d19defb14..602e7e9156e 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamConfiguration.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamConfiguration.cs
@@ -59,10 +59,10 @@ internal MediaStreamConfiguration(MediaStreamSource owner, StreamType streamType
         public event EventHandler<MediaStreamSeekingOccurredEventArgs> SeekingOccurred;
 
         /// <summary>
-        /// Gets or sets the max size of the buffer.
+        /// Gets or sets the maximum size of the buffer for media stream.
         /// </summary>
-        /// <value>The max size of the buffer. The default is 200000.</value>
-        /// <remarks>If the buffer level overflows the max size, <see cref="BufferStatusChanged"/> will be raised with <see cref="MediaStreamBufferStatus.Overflow"/>.</remarks>
+        /// <value>The maximum size of the buffer in bytes. The default is 200000.</value>
+        /// <remarks>If the buffer level overflows the maximum size, <see cref="BufferStatusChanged"/> will be raised with <see cref="MediaStreamBufferStatus.Overflow"/>.</remarks>
         /// <exception cref="InvalidOperationException">The <see cref="MediaStreamSource"/> is not assigned to a player.</exception>
         /// <exception cref="ArgumentOutOfRangeException"><paramref name="value"/> is zero.</exception>
         /// <seealso cref="BufferStatusChanged"/>
@@ -95,7 +95,7 @@ public ulong BufferMaxSize
         }
 
         /// <summary>
-        /// Gets or sets the minimum threshold of the buffer.
+        /// Gets or sets the minimum threshold of the media stream buffer.
         /// </summary>
         /// <value>The minimum threshold of the buffer in percentage. The default is zero.</value>
         /// <remarks>If the buffer level drops below the threshold value, <see cref="BufferStatusChanged"/> will be raised with <see cref="MediaStreamBufferStatus.Underrun"/>.</remarks>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSeekingOccurredEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSeekingOccurredEventArgs.cs
index e033ed9f6e8..7de8cc98790 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSeekingOccurredEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSeekingOccurredEventArgs.cs
@@ -35,8 +35,14 @@ public MediaStreamSeekingOccurredEventArgs(ulong offset)
         }
 
         /// <summary>
-        /// Gets the offset.
+        /// Gets the next new play position as a result of seeking operation.
         /// </summary>
+        /// <remarks>
+        /// The next playback position after SetPlayPositionAsync could be a little bit different with its input position.<br/>
+        /// So user should push the next media data using this time offset in nanoseconds.
+        /// </remarks>
+        /// <seealso cref="Player.SetPlayPositionAsync"/>
+        /// <seealso cref="Player.SetPlayPositionNanosecondsAsync"/>
         /// <since_tizen> 3 </since_tizen>
         public ulong Offset { get; }
 
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSource.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSource.cs
index 0bee6d10c89..ea57faa6d78 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSource.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaStreamSource.cs
@@ -37,7 +37,7 @@ public sealed class MediaStreamSource : MediaSource
         private static List<MediaFormatVideoMimeType> _supportedVideoFormats;
 
         /// <summary>
-        /// Gets all supported audio types.
+        /// Gets the supported audio types of the media stream source.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public static IEnumerable<MediaFormatAudioMimeType> SupportedAudioTypes
@@ -50,7 +50,7 @@ public static IEnumerable<MediaFormatAudioMimeType> SupportedAudioTypes
         }
 
         /// <summary>
-        /// Gets all supported video types.
+        /// Gets the supported video types of the media stream source.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public static IEnumerable<MediaFormatVideoMimeType> SupportedVideoTypes
@@ -214,7 +214,7 @@ public MediaStreamSource(VideoMediaFormat videoMediaFormat)
         public MediaStreamConfiguration VideoConfiguration { get; }
 
         /// <summary>
-        /// Pushes elementary stream to decode audio or video.
+        /// Pushes elementary stream to decode audio or video packet.
         /// </summary>
         /// <remarks>This source must be set as a source to a player and the player must be in the <see cref="PlayerState.Ready"/>,
         /// <see cref="PlayerState.Playing"/>, or <see cref="PlayerState.Paused"/> state.</remarks>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/MediaUriSource.cs b/src/Tizen.Multimedia.MediaPlayer/Player/MediaUriSource.cs
index 240b7a37f03..415a8f17f96 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/MediaUriSource.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/MediaUriSource.cs
@@ -20,7 +20,7 @@
 namespace Tizen.Multimedia
 {
     /// <summary>
-    /// Represents a media source with a uri.
+    /// Represents a media source with a URI.
     /// </summary>
     /// <remarks>
     /// The internet privilege(http://tizen.org/privilege/internet) must be added if any URLs are used to play from a network.
@@ -31,9 +31,10 @@ namespace Tizen.Multimedia
     /// <since_tizen> 3 </since_tizen>
     public sealed class MediaUriSource : MediaSource
     {
-        // TODO consider using Uri class.
+        // TODO consider using URI class.
         /// <summary>
-        /// Initializes a new instance of the MediaUriSource class with the specified uri.</summary>
+        /// Initializes a new instance of the MediaUriSource class with the specified URI.
+        /// </summary>
         /// <param name="uri">The uri string.</param>
         /// <remarks>For HTTP or RSTP, uri should start with "http://" or "rtsp://".
         /// The default protocol is "file://".
@@ -57,4 +58,3 @@ internal override void OnAttached(Player player)
         }
     }
 }
-
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/PlaybackInterruptedEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/PlaybackInterruptedEventArgs.cs
index 47475e7a858..679ab19ddac 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/PlaybackInterruptedEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/PlaybackInterruptedEventArgs.cs
@@ -35,7 +35,7 @@ public PlaybackInterruptedEventArgs(PlaybackInterruptionReason reason)
         }
 
         /// <summary>
-        /// Gets the reason.
+        /// Gets the reason for the playback interruption.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public PlaybackInterruptionReason Reason { get; }
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/Player.Properties.cs b/src/Tizen.Multimedia.MediaPlayer/Player/Player.Properties.cs
index 1b5d2f3939a..e60b994b413 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/Player.Properties.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/Player.Properties.cs
@@ -193,7 +193,7 @@ public PlayerBufferingTime BufferingTime
         #endregion
 
         /// <summary>
-        /// Gets the state of the player.
+        /// Gets the current state of the player.
         /// </summary>
         /// <value>The current state of the player.</value>
         /// <exception cref="ObjectDisposedException">The player has already been disposed of.</exception>
@@ -258,7 +258,7 @@ public AudioLatencyMode AudioLatencyMode
         }
 
         /// <summary>
-        /// Gets or sets the looping state.
+        /// Gets or sets a value indicationg whether the media source is played in a loop.
         /// </summary>
         /// <value>true if the playback is looping; otherwise, false. The default value is false.</value>
         /// <exception cref="ObjectDisposedException">The player has already been disposed of.</exception>
@@ -286,7 +286,7 @@ public bool IsLooping
         private PlayerDisplaySettings _displaySettings;
 
         /// <summary>
-        /// Gets the display settings.
+        /// Gets the display settings of the player.
         /// </summary>
         /// <value>A <see cref="PlayerDisplaySettings"/> that specifies the display settings.</value>
         /// <since_tizen> 3 </since_tizen>
@@ -314,7 +314,7 @@ private void ReplaceDisplay(Display newDisplay)
         }
 
         /// <summary>
-        /// Gets or sets the display.
+        /// Gets or sets the display of the player.
         /// </summary>
         /// <value>A <see cref="Multimedia.Display"/> that specifies the display.</value>
         /// <remarks>
@@ -387,7 +387,7 @@ PlayerErrorCode IDisplayable<PlayerErrorCode>.ApplyEcoreWindow(IntPtr windowHand
         private PlayerTrackInfo _audioTrack;
 
         /// <summary>
-        /// Gets the track info for the audio.
+        /// Gets the audio track information of the player.
         /// </summary>
         /// <value>A <see cref="PlayerTrackInfo"/> for audio.</value>
         /// <since_tizen> 3 </since_tizen>
@@ -406,7 +406,7 @@ public PlayerTrackInfo AudioTrackInfo
         private PlayerTrackInfo _subtitleTrackInfo;
 
         /// <summary>
-        /// Gets the track info for the subtitle.
+        /// Gets the sutitle track information of the player.
         /// </summary>
         /// <value>A <see cref="PlayerTrackInfo"/> for the subtitle.</value>
         /// <since_tizen> 3 </since_tizen>
@@ -425,7 +425,7 @@ public PlayerTrackInfo SubtitleTrackInfo
         private StreamInfo _streamInfo;
 
         /// <summary>
-        /// Gets the stream information.
+        /// Gets the stream information of the player.
         /// </summary>
         /// <value>A <see cref="StreamInfo"/> for this player.</value>
         /// <since_tizen> 3 </since_tizen>
@@ -444,7 +444,7 @@ public StreamInfo StreamInfo
         private AudioEffect _audioEffect;
 
         /// <summary>
-        /// Gets the audio effect.
+        /// Gets the audio effect of the player.
         /// </summary>
         /// <feature>http://tizen.org/feature/multimedia.custom_audio_effect</feature>
         /// <exception cref="NotSupportedException">The required feature is not supported.</exception>
@@ -463,7 +463,7 @@ public AudioEffect AudioEffect
         }
 
         /// <summary>
-        /// Gets or sets the mute state.
+        /// Gets or sets a value indicationg whether the audio output of the player is muted.
         /// </summary>
         /// <value>true if the player is muted; otherwise, false.</value>
         /// <exception cref="ObjectDisposedException">The player has already been disposed of.</exception>
@@ -486,7 +486,7 @@ public bool Muted
         }
 
         /// <summary>
-        /// Gets or sets the current volume.
+        /// Gets or sets the current volume of the player.
         /// </summary>
         /// <remarks>Valid volume range is from 0 to 1.0, inclusive.</remarks>
         /// <exception cref="ObjectDisposedException">The player has already been disposed of.</exception>
@@ -520,7 +520,7 @@ public float Volume
         }
 
         /// <summary>
-        /// Gets or sets the audio-only state.
+        /// Gets or sets a value indicating whether the player is in audio only mode.
         /// </summary>
         /// <value>true if the playback is audio-only mode; otherwise, false. The default value is false.</value>
         /// The <see cref="Player"/> must be in the <see cref="PlayerState.Ready"/>,
@@ -547,7 +547,7 @@ public bool IsAudioOnly
         }
 
         /// <summary>
-        /// Gets or sets the player's replaygain state.
+        /// Gets or sets a value indicating whether the replay gain is enabled.
         /// </summary>
         /// <value>If the replaygain status is true, replaygain is applied (if contents has a replaygain tag);
         /// otherwise, the replaygain is not affected by tag and properties.</value>
@@ -585,8 +585,7 @@ public bool ReplayGain
         }
 
         /// <summary>
-        /// Enables or disables controlling the pitch of audio.
-        /// Gets the status of controlling the pitch of audio.
+        /// Gets or sets a value indicating whether the audio pitch control is enabled.
         /// </summary>
         /// <value>The value indicating whether or not AudioPitch is enabled. The default is false.</value>
         /// <remarks>This function is used for audio content only.
@@ -628,7 +627,7 @@ public bool AudioPitchEnabled
         }
 
         /// <summary>
-        /// Gets or sets the pitch of audio.
+        /// Gets or sets the pitch of the audio.
         /// </summary>
         /// <value>The audio stream pitch value. The default is 1.</value>
         /// <remarks>Enabling pitch control could increase the CPU usage on some devices.
@@ -783,7 +782,7 @@ public CodecType VideoCodecType
         private SphericalVideo _sphericalVideo;
 
         /// <summary>
-        /// Gets the spherical video settings.
+        /// Gets the spherical video porperties of the player.
         /// </summary>
         /// <since_tizen> 5 </since_tizen>
         public SphericalVideo SphericalVideo
@@ -802,7 +801,7 @@ public SphericalVideo SphericalVideo
         private AdaptiveVariants _adaptiveVariants;
 
         /// <summary>
-        /// Gets the adaptive variants settings.
+        /// Gets the adaptive variants of the player.
         /// </summary>
         /// <since_tizen> 5 </since_tizen>
         public AdaptiveVariants AdaptiveVariants
@@ -821,7 +820,7 @@ public AdaptiveVariants AdaptiveVariants
         private AudioOffload _audioOffload;
 
         /// <summary>
-        /// Gets the setting for audio offload.
+        /// Gets the audio offload settings of the player.
         /// </summary>
         /// <since_tizen> 6 </since_tizen>
         public AudioOffload AudioOffload
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/Player.cs b/src/Tizen.Multimedia.MediaPlayer/Player/Player.cs
index 3b3e2c2e105..0ec788b05e1 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/Player.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/Player.cs
@@ -184,7 +184,7 @@ internal void ValidateNotDisposed()
         #region Methods
 
         /// <summary>
-        /// Gets the streaming download progress.
+        /// Gets the download progress of the streaming contents.
         /// </summary>
         /// <returns>The <see cref="DownloadProgress"/> containing current download progress.</returns>
         /// <remarks>The player must be in the <see cref="PlayerState.Ready"/>, <see cref="PlayerState.Playing"/>,
@@ -211,7 +211,7 @@ public DownloadProgress GetDownloadProgress()
         }
 
         /// <summary>
-        /// Sets the subtitle path for playback.
+        /// Sets the subtitle file path for playback.
         /// </summary>
         /// <param name="path">The absolute path of the subtitle file, it can be NULL in the <see cref="PlayerState.Idle"/> state.</param>
         /// <remarks>Only MicroDVD/SubViewer(*.sub), SAMI(*.smi), and SubRip(*.srt) subtitle formats are supported.
@@ -247,7 +247,7 @@ public void SetSubtitle(string path)
         }
 
         /// <summary>
-        /// Removes the subtitle path.
+        /// Clears the the subtitle path of the media player.
         /// </summary>
         /// <remarks>The player must be in the <see cref="PlayerState.Idle"/> state.</remarks>
         /// <exception cref="ObjectDisposedException">The player has already been disposed of.</exception>
@@ -262,7 +262,7 @@ public void ClearSubtitle()
         }
 
         /// <summary>
-        /// Sets the offset for the subtitle.
+        /// Sets the seek offset for the subtitle.
         /// </summary>
         /// <param name="offset">The value indicating a desired offset in milliseconds.</param>
         /// <remarks>The player must be in the <see cref="PlayerState.Playing"/> or <see cref="PlayerState.Paused"/> state.</remarks>
@@ -410,7 +410,7 @@ public virtual async Task PrepareAsync(CancellationToken cancellationToken)
         }
 
         /// <summary>
-        /// Unprepares the player.
+        /// Unprepares the media player.
         /// </summary>
         /// <remarks>
         ///     The most recently used source is reset and is no longer associated with the player. Playback is no longer possible.
@@ -450,7 +450,7 @@ protected virtual void OnUnprepared()
         }
 
         /// <summary>
-        /// Starts or resumes playback.
+        /// Starts or resumes playing the media content.
         /// </summary>
         /// <remarks>
         /// Sound can be mixed with other sounds if you don't control the stream focus using <see cref="ApplyAudioStreamPolicy"/>.<br/>
@@ -505,7 +505,7 @@ public virtual void Stop()
         }
 
         /// <summary>
-        /// Pauses the player.
+        /// Pauses playing the media content.
         /// </summary>
         /// <remarks>
         /// The player must be in the <see cref="PlayerState.Playing"/> state.
@@ -570,7 +570,7 @@ public void SetSource(MediaSource source)
         }
 
         /// <summary>
-        /// Captures a video frame, asynchronously.
+        /// Captures the current frame of the video being played, asynchronously.
         /// </summary>
         /// <returns>A task that represents the asynchronous capture operation.</returns>
         /// <feature>http://tizen.org/feature/multimedia.raw_video</feature>
@@ -607,7 +607,7 @@ public async Task<CapturedFrame> CaptureVideoAsync()
         }
 
         /// <summary>
-        /// Gets the play position in milliseconds.
+        /// Gets the current playback position in milliseconds of the media.
         /// </summary>
         /// <returns>The current position in milliseconds.</returns>
         /// <remarks>The player must be in the <see cref="PlayerState.Ready"/>, <see cref="PlayerState.Playing"/>,
@@ -673,7 +673,7 @@ private async Task SetPlayPosition(long position, bool accurate, bool nanosecond
         }
 
         /// <summary>
-        /// Sets the seek position for playback, asynchronously.
+        /// Sets the playback position in milliseconds of the player, asynchronously.
         /// </summary>
         /// <param name="position">The value indicating a desired position in milliseconds.</param>
         /// <param name="accurate">The value indicating whether the operation performs with accuracy.</param>
@@ -701,7 +701,7 @@ public async Task SetPlayPositionAsync(int position, bool accurate)
         }
 
         /// <summary>
-        /// Gets the play position in nanoseconds.
+        /// Gets the current playback position in nanoseconds of the media.
         /// </summary>
         /// <returns>The current position in nanoseconds.</returns>
         /// <remarks>The player must be in the <see cref="PlayerState.Ready"/>, <see cref="PlayerState.Playing"/>,
@@ -725,7 +725,7 @@ public long GetPlayPositionNanoseconds()
         }
 
         /// <summary>
-        /// Sets the seek position in nanoseconds for playback, asynchronously.
+        /// Sets the playback position in nanoseconds of the player, asynchronously.
         /// </summary>
         /// <param name="position">The value indicating a desired position in nanoseconds.</param>
         /// <param name="accurate">The value indicating whether the operation performs with accuracy.</param>
@@ -753,7 +753,7 @@ public async Task SetPlayPositionNanosecondsAsync(long position, bool accurate)
         }
 
         /// <summary>
-        /// Sets the playback rate.
+        /// Sets the playback rate of the player.
         /// </summary>
         /// <param name="rate">The value for the playback rate. Valid range is -5.0 to 5.0, inclusive.</param>
         /// <remarks>
@@ -793,7 +793,7 @@ public void SetPlaybackRate(float rate)
         }
 
         /// <summary>
-        /// Applies the audio stream policy.
+        /// Applies the specified audio stream policy to the player.
         /// </summary>
         /// <param name="policy">The <see cref="AudioStreamPolicy"/> to apply.</param>
         /// <remarks>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/PlayerDisplaySettings.cs b/src/Tizen.Multimedia.MediaPlayer/Player/PlayerDisplaySettings.cs
index 38b8a947a24..b8c889befc0 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/PlayerDisplaySettings.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/PlayerDisplaySettings.cs
@@ -53,7 +53,7 @@ protected PlayerDisplaySettings(Player player)
         protected Player Player { get; }
 
         /// <summary>
-        /// Gets or sets the <see cref="PlayerDisplayMode"/>.
+        /// Gets or sets the <see cref="PlayerDisplayMode"/> of the player.
         /// </summary>
         /// <exception cref="InvalidOperationException">
         ///     Operation failed; internal error.
@@ -134,7 +134,7 @@ public Rotation Rotation
         }
 
         /// <summary>
-        /// Sets the roi(region of interest).
+        /// Sets the ROI(Region Of Interest) for the video display.
         /// </summary>
         /// <param name="roi">The region.</param>
         /// <remarks>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/PlayerEnums.cs b/src/Tizen.Multimedia.MediaPlayer/Player/PlayerEnums.cs
index 137fc1a5865..27378e44466 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/PlayerEnums.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/PlayerEnums.cs
@@ -38,7 +38,7 @@ public enum PlayerError
         InternalError = ErrorCode.InvalidOperation,
 
         /// <summary>
-        /// No space.
+        /// No space on the device.
         /// </summary>
         NoSpaceOnDevice = PlayerErrorCode.NoSpaceOnDevice,
 
@@ -269,32 +269,32 @@ public enum PlaybackInterruptionReason
     public enum StreamMetadataKey
     {
         /// <summary>
-        /// Album.
+        /// The album name of the media content
         /// </summary>
         Album,
 
         /// <summary>
-        /// Artists.
+        /// The artist(s) of the media content.
         /// </summary>
         Artist,
 
         /// <summary>
-        /// Author.
+        /// The author(s) of the media content.
         /// </summary>
         Author,
 
         /// <summary>
-        /// Genre.
+        /// The genre(s) of the media content.
         /// </summary>
         Genre,
 
         /// <summary>
-        /// Title.
+        /// The title of the media content.
         /// </summary>
         Title,
 
         /// <summary>
-        /// Year.
+        /// The year of the media content.
         /// </summary>
         Year
     }
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/SubtitleUpdatedEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/SubtitleUpdatedEventArgs.cs
index f3ee00a62ab..b46489e700f 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/SubtitleUpdatedEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/SubtitleUpdatedEventArgs.cs
@@ -30,7 +30,7 @@ internal SubtitleUpdatedEventArgs(uint duration, string text)
         }
 
         /// <summary>
-        /// Gets the duration of the updated subtitle.
+        /// Gets the duration in milliseconds of the updated subtitle.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public uint Duration { get; }
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/VideoFrameDecodedEventArgs.cs b/src/Tizen.Multimedia.MediaPlayer/Player/VideoFrameDecodedEventArgs.cs
index 89e97d2c5ca..cb978a69cc4 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/VideoFrameDecodedEventArgs.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/VideoFrameDecodedEventArgs.cs
@@ -32,7 +32,7 @@ internal VideoFrameDecodedEventArgs(MediaPacket packet)
         }
 
         /// <summary>
-        /// Gets the packet containing the decoded frame.
+        /// Gets the media packet containing the decoded frame.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public MediaPacket Packet { get; }