[dotnet] [bidi] Simplify namespace for communications (#16602)

Author: nvborisenko

dotnet/src/webdriver/BiDi/BiDi.cs

@@ -17,14 +17,13 @@
 // under the License.
 // </copyright>
 
+using OpenQA.Selenium.BiDi.Json;
+using OpenQA.Selenium.BiDi.Json.Converters;
 using System;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 using System.Threading;
 using System.Threading.Tasks;
-using OpenQA.Selenium.BiDi.Communication;
-using OpenQA.Selenium.BiDi.Communication.Json;
-using OpenQA.Selenium.BiDi.Communication.Json.Converters;
 
 namespace OpenQA.Selenium.BiDi;
 

dotnet/src/webdriver/BiDi/Communication/Broker.cs renamed to dotnet/src/webdriver/BiDi/Broker.cs

@@ -17,7 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication.Transport;
 using OpenQA.Selenium.Internal.Logging;
 using System;
 using System.Collections.Concurrent;
@@ -28,7 +27,7 @@
 using System.Threading;
 using System.Threading.Tasks;
 
-namespace OpenQA.Selenium.BiDi.Communication;
+namespace OpenQA.Selenium.BiDi;
 
 public sealed class Broker : IAsyncDisposable
 {
@@ -214,13 +213,13 @@ public async Task<Subscription> SubscribeAsync<TEventArgs>(string eventName, Fun
         }
     }
 
-    public async Task UnsubscribeAsync(Session.Subscription subscription, EventHandler eventHandler)
+    public async Task UnsubscribeAsync(Subscription subscription)
     {
-        var eventHandlers = _eventHandlers[eventHandler.EventName];
+        var eventHandlers = _eventHandlers[subscription.EventHandler.EventName];
 
-        eventHandlers.Remove(eventHandler);
+        eventHandlers.Remove(subscription.EventHandler);
 
-        await _bidi.SessionModule.UnsubscribeAsync([subscription]).ConfigureAwait(false);
+        await _bidi.SessionModule.UnsubscribeAsync([subscription.SubscriptionId]).ConfigureAwait(false);
     }
 
     public async ValueTask DisposeAsync()

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

@@ -17,7 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
 using System.Threading.Tasks;
 
 namespace OpenQA.Selenium.BiDi.Browser;

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

@@ -17,7 +17,7 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication.Json.Converters;
+using OpenQA.Selenium.BiDi.Json.Converters;
 using System.Text.Json.Serialization;
 
 namespace OpenQA.Selenium.BiDi.Browser;

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

@@ -17,7 +17,7 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication.Json.Converters;
+using OpenQA.Selenium.BiDi.Json.Converters;
 using System.Text.Json.Serialization;
 
 namespace OpenQA.Selenium.BiDi.Browser;

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

@@ -17,8 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
-
 namespace OpenQA.Selenium.BiDi.Browser;
 
 internal sealed class CloseCommand()

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

@@ -17,8 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
-
 namespace OpenQA.Selenium.BiDi.Browser;
 
 internal sealed class CreateUserContextCommand(CreateUserContextParameters @params)

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

@@ -17,7 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
 using System.Collections.Generic;
 
 namespace OpenQA.Selenium.BiDi.Browser;

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

@@ -17,7 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
 using System.Collections.Generic;
 
 namespace OpenQA.Selenium.BiDi.Browser;

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

@@ -17,8 +17,6 @@
 // under the License.
 // </copyright>
 
-using OpenQA.Selenium.BiDi.Communication;
-
 namespace OpenQA.Selenium.BiDi.Browser;
 
 internal sealed class RemoveUserContextCommand(RemoveUserContextParameters @params)