Suppressing .NET static analysis warnings

jimevans · jimevans · commit b345522fd179 · 2016-01-27T14:57:02.000-08:00
diff --git a/dotnet/src/webdriver/Remote/RemoteWebDriver.cs b/dotnet/src/webdriver/Remote/RemoteWebDriver.cs
@@ -568,7 +568,7 @@ public ReadOnlyCollection<IWebElement> FindElementsById(string id)
             if (this.IsSpecificationCompliant)
             {
                 string selector = EscapeCssSelector(id);
-                if (selector == string.Empty)
+                if (string.IsNullOrEmpty(selector))
                 {
                     // Finding multiple elements with an empty ID will return
                     // an empty list. However, finding by a CSS selector of '#'

Original file line number	Diff line number	Diff line change
`@@ -568,7 +568,7 @@ public ReadOnlyCollection<IWebElement> FindElementsById(string id)`
`568`	`568`	`if (this.IsSpecificationCompliant)`
`569`	`569`	`{`
`570`	`570`	`string selector = EscapeCssSelector(id);`
`571`		`- if (selector == string.Empty)`
	`571`	`+ if (string.IsNullOrEmpty(selector))`
`572`	`572`	`{`
`573`	`573`	`// Finding multiple elements with an empty ID will return`
`574`	`574`	`// an empty list. However, finding by a CSS selector of '#'`