From 3259a71ccbcf234fe798b99da41ab7b7ad966b58 Mon Sep 17 00:00:00 2001
From: carlossanlop <calope@microsoft.com>
Date: Tue, 6 Aug 2019 10:46:44 -0700
Subject: [PATCH 1/2] Document System.Security.Cryptography.Pkcs.SignedCms

---
 xml/System.Security.Cryptography.Pkcs/SignedCms.xml | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/xml/System.Security.Cryptography.Pkcs/SignedCms.xml b/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
index 1b257a6f2c3..6f205163791 100644
--- a/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
+++ b/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
@@ -350,9 +350,10 @@
         <Parameter Name="certificate" Type="System.Security.Cryptography.X509Certificates.X509Certificate2" Index="0" FrameworkAlternate="dotnet-plat-ext-3.0" />
       </Parameters>
       <Docs>
-        <param name="certificate">To be added.</param>
-        <summary>To be added.</summary>
+        <param name="certificate">The certificate to add to the collection.</param>
+        <summary>Adds a certificate to the collection of certificates for the encoded CMS/PKCS #7 message.</summary>
         <remarks>To be added.</remarks>
+        <exception cref="T:System.Security.Cryptography.CryptographicException">The certificate already exists in the collection.</exception>
       </Docs>
     </Member>
     <Member MemberName="Certificates">
@@ -926,8 +927,8 @@ The following permissions are required to access the signature key:
         <Parameter Name="certificate" Type="System.Security.Cryptography.X509Certificates.X509Certificate2" Index="0" FrameworkAlternate="dotnet-plat-ext-3.0" />
       </Parameters>
       <Docs>
-        <param name="certificate">To be added.</param>
-        <summary>To be added.</summary>
+        <param name="certificate">The certificate to remove from the collection.</param>
+        <summary>Removes the specified certificate from the collection of certificates for the encoded CMS/PKCS #7 message, if it exists.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

From b65d0c3dc6f646bb77723994aa64b069e80a6e04 Mon Sep 17 00:00:00 2001
From: carlossanlop <calope@microsoft.com>
Date: Tue, 6 Aug 2019 11:58:47 -0700
Subject: [PATCH 2/2] Exception thrown on removal

---
 xml/System.Security.Cryptography.Pkcs/SignedCms.xml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/xml/System.Security.Cryptography.Pkcs/SignedCms.xml b/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
index 6f205163791..8a8e1fab445 100644
--- a/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
+++ b/xml/System.Security.Cryptography.Pkcs/SignedCms.xml
@@ -928,8 +928,9 @@ The following permissions are required to access the signature key:
       </Parameters>
       <Docs>
         <param name="certificate">The certificate to remove from the collection.</param>
-        <summary>Removes the specified certificate from the collection of certificates for the encoded CMS/PKCS #7 message, if it exists.</summary>
+        <summary>Removes the specified certificate from the collection of certificates for the encoded CMS/PKCS #7 message.</summary>
         <remarks>To be added.</remarks>
+        <exception cref="T:System.Security.Cryptography.CryptographicException">The certificate was not found.</exception>
       </Docs>
     </Member>
     <MemberGroup MemberName="RemoveSignature">