[dotnet][py][java][js]: add default as a value for SameSite

[navin772]

User description

🔗 Related Issues

Fixes #16128

💥 What does this PR do?

Adds default as a possible value for the Samesite attribute.

🔧 Implementation Notes

💡 Additional Considerations

🔄 Types of changes

• New feature (non-breaking change which adds functionality and tests!)

---

PR Type

Enhancement

---

Description

• Add Default value to SameSite enum for cookies

• Implements cookie specification compliance across language bindings

---

Diagram Walkthrough

flowchart LR
  A["SameSite Enum"] --> B["Add Default Value"]
  B --> C["Cookie Spec Compliance"]

Loading

File Walkthrough

	Relevant files
Enhancement	Cookie.cs Add Default to SameSite enum                                                          dotnet/src/webdriver/BiDi/Network/Cookie.cs Add Default enum value to SameSite enumeration +2/-1

---

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶 16128 - Partially compliant Compliant requirements: • Add default as a possible value for the SameSite attribute (for dotnet binding) Non-compliant requirements: • Implement cookie specification compliance across language bindings (only dotnet shown) • Support for dotnet, py, java, and js language bindings (only dotnet implementation visible) Requires further human verification: • Verification that similar changes are implemented in py, java, and js bindings • Testing that the Default value works correctly with actual cookie operations

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Missing Tests The PR adds a new enum value but doesn't include any tests to verify the Default value works correctly in cookie operations or serialization scenarios. Default }

[qodo-merge-pro]

PR Code Suggestions ✨

No code suggestions found for the PR.

[nvborisenko]

.net part is correct, thanks.

[navin772]

Failing js fedcm test is expected due to https://issues.chromium.org/issues/425801332