Remove obsolete comment about cookie secure context

[mjhenkes]

See

cypress/packages/server/lib/browsers/cdp_automation.ts

Lines 266 to 273 in 885541e

	if (this.experimentalSessionAndOrigin) {
	// localhost is considered a secure context (even when http:)
	// and it's required for cross origin support when visiting a secondary
	// origin so that all its cookies are sent. This may be a
	// breaking change, so put it behind the flag for now. Need to
	// investigate further when we remove the experimental flag.
	return !(cookie.secure && url.startsWith('http:') && !isLocalhost)
	}

Is this a breaking change?

[AtofStryker]

we might also need to add this in the simulated cookie handling in the server side jar here to be consistent

[mjhenkes]

We don't think this is breaking so remove the condition and the comment.

[chrisbreiding]

This was completed in #24340

[cypress-bot]

Released in 12.0.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v12.0.0, please open a new issue.