Skip to content

Commit a549e49

Browse files
committed
Code linting with StyleCop. No functional changes
1 parent 4cf6168 commit a549e49

File tree

5 files changed

+19
-18
lines changed

5 files changed

+19
-18
lines changed

dotnet/src/webdriver/Chrome/ChromeOptions.cs

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ public void AddArguments(IEnumerable<string> argumentsToAdd)
189189
{
190190
if (argumentsToAdd == null)
191191
{
192-
throw new ArgumentNullException("arguments", "arguments must not be null");
192+
throw new ArgumentNullException("argumentsToAdd", "argumentsToAdd must not be null");
193193
}
194194

195195
this.arguments.AddRange(argumentsToAdd);
@@ -214,25 +214,25 @@ public void AddExcludedArgument(string argument)
214214
/// Adds arguments to be excluded from the list of arguments passed by default
215215
/// to the Chrome.exe command line by chromedriver.exe.
216216
/// </summary>
217-
/// <param name="arguments">An array of arguments to exclude.</param>
218-
public void AddExcludedArguments(params string[] arguments)
217+
/// <param name="argumentsToExclude">An array of arguments to exclude.</param>
218+
public void AddExcludedArguments(params string[] argumentsToExclude)
219219
{
220-
this.AddExcludedArguments(new List<string>(arguments));
220+
this.AddExcludedArguments(new List<string>(argumentsToExclude));
221221
}
222222

223223
/// <summary>
224224
/// Adds arguments to be excluded from the list of arguments passed by default
225225
/// to the Chrome.exe command line by chromedriver.exe.
226226
/// </summary>
227-
/// <param name="arguments">An <see cref="IEnumerable{T}"/> object of arguments to exclude.</param>
228-
public void AddExcludedArguments(IEnumerable<string> arguments)
227+
/// <param name="argumentsToExclude">An <see cref="IEnumerable{T}"/> object of arguments to exclude.</param>
228+
public void AddExcludedArguments(IEnumerable<string> argumentsToExclude)
229229
{
230-
if (arguments == null)
230+
if (argumentsToExclude == null)
231231
{
232-
throw new ArgumentNullException("arguments", "arguments must not be null");
232+
throw new ArgumentNullException("argumentsToExclude", "argumentsToExclude must not be null");
233233
}
234234

235-
this.excludedSwitches.AddRange(arguments);
235+
this.excludedSwitches.AddRange(argumentsToExclude);
236236
}
237237

238238
/// <summary>

dotnet/src/webdriver/GlobalSuppressions.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1056:UriPropertiesShouldNotBeStrings", Scope = "member", Target = "OpenQA.Selenium.Proxy.#ProxyAutoConfigUrl", Justification = "Proxy configuration can be string instead of Uri.")]
4545
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1056:UriPropertiesShouldNotBeStrings", Scope = "member", Target = "OpenQA.Selenium.IE.InternetExplorerOptions.#InitialBrowserUrl", Justification = "InitialBrowserUrl should be string instead of Uri.")]
4646
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1056:UriPropertiesShouldNotBeStrings", Scope = "member", Target = "OpenQA.Selenium.Chrome.ChromeDriverService.#UrlPathPrefix", Justification = "UrlPathPrefix is a prefix for use with ChromeDriver, and should be a string.")]
47+
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1056:UriPropertiesShouldNotBeStrings", Scope = "member", Target = "OpenQA.Selenium.Opera.OperaDriverService.#UrlPathPrefix", Justification = "UrlPathPrefix is a prefix for use with OperaDriver, and should be a string.")]
4748
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Globalization", "CA1308:NormalizeStringsToUppercase", Scope = "member", Target = "OpenQA.Selenium.Proxy.#SerializableProxyKind", Justification = "Strings are normalized to lower case by JSON wire protocol.")]
4849
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Globalization", "CA1308:NormalizeStringsToUppercase", Scope = "member", Target = "OpenQA.Selenium.Firefox.FirefoxProfile.#UpdateUserPreferences()", Justification = "Strings are normalized to lower case by JSON wire protocol.")]
4950
[assembly: System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Globalization", "CA1308:NormalizeStringsToUppercase", Scope = "member", Target = "OpenQA.Selenium.Firefox.Preferences.#SetPreferenceValue(System.String,System.Object)", Justification = "Strings are normalized to lower case by JSON wire protocol.")]

dotnet/src/webdriver/IE/InternetExplorerDriverEngine.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ namespace OpenQA.Selenium.IE
2929
public enum InternetExplorerDriverEngine
3030
{
3131
/// <summary>
32-
/// Represents the Legacy value, forcing the driver tu use only the open-source
32+
/// Represents the Legacy value, forcing the driver to use only the open-source
3333
/// driver engine implementation.
3434
/// </summary>
3535
Legacy,

dotnet/src/webdriver/IE/InternetExplorerOptions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -239,8 +239,8 @@ public bool ForceShellWindowsApi
239239
/// </summary>
240240
public bool ValidateCookieDocumentType
241241
{
242-
get { return validateCookieDocumentType; }
243-
set { validateCookieDocumentType = value; }
242+
get { return this.validateCookieDocumentType; }
243+
set { this.validateCookieDocumentType = value; }
244244
}
245245

246246
/// <summary>

dotnet/src/webdriver/Opera/OperaOptions.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ public void AddArguments(IEnumerable<string> argumentsToAdd)
189189
{
190190
if (argumentsToAdd == null)
191191
{
192-
throw new ArgumentNullException("arguments", "arguments must not be null");
192+
throw new ArgumentNullException("argumentsToAdd", "argumentsToAdd must not be null");
193193
}
194194

195195
this.arguments.AddRange(argumentsToAdd);
@@ -224,15 +224,15 @@ public void AddExcludedArguments(params string[] arguments)
224224
/// Adds arguments to be excluded from the list of arguments passed by default
225225
/// to the Opera.exe command line by operadriver.exe.
226226
/// </summary>
227-
/// <param name="arguments">An <see cref="IEnumerable{T}"/> object of arguments to exclude.</param>
228-
public void AddExcludedArguments(IEnumerable<string> arguments)
227+
/// <param name="argumentsToExclude">An <see cref="IEnumerable{T}"/> object of arguments to exclude.</param>
228+
public void AddExcludedArguments(IEnumerable<string> argumentsToExclude)
229229
{
230-
if (arguments == null)
230+
if (argumentsToExclude == null)
231231
{
232-
throw new ArgumentNullException("arguments", "arguments must not be null");
232+
throw new ArgumentNullException("argumentsToExclude", "argumentsToExclude must not be null");
233233
}
234234

235-
this.excludedSwitches.AddRange(arguments);
235+
this.excludedSwitches.AddRange(argumentsToExclude);
236236
}
237237

238238
/// <summary>

0 commit comments

Comments
 (0)