From 95c3b56680ee87635919fc4c1c01d8cb1849520b Mon Sep 17 00:00:00 2001
From: nam <36914158+aferin@users.noreply.github.com>
Date: Wed, 18 Sep 2019 10:14:07 +0900
Subject: [PATCH] [MediaPlayer] Fix descriptions according to the guideline
 (#1017)

---
 .../Player/AudioEffect.cs                        |  2 +-
 .../Player/AudioOffload.cs                       | 16 ++++++++--------
 .../Player/EqualizerBand.cs                      |  4 ++--
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
index 8cc310d93..d58edefb4 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AudioEffect.cs
@@ -113,7 +113,7 @@ namespace Tizen.Multimedia
         }
 
         /// <summary>
-        /// Gets the band level range of the bands in the dB.
+        /// Gets the range of band level in dB.
         /// </summary>
         /// <exception cref="InvalidOperationException">
         ///     If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs b/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
index 30569bfe1..bc633810e 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/AudioOffload.cs
@@ -22,7 +22,7 @@ using static Interop;
 namespace Tizen.Multimedia
 {
     /// <summary>
-    /// Provides the ability to control the audio offload for <see cref="Multimedia.Player"/>.
+    /// The class that control the audio offload for <see cref="Multimedia.Player"/>.
     /// </summary>
     /// <since_tizen> 6 </since_tizen>
     public class AudioOffload
@@ -52,12 +52,11 @@ namespace Tizen.Multimedia
         /// <summary>
         /// Enables or disables the audio offload.
         /// </summary>
-        /// <value>The value indicating whether or not AudioOffload is enabled. The default value is false.</value>
+        /// <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.
-        /// This will reduce the power consumption, but will disable the ability to handle output PCM.
+        /// Audio offload can reduce the power consumption, but disable the ability to handle output PCM.
         /// Please check the below list of functions which will not work if offloading is enabled.</para>
-        /// <para>If audio offload is enabled, the following functions will return <see cref="InvalidOperationException"/>
-        /// and they will not work at all even if they were called before offload is enabled.<br/>
+        /// <para>If audio offload is enabled, the following functions will return <see cref="InvalidOperationException"/>: <br/>
         /// <see cref="AudioEffect"/><br/>
         /// <see cref="EqualizerBand"/><br/>
         /// <see cref="PlayerTrackInfo"/><br/>
@@ -67,6 +66,7 @@ namespace Tizen.Multimedia
         /// <see cref="Player.ReplayGain"/><br/>
         /// <see cref="Player.AudioPitch"/><br/>
         /// <see cref="Player.AudioPitchEnabled"/><br/></para>
+        /// <para>Although they are called before offload is enabled, they don't work normally.</para>
         /// <para>To set, the player must be in the <see cref="PlayerState.Idle"/> state.
         /// The sound stream type of the player should be <see cref="AudioStreamType.Media"/>.</para></remarks>
         /// <feature>http://tizen.org/feature/multimedia.player.audio_offload</feature>
@@ -107,7 +107,7 @@ namespace Tizen.Multimedia
         /// </summary>
         /// <value>The value indicating whether or not AudioOffload is activated.</value>
         /// <remarks>
-        /// Audio offload could be inactivated depending on the audio device capability even if the audio offload feature is supported.
+        /// Audio offload could be inactivated depending on the audio device capability even though the audio offload feature is supported.
         /// The <see cref="Player"/> that owns this instance must be in the <see cref="PlayerState.Ready"/>,
         /// <see cref="PlayerState.Playing"/>, or <see cref="PlayerState.Paused"/> state.
         /// </remarks>
@@ -135,10 +135,10 @@ namespace Tizen.Multimedia
         }
 
         /// <summary>
-        /// Retrieves all formats for audio offload.
+        /// Retrieves the supported audio formats for audio offload.
         /// </summary>
         /// <returns>
-        /// It returns a list contained all formats for audio offload.
+        /// It returns a list containing supported audio formats for audio offload.
         /// </returns>
         /// <remarks>The supported media format can vary depending on the device capabilities.</remarks>
         /// <feature>http://tizen.org/feature/multimedia.player.audio_offload</feature>
diff --git a/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs b/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
index 569572d44..303f91e46 100644
--- a/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
+++ b/src/Tizen.Multimedia.MediaPlayer/Player/EqualizerBand.cs
@@ -85,7 +85,7 @@ namespace Tizen.Multimedia
 
 
         /// <summary>
-        /// Gets the frequency in the dB.
+        /// Gets the frequency in dB.
         /// </summary>
         /// <exception cref="InvalidOperationException">
         ///     If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
@@ -105,7 +105,7 @@ namespace Tizen.Multimedia
         }
 
         /// <summary>
-        /// Gets the frequency range in the dB.
+        /// Gets the frequency range in dB.
         /// </summary>
         /// <exception cref="InvalidOperationException">
         ///     If audio offload is enabled by calling <see cref="AudioOffload.IsEnabled"/>. (Since tizen 6.0)
-- 
2.34.1