Suppressing .NET static analysis warnings.

SeleniumHQ · Jan 26, 2016 · 75243dd · 75243dd
1 parent 8234938
commit 75243dd
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/dotnet/src/support/PageObjects/WebElementProxy.cs b/dotnet/src/support/PageObjects/WebElementProxy.cs
@@ -39,8 +39,8 @@ internal sealed class WebElementProxy : WebDriverObjectProxy, IWrapsElement
         /// Initializes a new instance of the <see cref="WebElementProxy"/> class.
         /// </summary>
         /// <param name="classToProxy">The <see cref="Type"/> of object for which to create a proxy.</param>
-        /// <param name="locator">The <see cref="IElementLocatorFactory"/> implementation that
-        /// determines how elements are located.</param>
+        /// <param name="locator">The <see cref="IElementLocator"/> implementation that determines
+        /// how elements are located.</param>
         /// <param name="bys">The list of methods by which to search for the elements.</param>
         /// <param name="cache"><see langword="true"/> to cache the lookup to the element; otherwise, <see langword="false"/>.</param>
         private WebElementProxy(Type classToProxy, IElementLocator locator, IEnumerable<By> bys, bool cache)

diff --git a/dotnet/src/webdriver/Internal/ZipStorer.cs b/dotnet/src/webdriver/Internal/ZipStorer.cs
@@ -70,15 +70,15 @@ public enum CompressionMethod : ushort
         }
 
         /// <summary>
-        /// Gets or sets a value indicating whether file names and comments should be encoded using UTF-8.
+        /// Gets a value indicating whether file names and comments should be encoded using UTF-8.
         /// </summary>
         public bool EncodeUtf8
         {
             get { return this.encodeUtf8; }
         }
 
         /// <summary>
-        /// Gets or sets a value indicating whether to force using the deflate algorithm,
+        /// Gets a value indicating whether to force using the deflate algorithm,
         /// even if doing so inflates the stored file.
         /// </summary>
         public bool ForceDeflating

diff --git a/dotnet/src/webdriver/Remote/RemoteAlert.cs b/dotnet/src/webdriver/Remote/RemoteAlert.cs
@@ -72,6 +72,11 @@ public void Accept()
         /// <param name="keysToSend">The keystrokes to send.</param>
         public void SendKeys(string keysToSend)
         {
+            if (keysToSend == null)
+            {
+                throw new ArgumentNullException("keysToSend", "Keys to send must not be null.");
+            }
+
             Dictionary<string, object> parameters = new Dictionary<string, object>();
             if (this.driver.IsSpecificationCompliant)
             {