Resolving Comments

Author: ashuray007

specs/WebRtcPortConfiguration.md

@@ -23,8 +23,37 @@ Common scenarios:
 
 Usage steps:  
 1. Create `CoreWebView2EnvironmentOptions`.   
-2. Call `SetAllowedPortRange` for `COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C` and `COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP`.  
-3. Pass the options when creating the WebView2 environment.  
+2. Call `SetAllowedPortRange` for `COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC` and `COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP`.  
+3. Pass the options when creating the WebView2 environment.
+
+API Rules and Precedence
+
+1. Network Scope param in SetAllowedPortRange
+- A network component-specific scope (e.g. _WEB_RTC) always takes precedence over _ALL for that component in `SetAllowedPortRange`.
+- `_ALL` defines the port range restrictions for all components without specific overrides.
+- Passing `(0, 0)` for a network component scope removes its restriction.
+- If `_ALL` is set and a specific scope is reset, that component becomes unrestricted while `_ALL` still applies to others.
+
+| Network Scope State                        | Behaviour                                                                      |
+| ------------------------------------------ | ------------------------------------------------------------------------------ |
+| Only `_ALL` is set                         | `_ALL` applies port range restrictions to all network components               |
+| `_ALL` and `_WEB_RTC` are both set         | `_WEB_RTC` port range restrictions applies to WebRTC; `_ALL` applies to others |
+| `_WEB_RTC` only is set                     | `_WEB_RTC` applies port range restrictions only to WebRTC; others unrestricted | 
+| `_ALL` set and `_WEB_RTC` reset to `(0,0)` | `_ALL` applies port range restrictions to all except WebRTC (unrestricted)     |
+
+2. Network Scope param in GetAllowedPortRange
+- `GetAllowedPortRange` returns the range explicitly set for the queried scope.
+- If a specific scope is unset, it inherits `_ALL`.
+- Querying `_ALL` only returns `_ALL`; it does not aggregate component-specific settings.
+- If neither `_ALL` nor a component-specific scope is set, the default `(0,0)` (unrestricted) is returned.
+
+| `GetAllowedPortRange` Network Scope query                       | Returned Range                |
+| --------------------------------------------------------------- | ----------------------------- |
+| Pass `_WEB_RTC` when only `_ALL` is set                         | Returns `_ALL` range          |
+| Pass `_WEB_RTC` when `_WEB_RTC` explicitly set                  | Returns `_WEB_RTC` range      |
+| Pass `_WEB_RTC` when `_ALL` unset and `_WEB_RTC` unset           | Returns `(0, 0)`             |
+| Pass `_WEB_RTC` when `_ALL` set and `_WEB_RTC` reset to `(0, 0)` | Returns `(0, 0)`             |
+| Pass `_ALL` when only `_WEB_RTC` set                            | Returns  `(0,0)`              |
 
 
 # Examples
@@ -38,12 +67,12 @@ if (options.As(&optionsStaging10) == S_OK)
     const INT32 udpMin = 50000, udpMax = 55000;
 
     CHECK_FAILURE(optionsStaging10->SetAllowedPortRange(
-        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C,
+        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC,
         COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP, udpMin, udpMax));
 
     // Get the configured port range
     CHECK_FAILURE(optionsStaging10->GetAllowedPortRange(
-        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C,
+        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC,
         COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP, &m_udpPortRange.minPort,
         &m_udpPortRange.maxPort));
 }
@@ -66,12 +95,12 @@ if (optionsStaging10 != null)
     const int udpMin = 50000, udpMax = 55000;
 
     optionsStaging10.SetAllowedPortRange(
-        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C,
+        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC,
         COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP, udpMin, udpMax);
 
     // Get the configured port range
     optionsStaging10.GetAllowedPortRange(
-        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C,
+        COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC,
         COREWEBVIEW2_TRANSPORT_PROTOCOL_KIND_UDP, out m_udpPortRange.minPort,
         out m_udpPortRange.maxPort);
 }
@@ -90,7 +119,7 @@ typedef enum COREWEBVIEW2_NETWORK_COMPONENT_SCOPE {
   /// Scope applies to all components.
   COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_ALL,
   /// Applies only to WebRTC peer-to-peer connection.
-  COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_R_T_C,
+  COREWEBVIEW2_NETWORK_COMPONENT_SCOPE_WEB_RTC,
 } COREWEBVIEW2_NETWORK_COMPONENT_SCOPE;
 
 /// Specifies the network protocol for port configuration.
