Call_Content_Codec_Offer: more doc fixes

Signed-off-by: Jonny Lamb <jonny.lamb@collabora.co.uk>

1 files changed, 19 insertions, 5 deletions

diff --git a/spec/Call_Content_Codec_Offer.xml b/spec/Call_Content_Codec_Offer.xml
index 5fc45258..23639b25 100644
--- a/spec/Call_Content_Codec_Offer.xml
+++ b/spec/Call_Content_Codec_Offer.xml
@@ -26,15 +26,27 @@
 
     <tp:docstring xmlns="http://www.w3.org/1999/xhtml">
       This object represents an offer of a Codec payload mapping.
-      FIXME add Accept and Reject signals ?
     </tp:docstring>
 
     <method name="Accept" tp:name-for-bindings="Accept">
       <arg name="Codecs" direction="in"
-        type="a(usuua{ss})" tp:type="Codec[]" />
+        type="a(usuua{ss})" tp:type="Codec[]">
+        <tp:docstring>
+          The local codec mapping to send to the remote contacts and
+          to use in the <tp:dbus-ref
+          namespace="ofdT.Call">Content.DRAFT</tp:dbus-ref>.
+        </tp:docstring>
+      </arg>
       <tp:docstring>
-        Accept the updated Codec mapping and update the local mapping
+        Accept the updated Codec mapping and update the local mapping.
       </tp:docstring>
+      <tp:possible-errors>
+        <tp:error name="org.freedesktop.Telepathy.Error.InvalidArgument">
+          <tp:docstring>
+            The codecs given as the argument are invalid in some way.
+          </tp:docstring>
+        </tp:error>
+      </tp:possible-errors>
     </method>
 
     <method name="Reject" tp:name-for-bindings="Reject">
@@ -46,9 +58,11 @@
 
     <property name="RemoteContactCodecMap"
       tp:name-for-bindings="Remote_Contact_Codec_Map"
-      type="a{ua(usuua{ss})}" tp:type="Contact_Codec_Map" access="read">
+      type="a{ua(usuua{ss})}" tp:type="Contact_Codec_Map" access="read"
+      tp:immutable="yes">
       <tp:docstring>
-        A map from remote contacts to the lists of codecs they support.
+        A map from remote contact to the list of codecs he or she
+        supports.
       </tp:docstring>
     </property>