Skip to content

Commit facc117

Browse files
aguspeaguspe
authored
Merge branch 'trunk' into relax-ruby-base64-dependency-version
2 parents b9a81a8 + 3c87e6b commit facc117

31 files changed

+143
-433
lines changed

‎MODULE.bazel‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ bazel_dep(name = "platforms", version = "0.0.11")
1515
bazel_dep(name = "protobuf", version = "29.2", dev_dependency = True, repo_name = "com_google_protobuf")
1616

1717
# Required for rules_rust to import the crates properly
18-
bazel_dep(name = "rules_cc", version = "0.1.0", dev_dependency = True)
18+
bazel_dep(name = "rules_cc", version = "0.1.1", dev_dependency = True)
1919

2020
bazel_dep(name = "rules_dotnet", version = "0.17.5")
2121
bazel_dep(name = "rules_java", version = "8.7.1")

‎dotnet/src/webdriver/BiDi/WebDriver.Extensions.cs‎

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

20-
using OpenQA.Selenium.BiDi.Modules.BrowsingContext;
2120
using System;
2221
using System.Threading.Tasks;
2322

@@ -44,13 +43,4 @@ public static async Task<BiDi> AsBiDiAsync(this IWebDriver webDriver)
4443

4544
return bidi;
4645
}
47-
48-
public static async Task<BrowsingContext> AsBiDiContextAsync(this IWebDriver webDriver)
49-
{
50-
var bidi = await webDriver.AsBiDiAsync();
51-
52-
var currentBrowsingContext = new BrowsingContext(bidi, webDriver.CurrentWindowHandle);
53-
54-
return currentBrowsingContext;
55-
}
5646
}