@@ -109,7 +138,7 @@ interface ICoreWebView2StagingEnvironmentOptions10 : IUnknown {
   /// This API enables WebView2 to operate within enterprise network or firewall
   /// restrictions by limiting network communication to a defined port range.
   /// It provides fine-grained control by allowing port restrictions to be applied
-  /// per network component scope, such as WebRTC or QUIC.
+  /// per network component scope, such as WebRTC.
   /// 
   /// Currently, only WebRTC UDP Port Range restriction is supported.
   /// 
@@ -121,7 +150,19 @@ interface ICoreWebView2StagingEnvironmentOptions10 : IUnknown {
   /// are applied and the system assigns ports from the full ephemeral range.
   /// 
   /// Calls with invalid ranges fail with `E_INVALIDARG`.
-  /// 
+  ///
+  /// A network component-specific scope (e.g. _WEB_RTC) always takes precedence over _ALL for that component in `SetAllowedPortRange`.
+  /// `_ALL` defines the port range restrictions for all components without specific overrides.
+  ///  Passing `(0, 0)` for a network component scope removes its restriction.
+  ///  If `_ALL` is set and a specific scope is reset, that component becomes unrestricted while `_ALL` still applies to others.
+
+  /// | Network Scope State                        | Behaviour                                                                      |
+  /// | ------------------------------------------ | ------------------------------------------------------------------------------ |
+  /// | Only `_ALL` is set                         | `_ALL` applies port range restrictions to all network components               |
+  /// | `_ALL` and `_WEB_RTC` are both set         | `_WEB_RTC` port range restrictions applies to WebRTC; `_ALL` applies to others |
+  /// | `_WEB_RTC` only is set                     | `_WEB_RTC` applies port range restrictions only to WebRTC; others unrestricted | 
+  /// | `_ALL` set and `_WEB_RTC` reset to `(0,0)` | `_ALL` applies port range restrictions to all except WebRTC (unrestricted)     |
+
   /// `scope` Network scope on which restrictions will apply.
   /// `protocol` Transport protocol on which restrictions will apply.
   /// `minPort` The minimum allowed port number (inclusive).
@@ -140,7 +181,20 @@ interface ICoreWebView2StagingEnvironmentOptions10 : IUnknown {
   /// 
   /// By default, `(0, 0)` is returned, which indicates no restrictions are applied
   /// and ports are allocated from the system’s ephemeral range (1025–65535 inclusive).
-  /// 
+  ///
+  /// `GetAllowedPortRange` returns the range explicitly set for the queried scope.
+  /// If a specific scope is unset, it inherits `_ALL`.
+  /// Querying `_ALL` only returns `_ALL`; it does not aggregate component-specific settings.
+  /// If neither `_ALL` nor a component-specific scope is set, the default `(0,0)` (unrestricted) is returned.
+
+  /// | `GetAllowedPortRange` Network Scope query                       | Returned Range                |
+  /// | --------------------------------------------------------------- | ----------------------------- |
+  /// | Pass `_WEB_RTC` when only `_ALL` is set                         | Returns `_ALL` range          |
+  /// | Pass `_WEB_RTC` when `_WEB_RTC` explicitly set                  | Returns `_WEB_RTC` range      |
+  /// | Pass `_WEB_RTC` when `_ALL` unset and `_WEB_RTC` unset           | Returns `(0, 0)`             |
+  /// | Pass `_WEB_RTC` when `_ALL` set and `_WEB_RTC` reset to `(0, 0)` | Returns `(0, 0)`             |
+  /// | Pass `_ALL` when only `_WEB_RTC` set                            | Returns  `(0,0)`              |
+
   /// `scope` Network scope on which restrictions is applied.
   /// `protocol` Transport protocol on which restrictions is applied.
   /// `minPort` Receives the minimum allowed port number (inclusive).
@@ -164,7 +218,7 @@ namespace Microsoft.Web.WebView2.Core
     enum CoreWebview2NetworkComponentScope
     {
         All = 0,
-        WebRTC = 1,
+        WebRtc = 1,
     };
 
     enum CoreWebView2TransportProtocolKind