Skip to content

Commit a000d61

Browse files
committed
Annotate some easy wins in the firefox namespace
1 parent 76b9cc4 commit a000d61

File tree

3 files changed

+10
-5
lines changed

3 files changed

+10
-5
lines changed

dotnet/src/webdriver/Firefox/FirefoxAndroidOptions.cs

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,17 +18,20 @@
1818
// </copyright>
1919

2020
using OpenQA.Selenium.Internal;
21+
using System;
2122
using System.Collections.Generic;
2223
using System.Collections.ObjectModel;
2324

25+
#nullable enable
26+
2427
namespace OpenQA.Selenium.Firefox
2528
{
2629
/// <summary>
2730
/// Generates the capabilities for automating Firefox applications on Android
2831
/// </summary>
2932
public class FirefoxAndroidOptions : AndroidOptions
3033
{
31-
private List<string> androidIntentArguments = new List<string>();
34+
private readonly List<string> androidIntentArguments = new List<string>();
3235

3336
/// <summary>
3437
/// Initializes a new instance of the <see cref="FirefoxAndroidOptions"/> class.
@@ -41,10 +44,7 @@ public FirefoxAndroidOptions(string androidPackage) : base(androidPackage)
4144
/// <summary>
4245
/// Gets a read-only list of the intent arguments set for this set of options.
4346
/// </summary>
44-
public ReadOnlyCollection<string> AndroidIntentArguments
45-
{
46-
get { return this.androidIntentArguments.AsReadOnly(); }
47-
}
47+
public ReadOnlyCollection<string> AndroidIntentArguments => this.androidIntentArguments.AsReadOnly();
4848

4949
/// <summary>
5050
/// Argument to launch the intent with. The given intent arguments are appended to the "am start" command.
@@ -59,6 +59,7 @@ public void AddIntentArgument(string argument)
5959
/// Arguments to launch the intent with. The given intent arguments are appended to the "am start" command.
6060
/// </summary>
6161
/// <param name="arguments">The arguments to add.</param>
62+
/// <exception cref="ArgumentNullException">If <paramref name="arguments"/> is <see langword="null"/>.</exception>
6263
public void AddIntentArguments(params string[] arguments)
6364
{
6465
this.androidIntentArguments.AddRange(arguments);

dotnet/src/webdriver/Firefox/FirefoxCommandContext.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
// under the License.
1818
// </copyright>
1919

20+
#nullable enable
21+
2022
namespace OpenQA.Selenium.Firefox
2123
{
2224
/// <summary>

dotnet/src/webdriver/Firefox/FirefoxDriverLogLevel.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
// under the License.
1818
// </copyright>
1919

20+
#nullable enable
21+
2022
namespace OpenQA.Selenium.Firefox
2123
{
2224
/// <summary>

0 commit comments

Comments
 (0)