‎dotnet/src/webdriver/DevTools/v130/V130Network.cs‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -338,27 +338,27 @@ private void OnFetchRequestPaused(object? sender, Fetch.RequestPausedEventArgs e
338338
if (e.ResponseErrorReason == null && e.ResponseStatusCode == null)
339339
{
340340
var requestData = new HttpRequestData
341-
(
342-
requestId: e.RequestId,
343-
method: e.Request.Method,
344-
url: e.Request.Url,
345-
postData: e.Request.PostData,
346-
headers: new Dictionary<string, string>(e.Request.Headers)
347-
);
341+
{
342+
RequestId = e.RequestId,
343+
Method = e.Request.Method,
344+
Url = e.Request.Url,
345+
PostData = e.Request.PostData,
346+
Headers = new Dictionary<string, string>(e.Request.Headers)
347+
};
348348

349349
RequestPausedEventArgs wrapped = new RequestPausedEventArgs(null, requestData);
350350
this.OnRequestPaused(wrapped);
351351
}
352352
else
353353
{
354-
var responseData = new HttpResponseData
355-
(
356-
requestId: e.RequestId,
357-
url: e.Request.Url,
358-
resourceType: e.ResourceType.ToString(),
359-
statusCode: e.ResponseStatusCode.GetValueOrDefault(),
360-
errorReason: e.ResponseErrorReason?.ToString()
361-
);
354+
var responseData = new HttpResponseData()
355+
{
356+
RequestId = e.RequestId,
357+
Url = e.Request.Url,
358+
ResourceType = e.ResourceType.ToString(),
359+
StatusCode = e.ResponseStatusCode.GetValueOrDefault(),
360+
ErrorReason = e.ResponseErrorReason?.ToString()
361+
};
362362

363363
if (e.ResponseHeaders != null)
364364
{

‎dotnet/src/webdriver/DevTools/v131/V131Network.cs‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -338,27 +338,27 @@ private void OnFetchRequestPaused(object? sender, Fetch.RequestPausedEventArgs e
338338
if (e.ResponseErrorReason == null && e.ResponseStatusCode == null)
339339
{
340340
var requestData = new HttpRequestData
341-
(
342-
requestId: e.RequestId,
343-
method: e.Request.Method,
344-
url: e.Request.Url,
345-
postData: e.Request.PostData,
346-
headers: new Dictionary<string, string>(e.Request.Headers)
347-
);
341+
{
342+
RequestId = e.RequestId,
343+
Method = e.Request.Method,
344+
Url = e.Request.Url,
345+
PostData = e.Request.PostData,
346+
Headers = new Dictionary<string, string>(e.Request.Headers)
347+
};
348348

349349
RequestPausedEventArgs wrapped = new RequestPausedEventArgs(null, requestData);
350350
this.OnRequestPaused(wrapped);
351351
}
352352
else
353353
{
354-
var responseData = new HttpResponseData
355-
(
356-
requestId: e.RequestId,
357-
url: e.Request.Url,
358-
resourceType: e.ResourceType.ToString(),
359-
statusCode: e.ResponseStatusCode.GetValueOrDefault(),
360-
errorReason: e.ResponseErrorReason?.ToString()
361-
);
354+
var responseData = new HttpResponseData()
355+
{
356+
RequestId = e.RequestId,
357+
Url = e.Request.Url,
358+
ResourceType = e.ResourceType.ToString(),
359+
StatusCode = e.ResponseStatusCode.GetValueOrDefault(),
360+
ErrorReason = e.ResponseErrorReason?.ToString()
361+
};
362362

363363
if (e.ResponseHeaders != null)
364364
{

‎dotnet/src/webdriver/DevTools/v132/V132Network.cs‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -338,27 +338,27 @@ private void OnFetchRequestPaused(object? sender, Fetch.RequestPausedEventArgs e
338338
if (e.ResponseErrorReason == null && e.ResponseStatusCode == null)
339339
{
340340
var requestData = new HttpRequestData
341-
(
342-
requestId: e.RequestId,
343-
method: e.Request.Method,
344-
url: e.Request.Url,
345-
postData: e.Request.PostData,
346-
headers: new Dictionary<string, string>(e.Request.Headers)
347-
);
341+
{
342+
RequestId = e.RequestId,
343+
Method = e.Request.Method,
344+
Url = e.Request.Url,
345+
PostData = e.Request.PostData,
346+
Headers = new Dictionary<string, string>(e.Request.Headers)
347+
};
348348

349349
RequestPausedEventArgs wrapped = new RequestPausedEventArgs(null, requestData);
350350
this.OnRequestPaused(wrapped);
351351
}
352352
else
353353
{
354-
var responseData = new HttpResponseData
355-
(
356-
requestId: e.RequestId,
357-
url: e.Request.Url,
358-
resourceType: e.ResourceType.ToString(),
359-
statusCode: e.ResponseStatusCode.GetValueOrDefault(),
360-
errorReason: e.ResponseErrorReason?.ToString()
361-
);
354+
var responseData = new HttpResponseData()
355+
{
356+
RequestId = e.RequestId,
357+
Url = e.Request.Url,
358+
ResourceType = e.ResourceType.ToString(),
359+
StatusCode = e.ResponseStatusCode.GetValueOrDefault(),
360+
ErrorReason = e.ResponseErrorReason?.ToString()
361+
};
362362

363363
if (e.ResponseHeaders != null)
364364
{

‎dotnet/src/webdriver/DevTools/v85/V85Network.cs‎

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -335,27 +335,27 @@ private void OnFetchRequestPaused(object? sender, Fetch.RequestPausedEventArgs e
335335
if (e.ResponseErrorReason == null && e.ResponseStatusCode == null)
336336
{
337337
var requestData = new HttpRequestData
338-
(
339-
requestId: e.RequestId,
340-
method: e.Request.Method,
341-
url: e.Request.Url,
342-
postData: e.Request.PostData,
343-
headers: new Dictionary<string, string>(e.Request.Headers)
344-
);
338+
{
339+
RequestId = e.RequestId,
340+
Method = e.Request.Method,
341+
Url = e.Request.Url,
342+
PostData = e.Request.PostData,
343+
Headers = new Dictionary<string, string>(e.Request.Headers)
344+
};
345345

346346
RequestPausedEventArgs wrapped = new RequestPausedEventArgs(null, requestData);
347347
this.OnRequestPaused(wrapped);
348348
}
349349
else
350350
{
351351
var responseData = new HttpResponseData
352-
(
353-
requestId: e.RequestId,
354-
url: e.Request.Url,
355-
resourceType: e.ResourceType.ToString(),
356-
statusCode: e.ResponseStatusCode.GetValueOrDefault(),
357-
errorReason: e.ResponseErrorReason?.ToString()
358-
);
352+
{
353+
RequestId = e.RequestId,
354+
Url = e.Request.Url,
355+
ResourceType = e.ResourceType.ToString(),
356+
StatusCode = e.ResponseStatusCode.GetValueOrDefault(),
357+
ErrorReason = e.ResponseErrorReason?.ToString()
358+
};
359359

360360
if (e.ResponseHeaders != null)
361361
{

‎dotnet/src/webdriver/HttpRequestData.cs‎

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,6 @@ public HttpRequestData()
3535
{
3636
}
3737

38-
internal HttpRequestData(string? method, string? url, string? postData, Dictionary<string, string>? headers, string? requestId)
39-
{
40-
this.Method = method;
41-
this.Url = url;
42-
this.PostData = postData;
43-
this.Headers = headers;
44-
this.RequestId = requestId;
45-
}
46-
4738
/// <summary>
4839
/// Gets the method of the HTTP request.
4940
/// </summary>
@@ -67,6 +58,6 @@ internal HttpRequestData(string? method, string? url, string? postData, Dictiona
6758
/// <summary>
6859
/// Gets the ID of the HTTP request.
6960
/// </summary>
70-
public string? RequestId { get; }
61+
public string? RequestId { get; internal set; }
7162
}
7263
}

‎dotnet/src/webdriver/HttpResponseData.cs‎

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,29 +32,19 @@ public class HttpResponseData
3232
/// <summary>
3333
/// Initializes a new instance of the <see cref="HttpResponseData"/> type.
3434
/// </summary>
35-
/// <param name="requestId">The ID of the request that generated this response.</param>
36-
/// <param name="url">The URL of the HTTP response.</param>
37-
/// <param name="resourceType">The type of resource for this response.</param>
38-
/// <param name="statusCode">The numeric status code of the HTTP response.</param>
39-
/// <param name="errorReason">The reason for an error response.</param>
40-
public HttpResponseData(string requestId, string url, string resourceType, long statusCode, string? errorReason)
35+
public HttpResponseData()
4136
{
42-
RequestId = requestId;
43-
Url = url;
44-
ResourceType = resourceType;
45-
StatusCode = statusCode;
46-
ErrorReason = errorReason;
4737
}
4838

4939
/// <summary>
5040
/// Gets or sets the ID of the request that generated this response.
5141
/// </summary>
52-
public string RequestId { get; set; }
42+
public string? RequestId { get; set; }
5343

5444
/// <summary>
5545
/// Gets or sets the URL of the HTTP response.
5646
/// </summary>
57-
public string Url { get; set; }
47+
public string? Url { get; set; }
5848

5949
/// <summary>
6050
/// Gets or sets the numeric status code of the HTTP response.
@@ -79,7 +69,7 @@ public string? Body
7969
/// <summary>
8070
/// Gets or sets the type of resource for this response.
8171
/// </summary>
82-
public string ResourceType { get; set; }
72+
public string? ResourceType { get; set; }
8373

8474
/// <summary>
8575
/// Gets or sets the reason for an error response.

‎dotnet/src/webdriver/NetworkManager.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ private async Task OnResponsePaused(object sender, ResponsePausedEventArgs e)
262262
// method demands one; however, the only property used by that method is the RequestId property.
263263
// It might be better to refactor that method signature to simply pass the request ID, or
264264
// alternatively, just pass the response data, which should also contain the request ID anyway.
265-
HttpRequestData requestData = new HttpRequestData(null, null, null, null, requestId: e.ResponseData.RequestId);
265+
HttpRequestData requestData = new HttpRequestData { RequestId = e.ResponseData.RequestId };
266266
await this.session.Value.Domains.Network.ContinueRequestWithResponse(requestData, handler.ResponseTransformer(e.ResponseData)).ConfigureAwait(false);
267267
return;
268268
}

‎dotnet/src/webdriver/NetworkResponseReceivedEventArgs.cs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,12 @@ public NetworkResponseReceivedEventArgs(HttpResponseData responseData)
5151
/// <summary>
5252
/// Gets the request ID of the network request that generated this response.
5353
/// </summary>
54-
public string RequestId { get; }
54+
public string? RequestId { get; }
5555

5656
/// <summary>
5757
/// Gets the URL of the network response.
5858
/// </summary>
59-
public string ResponseUrl { get; }
59+
public string? ResponseUrl { get; }
6060

6161
/// <summary>
6262
/// Gets the HTTP status code of the network response.
@@ -79,7 +79,7 @@ public NetworkResponseReceivedEventArgs(HttpResponseData responseData)
7979
/// <summary>
8080
/// Gets the type of resource of the network response.
8181
/// </summary>
82-
public string ResponseResourceType { get; }
82+
public string? ResponseResourceType { get; }
8383

8484
/// <summary>
8585
/// Gets the headers associated with this network response.

0 commit comments

Comments
 (0)