Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update interfaces/cookie-store.idl #23957

Merged

Conversation

autofoolip
Copy link
Contributor

@autofoolip autofoolip commented Jun 3, 2020

This PR was automatically created by a bot.

Before merging, please check that any tests that depend on the updated IDL files still work.

If additional changes are needed, please manually create another PR based on this one.

See the README for how the IDL files in this directory are used.


Source: https://github.com/tidoust/reffy-reports/blob/3ef9f83/ed/idl/cookie-store.idl
Build: https://travis-ci.org/tidoust/reffy-reports/builds/696958118

@foolip
Copy link
Member

foolip commented Jun 3, 2020

This introduces a syntax error (missing semicolon) and will need a spec fix.

DOMTimeStamp? expires;
boolean secure;
CookieSameSite sameSite;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are failing because of the missing semicolon here.

@foolip
Copy link
Member

foolip commented Jun 10, 2020

I've sent WICG/cookie-store#145 to fix the problem.

inexorabletash pushed a commit to WICG/cookie-store that referenced this pull request Jun 10, 2020
Copy link
Member

@foolip foolip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@foolip foolip merged commit 555a1b3 into web-platform-tests:master Jun 10, 2020
@foolip foolip deleted the reffy-reports/cookie-store branch June 10, 2020 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants