Skip to content

Conversation

@annevk
Copy link
Member

@annevk annevk commented Aug 17, 2017

Currently only Edge passes this and that is my fault for not paying enough attention.

@ghost
Copy link

ghost commented Aug 17, 2017

Build PASSED

Started: 2017-08-17 17:04:43
Finished: 2017-08-17 17:21:38

View more information about this build on:

Copy link
Contributor

@ayg ayg left a comment

Choose a reason for hiding this comment

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

I didn't look at the spec, but this change seems pretty obviously correct.

@ayg ayg merged commit 9730ed3 into master Aug 20, 2017
@annevk annevk deleted the annevk/xhr-data-url-head branch August 21, 2017 10:06
@wisniewskit
Copy link
Contributor

Ah, sorry that this slipped through the cracks for a few days. Yes, I agree that this looks correct. Thanks!

rachelandrew pushed a commit to rachelandrew/web-platform-tests that referenced this pull request Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants