[dotnet] [bidi] Protect DTO types from inheritance (#15919)

Author: nvborisenko

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

@@ -19,7 +19,7 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-public record ClientWindow
+public sealed record ClientWindow
 {
     internal ClientWindow(string id)
     {

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

@@ -19,7 +19,7 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-public record ClientWindowInfo(bool Active, ClientWindow ClientWindow, ClientWindowState State, int Height, int Width, int X, int Y);
+public sealed record ClientWindowInfo(bool Active, ClientWindow ClientWindow, ClientWindowState State, int Height, int Width, int X, int Y);
 
 public enum ClientWindowState
 {

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

@@ -21,7 +21,7 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal class CloseCommand()
+internal sealed class CloseCommand()
     : Command<CommandParameters, EmptyResult>(CommandParameters.Empty, "browser.close");
 
-public record CloseOptions : CommandOptions;
+public sealed class CloseOptions : CommandOptions;

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

@@ -21,12 +21,12 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal class CreateUserContextCommand(CreateUserContextCommandParameters @params)
+internal sealed class CreateUserContextCommand(CreateUserContextCommandParameters @params)
     : Command<CreateUserContextCommandParameters, UserContextInfo>(@params, "browser.createUserContext");
 
-internal record CreateUserContextCommandParameters(bool? AcceptInsecureCerts, Session.ProxyConfiguration? Proxy) : CommandParameters;
+internal sealed record CreateUserContextCommandParameters(bool? AcceptInsecureCerts, Session.ProxyConfiguration? Proxy) : CommandParameters;
 
-public record CreateUserContextOptions : CommandOptions
+public sealed class CreateUserContextOptions : CommandOptions
 {
     public bool? AcceptInsecureCerts { get; set; }
 

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

@@ -23,12 +23,12 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal class GetClientWindowsCommand()
+internal sealed class GetClientWindowsCommand()
     : Command<CommandParameters, GetClientWindowsResult>(CommandParameters.Empty, "browser.getClientWindows");
 
-public record GetClientWindowsOptions : CommandOptions;
+public sealed class GetClientWindowsOptions : CommandOptions;
 
-public record GetClientWindowsResult : EmptyResult, IReadOnlyList<ClientWindowInfo>
+public sealed record GetClientWindowsResult : EmptyResult, IReadOnlyList<ClientWindowInfo>
 {
     private readonly IReadOnlyList<ClientWindowInfo> _clientWindows;
 

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

@@ -23,12 +23,12 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal class GetUserContextsCommand()
+internal sealed class GetUserContextsCommand()
     : Command<CommandParameters, GetUserContextsResult>(CommandParameters.Empty, "browser.getUserContexts");
 
-public record GetUserContextsOptions : CommandOptions;
+public class GetUserContextsOptions : CommandOptions;
 
-public record GetUserContextsResult : EmptyResult, IReadOnlyList<UserContextInfo>
+public sealed record GetUserContextsResult : EmptyResult, IReadOnlyList<UserContextInfo>
 {
     private readonly IReadOnlyList<UserContextInfo> _userContexts;
 

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

@@ -21,9 +21,9 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-internal class RemoveUserContextCommand(RemoveUserContextCommandParameters @params)
+internal sealed class RemoveUserContextCommand(RemoveUserContextCommandParameters @params)
     : Command<RemoveUserContextCommandParameters, EmptyResult>(@params, "browser.removeUserContext");
 
-internal record RemoveUserContextCommandParameters(UserContext UserContext) : CommandParameters;
+internal sealed record RemoveUserContextCommandParameters(UserContext UserContext) : CommandParameters;
 
-public record RemoveUserContextOptions : CommandOptions;
+public sealed class RemoveUserContextOptions : CommandOptions;

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

@@ -22,7 +22,7 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-public class UserContext : IAsyncDisposable
+public sealed class UserContext : IAsyncDisposable
 {
     private readonly BiDi _bidi;
 

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

@@ -21,4 +21,4 @@
 
 namespace OpenQA.Selenium.BiDi.Browser;
 
-public record UserContextInfo(UserContext UserContext) : EmptyResult;
+public sealed record UserContextInfo(UserContext UserContext) : EmptyResult;

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

@@ -21,9 +21,9 @@
 
 namespace OpenQA.Selenium.BiDi.BrowsingContext;
 
-internal class ActivateCommand(ActivateCommandParameters @params)
+internal sealed class ActivateCommand(ActivateCommandParameters @params)
     : Command<ActivateCommandParameters, EmptyResult>(@params, "browsingContext.activate");
 
-internal record ActivateCommandParameters(BrowsingContext Context) : CommandParameters;
+internal sealed record ActivateCommandParameters(BrowsingContext Context) : CommandParameters;
 
-public record ActivateOptions : CommandOptions;
+public sealed class ActivateOptions : CommandOptions;