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

Specify the realm for the SSE MessageEvent #1988

Merged
merged 1 commit into from
Nov 1, 2016
Merged

Conversation

domenic
Copy link
Member

@domenic domenic commented Oct 27, 2016

Closes #1959. All other MessageEvents are created as part of "fire an
event", which is covered by whatwg/dom#352.

Tests are part of web-platform-tests/wpt#4067.

Closes #1959. All other MessageEvents are created as part of "fire an
event", which is covered by whatwg/dom#352.
@zcorpan
Copy link
Member

zcorpan commented Oct 28, 2016

This changes EventSource but web-platform-tests/wpt#4067 tests WebSocket. Wrong link or am I missing something? :-)

@domenic
Copy link
Member Author

domenic commented Oct 28, 2016

Good call, I mashed up web-platform-tests/wpt#4025 and web-platform-tests/wpt#4067 in my mind, but it turns out I haven't written a test for this exact scenario. Let me do that.

@domenic domenic added the needs tests Moving the issue forward requires someone to write tests label Oct 28, 2016
domenic added a commit to web-platform-tests/wpt that referenced this pull request Oct 31, 2016
@domenic
Copy link
Member Author

domenic commented Oct 31, 2016

Tests added at web-platform-tests/wpt#4134

@domenic domenic removed the needs tests Moving the issue forward requires someone to write tests label Oct 31, 2016
zcorpan pushed a commit to web-platform-tests/wpt that referenced this pull request Nov 1, 2016
@zcorpan zcorpan merged commit 91d19d6 into master Nov 1, 2016
@zcorpan zcorpan deleted the messagevent-realm branch November 1, 2016 10:24
tomoyukilabs pushed a commit to tomoyukilabs/web-platform-tests that referenced this pull request Nov 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants