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

Needs toJSON defined #31

Closed
toddreifsteck opened this issue Jun 29, 2017 · 4 comments
Closed

Needs toJSON defined #31

toddreifsteck opened this issue Jun 29, 2017 · 4 comments
Milestone

Comments

@toddreifsteck
Copy link
Member

Today, Chrome does not return the collection when toJSON is called. This is correct per the spec, but I believe is unexpected. I believe the toJSON should include the collection.

@igrigorik igrigorik added this to the Level 1 milestone Jul 21, 2017
@igrigorik
Copy link
Member

Re, spec. I think we just need to add https://github.com/w3c/resource-timing/pull/114/files?

@toddreifsteck
Copy link
Member Author

Maybe.. but today, Chrome has a bug where the inner collection isn't serialized although everything else is. I'm not sure if that is due to incorrect WebIDL or just a bug in Chrome.

@igrigorik
Copy link
Member

@spanicker do we have a crbug for this? If not, we should probably file one.

@tdresser
Copy link
Contributor

tdresser commented Jun 12, 2019

I think this is a dupe of #37.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants