[dotnet] [bidi] Immutable user facing collections

dotnet/src/webdriver/BiDi/BiDi.cs

@@ -101,15 +101,15 @@ public Task<ISubscription> SubscribeAsync<TEventArgs>(EventDescriptor<TEventArgs
         return SubscribeAsync([descriptor], handler, cancellationToken);
     }
 
-    public async Task<ISubscription> SubscribeAsync<TEventArgs>(IEnumerable<EventDescriptor> descriptors, Action<TEventArgs> handler, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
+    public async Task<ISubscription> SubscribeAsync<TEventArgs>(ImmutableArray<EventDescriptor> descriptors, Action<TEventArgs> handler, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
     {
         ArgumentNullException.ThrowIfNull(descriptors);
         ArgumentNullException.ThrowIfNull(handler);
 
         return await EventDispatcher.SubscribeAsync<TEventArgs>(descriptors, e => { handler(e); return default; }, cancellationToken: cancellationToken).ConfigureAwait(false);
     }
 
-    public async Task<ISubscription> SubscribeAsync<TEventArgs>(IEnumerable<EventDescriptor> descriptors, Func<TEventArgs, Task> handler, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
+    public async Task<ISubscription> SubscribeAsync<TEventArgs>(ImmutableArray<EventDescriptor> descriptors, Func<TEventArgs, Task> handler, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
     {
         ArgumentNullException.ThrowIfNull(descriptors);
         ArgumentNullException.ThrowIfNull(handler);
@@ -124,7 +124,7 @@ public Task<IEventStream<TEventArgs>> StreamAsync<TEventArgs>(EventDescriptor<TE
         return StreamAsync<TEventArgs>([descriptor], cancellationToken);
     }
 
-    public async Task<IEventStream<TEventArgs>> StreamAsync<TEventArgs>(IEnumerable<EventDescriptor> descriptors, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
+    public async Task<IEventStream<TEventArgs>> StreamAsync<TEventArgs>(ImmutableArray<EventDescriptor> descriptors, CancellationToken cancellationToken = default) where TEventArgs : EventArgs
     {
         ArgumentNullException.ThrowIfNull(descriptors);
 

dotnet/src/webdriver/BiDi/Browser/GetClientWindows.cs

@@ -21,4 +21,4 @@ namespace OpenQA.Selenium.BiDi.Browser;
 
 public sealed record GetClientWindowsOptions : CommandOptions;
 
-public sealed record GetClientWindowsResult(IReadOnlyList<ClientWindowInfo> ClientWindows) : EmptyResult;
+public sealed record GetClientWindowsResult(ImmutableArray<ClientWindowInfo> ClientWindows) : EmptyResult;

dotnet/src/webdriver/BiDi/Browser/GetUserContexts.cs

@@ -21,4 +21,4 @@ namespace OpenQA.Selenium.BiDi.Browser;
 
 public record GetUserContextsOptions : CommandOptions;
 
-public sealed record GetUserContextsResult(IReadOnlyList<UserContextInfo> UserContexts) : EmptyResult;
+public sealed record GetUserContextsResult(ImmutableArray<UserContextInfo> UserContexts) : EmptyResult;

dotnet/src/webdriver/BiDi/Browser/SetDownloadBehavior.cs

@@ -21,7 +21,7 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal sealed record SetDownloadBehaviorParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] DownloadBehavior? DownloadBehavior, IEnumerable<UserContext>? UserContexts) : Parameters;
+internal sealed record SetDownloadBehaviorParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] DownloadBehavior? DownloadBehavior, ImmutableArray<UserContext>? UserContexts) : Parameters;
 
 [JsonPolymorphic(TypeDiscriminatorPropertyName = "type")]
 [JsonDerivedType(typeof(DownloadBehaviorAllowed), "allowed")]
@@ -37,7 +37,7 @@ public sealed record DownloadBehaviorDenied : DownloadBehavior;
 
 public sealed record SetDownloadBehaviorOptions : CommandOptions
 {
-    public IEnumerable<UserContext>? UserContexts { get; init; }
+    public ImmutableArray<UserContext>? UserContexts { get; init; }
 }
 
 public sealed record SetDownloadBehaviorResult : EmptyResult;

dotnet/src/webdriver/BiDi/BrowsingContext/BrowsingContextInputModule.cs

@@ -23,7 +23,7 @@ namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
 internal sealed class BrowsingContextInputModule(BrowsingContext context, IInputModule inputModule, EventDispatcher dispatcher) : IBrowsingContextInputModule
 {
-    public Task<PerformActionsResult> PerformActionsAsync(IEnumerable<SourceActions> actions, PerformActionsOptions? options = null, CancellationToken cancellationToken = default)
+    public Task<PerformActionsResult> PerformActionsAsync(ImmutableArray<SourceActions> actions, PerformActionsOptions? options = null, CancellationToken cancellationToken = default)
     {
         return inputModule.PerformActionsAsync(context, actions, options, cancellationToken);
     }
@@ -33,7 +33,7 @@ public Task<ReleaseActionsResult> ReleaseActionsAsync(ReleaseActionsOptions? opt
         return inputModule.ReleaseActionsAsync(context, options, cancellationToken);
     }
 
-    public Task<SetFilesResult> SetFilesAsync(Script.ISharedReference element, IEnumerable<string> files, SetFilesOptions? options = null, CancellationToken cancellationToken = default)
+    public Task<SetFilesResult> SetFilesAsync(Script.ISharedReference element, ImmutableArray<string> files, SetFilesOptions? options = null, CancellationToken cancellationToken = default)
     {
         return inputModule.SetFilesAsync(context, element, files, options, cancellationToken);
     }

dotnet/src/webdriver/BiDi/BrowsingContext/BrowsingContextNetworkModule.cs

@@ -23,7 +23,7 @@ namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
 internal sealed class BrowsingContextNetworkModule(BrowsingContext context, INetworkModule networkModule, EventDispatcher dispatcher) : IBrowsingContextNetworkModule
 {
-    public Task<AddDataCollectorResult> AddDataCollectorAsync(IEnumerable<DataType> dataTypes, int maxEncodedDataSize, ContextAddDataCollectorOptions? options = null, CancellationToken cancellationToken = default)
+    public Task<AddDataCollectorResult> AddDataCollectorAsync(ImmutableArray<DataType> dataTypes, int maxEncodedDataSize, ContextAddDataCollectorOptions? options = null, CancellationToken cancellationToken = default)
     {
         return networkModule.AddDataCollectorAsync(dataTypes, maxEncodedDataSize, ContextAddDataCollectorOptions.WithContext(options, context), cancellationToken);
     }

dotnet/src/webdriver/BiDi/BrowsingContext/ContextCreatedEvent.cs

@@ -21,7 +21,7 @@ namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
 public sealed record ContextCreatedEventArgs(
     IBiDi BiDi,
-    IReadOnlyList<Info>? Children,
+    ImmutableArray<Info>? Children,
     Browser.ClientWindow ClientWindow,
     BrowsingContext Context,
     BrowsingContext? OriginalOpener,

dotnet/src/webdriver/BiDi/BrowsingContext/ContextDestroyedEvent.cs

@@ -19,7 +19,7 @@
 
 namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
-public sealed record ContextDestroyedEventArgs(IBiDi BiDi, IReadOnlyList<Info>? Children,
+public sealed record ContextDestroyedEventArgs(IBiDi BiDi, ImmutableArray<Info>? Children,
     Browser.ClientWindow ClientWindow,
     BrowsingContext Context,
     BrowsingContext? OriginalOpener,

dotnet/src/webdriver/BiDi/BrowsingContext/GetTree.cs

@@ -40,4 +40,4 @@ public sealed record ContextGetTreeOptions : CommandOptions
     };
 }
 
-public sealed record GetTreeResult(IReadOnlyList<Info> Contexts) : EmptyResult;
+public sealed record GetTreeResult(ImmutableArray<Info> Contexts) : EmptyResult;

dotnet/src/webdriver/BiDi/BrowsingContext/IBrowsingContextInputModule.cs

@@ -24,7 +24,7 @@ namespace OpenQA.Selenium.BiDi.BrowsingContext;
 public interface IBrowsingContextInputModule
 {
     IEventSource<FileDialogOpenedEventArgs> FileDialogOpened { get; }
-    Task<PerformActionsResult> PerformActionsAsync(IEnumerable<SourceActions> actions, PerformActionsOptions? options = null, CancellationToken cancellationToken = default);
+    Task<PerformActionsResult> PerformActionsAsync(ImmutableArray<SourceActions> actions, PerformActionsOptions? options = null, CancellationToken cancellationToken = default);
     Task<ReleaseActionsResult> ReleaseActionsAsync(ReleaseActionsOptions? options = null, CancellationToken cancellationToken = default);
-    Task<SetFilesResult> SetFilesAsync(Script.ISharedReference element, IEnumerable<string> files, SetFilesOptions? options = null, CancellationToken cancellationToken = default);
+    Task<SetFilesResult> SetFilesAsync(Script.ISharedReference element, ImmutableArray<string> files, SetFilesOptions? options = null, CancellationToken cancellationToken = default);
 }

dotnet/src/webdriver/BiDi/BrowsingContext/IBrowsingContextNetworkModule.cs

@@ -23,7 +23,7 @@ namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
 public interface IBrowsingContextNetworkModule
 {
-    Task<AddDataCollectorResult> AddDataCollectorAsync(IEnumerable<DataType> dataTypes, int maxEncodedDataSize, ContextAddDataCollectorOptions? options = null, CancellationToken cancellationToken = default);
+    Task<AddDataCollectorResult> AddDataCollectorAsync(ImmutableArray<DataType> dataTypes, int maxEncodedDataSize, ContextAddDataCollectorOptions? options = null, CancellationToken cancellationToken = default);
     IEventSource<AuthRequiredEventArgs> AuthRequired { get; }
     IEventSource<BeforeRequestSentEventArgs> BeforeRequestSent { get; }
     IEventSource<FetchErrorEventArgs> FetchError { get; }

dotnet/src/webdriver/BiDi/BrowsingContext/Info.cs

@@ -20,7 +20,7 @@
 namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
 public sealed record Info(
-    IReadOnlyList<Info>? Children,
+    ImmutableArray<Info>? Children,
     Browser.ClientWindow ClientWindow,
     BrowsingContext Context,
     BrowsingContext? OriginalOpener,

dotnet/src/webdriver/BiDi/BrowsingContext/LocateNodes.cs

@@ -19,15 +19,15 @@
 
 namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
-internal sealed record LocateNodesParameters(BrowsingContext Context, Locator Locator, long? MaxNodeCount, Script.SerializationOptions? SerializationOptions, IEnumerable<Script.ISharedReference>? StartNodes) : Parameters;
+internal sealed record LocateNodesParameters(BrowsingContext Context, Locator Locator, long? MaxNodeCount, Script.SerializationOptions? SerializationOptions, ImmutableArray<Script.ISharedReference>? StartNodes) : Parameters;
 
 public sealed record LocateNodesOptions : CommandOptions
 {
     public long? MaxNodeCount { get; init; }
 
     public Script.SerializationOptions? SerializationOptions { get; init; }
 
-    public IEnumerable<Script.ISharedReference>? StartNodes { get; init; }
+    public ImmutableArray<Script.ISharedReference>? StartNodes { get; init; }
 }
 
-public sealed record LocateNodesResult(IReadOnlyList<Script.NodeRemoteValue> Nodes) : EmptyResult;
+public sealed record LocateNodesResult(ImmutableArray<Script.NodeRemoteValue> Nodes) : EmptyResult;

dotnet/src/webdriver/BiDi/BrowsingContext/Print.cs

@@ -22,7 +22,7 @@
 
 namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
-internal sealed record PrintParameters(BrowsingContext Context, bool? Background, PrintMargin? Margin, PrintOrientation? Orientation, PrintPage? Page, IEnumerable<PrintPageRange>? PageRanges, double? Scale, bool? ShrinkToFit) : Parameters;
+internal sealed record PrintParameters(BrowsingContext Context, bool? Background, PrintMargin? Margin, PrintOrientation? Orientation, PrintPage? Page, ImmutableArray<PrintPageRange>? PageRanges, double? Scale, bool? ShrinkToFit) : Parameters;
 
 public sealed record PrintOptions : CommandOptions
 {
@@ -34,7 +34,7 @@ public sealed record PrintOptions : CommandOptions
 
     public PrintPage? Page { get; init; }
 
-    public IEnumerable<PrintPageRange>? PageRanges { get; init; }
+    public ImmutableArray<PrintPageRange>? PageRanges { get; init; }
 
     public double? Scale { get; init; }
 

dotnet/src/webdriver/BiDi/BrowsingContext/SetViewport.cs

@@ -26,7 +26,7 @@ internal sealed record SetViewportParameters(
     BrowsingContext? Context,
     [property: JsonConverter(typeof(OptionalConverter<Viewport?>))] Optional<Viewport?>? Viewport,
     [property: JsonConverter(typeof(OptionalConverter<double?>))] Optional<double?>? DevicePixelRatio,
-    IEnumerable<Browser.UserContext>? UserContexts)
+    ImmutableArray<Browser.UserContext>? UserContexts)
     : Parameters;
 
 public sealed record SetViewportOptions : CommandOptions
@@ -37,7 +37,7 @@ public sealed record SetViewportOptions : CommandOptions
 
     public Optional<double?>? DevicePixelRatio { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 public sealed record ContextSetViewportOptions : CommandOptions

dotnet/src/webdriver/BiDi/Emulation/SetForcedColorsModeThemeOverride.cs

@@ -22,13 +22,13 @@
 
 namespace OpenQA.Selenium.BiDi.Emulation;
 
-internal sealed record SetForcedColorsModeThemeOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ForcedColorsModeTheme? Theme, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal sealed record SetForcedColorsModeThemeOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ForcedColorsModeTheme? Theme, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
 public sealed record SetForcedColorsModeThemeOverrideOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 [JsonConverter(typeof(CamelCaseEnumConverter<ForcedColorsModeTheme>))]

dotnet/src/webdriver/BiDi/Emulation/SetGeolocationOverride.cs

@@ -36,11 +36,11 @@ public sealed record GeolocationPositionErrorOverride : GeolocationOverride;
 
 [JsonDerivedType(typeof(SetGeolocationOverrideCoordinatesParameters))]
 [JsonDerivedType(typeof(SetGeolocationOverridePositionErrorParameters))]
-internal abstract record SetGeolocationOverrideParameters(IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal abstract record SetGeolocationOverrideParameters(ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
-internal sealed record SetGeolocationOverrideCoordinatesParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] GeolocationCoordinates? Coordinates, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : SetGeolocationOverrideParameters(Contexts, UserContexts);
+internal sealed record SetGeolocationOverrideCoordinatesParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] GeolocationCoordinates? Coordinates, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : SetGeolocationOverrideParameters(Contexts, UserContexts);
 
-internal sealed record SetGeolocationOverridePositionErrorParameters(GeolocationPositionError Error, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : SetGeolocationOverrideParameters(Contexts, UserContexts);
+internal sealed record SetGeolocationOverridePositionErrorParameters(GeolocationPositionError Error, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : SetGeolocationOverrideParameters(Contexts, UserContexts);
 
 internal sealed record GeolocationCoordinates(double Latitude, double Longitude, double? Accuracy, double? Altitude, double? AltitudeAccuracy, double? Heading, double? Speed);
 
@@ -52,9 +52,9 @@ internal sealed record GeolocationPositionError
 
 public sealed record SetGeolocationOverrideOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 public sealed record SetGeolocationOverrideResult : EmptyResult;

dotnet/src/webdriver/BiDi/Emulation/SetLocaleOverride.cs

@@ -21,13 +21,13 @@
 
 namespace OpenQA.Selenium.BiDi.Emulation;
 
-internal sealed record SetLocaleOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] string? Locale, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal sealed record SetLocaleOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] string? Locale, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
 public sealed record SetLocaleOverrideOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 public sealed record SetLocaleOverrideResult : EmptyResult;

dotnet/src/webdriver/BiDi/Emulation/SetNetworkConditions.cs

@@ -21,7 +21,7 @@
 
 namespace OpenQA.Selenium.BiDi.Emulation;
 
-internal sealed record SetNetworkConditionsParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] NetworkConditions? NetworkConditions, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal sealed record SetNetworkConditionsParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] NetworkConditions? NetworkConditions, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
 [JsonPolymorphic(TypeDiscriminatorPropertyName = "type")]
 [JsonDerivedType(typeof(NetworkConditionsOffline), "offline")]
@@ -31,9 +31,9 @@ public sealed record NetworkConditionsOffline : NetworkConditions;
 
 public sealed record SetNetworkConditionsOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 public sealed record SetNetworkConditionsResult : EmptyResult;

dotnet/src/webdriver/BiDi/Emulation/SetScreenOrientationOverride.cs

@@ -22,13 +22,13 @@
 
 namespace OpenQA.Selenium.BiDi.Emulation;
 
-internal sealed record SetScreenOrientationOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ScreenOrientation? ScreenOrientation, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal sealed record SetScreenOrientationOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ScreenOrientation? ScreenOrientation, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
 public sealed record SetScreenOrientationOverrideOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 [JsonConverter(typeof(CamelCaseEnumConverter<ScreenOrientationNatural>))]

dotnet/src/webdriver/BiDi/Emulation/SetScreenSettingsOverride.cs

@@ -21,13 +21,13 @@
 
 namespace OpenQA.Selenium.BiDi.Emulation;
 
-internal sealed record SetScreenSettingsOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ScreenArea? ScreenArea, IEnumerable<BrowsingContext.BrowsingContext>? Contexts, IEnumerable<Browser.UserContext>? UserContexts) : Parameters;
+internal sealed record SetScreenSettingsOverrideParameters([property: JsonIgnore(Condition = JsonIgnoreCondition.Never)] ScreenArea? ScreenArea, ImmutableArray<BrowsingContext.BrowsingContext>? Contexts, ImmutableArray<Browser.UserContext>? UserContexts) : Parameters;
 
 public sealed record SetScreenSettingsOverrideOptions : CommandOptions
 {
-    public IEnumerable<BrowsingContext.BrowsingContext>? Contexts { get; init; }
+    public ImmutableArray<BrowsingContext.BrowsingContext>? Contexts { get; init; }
 
-    public IEnumerable<Browser.UserContext>? UserContexts { get; init; }
+    public ImmutableArray<Browser.UserContext>? UserContexts { get; init; }
 }
 
 public sealed record ScreenArea(long Width, long Height);