Skip to content

/ servo

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

Jump to bottom

Re-enable text-transform tests #9821

Open
Ms2ger opened this issue Mar 1, 2016 · 2 comments
Open

Re-enable text-transform tests #9821

Ms2ger opened this issue Mar 1, 2016 · 2 comments
Labels
A-testing

Comments

@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Mar 1, 2016

No description provided.

Ms2ger added a commit to Ms2ger/servo that referenced this issue Mar 1, 2016
@Ms2ger
Disable text-transform tests for now (see servo#9821).
45ee8c7
Ms2ger added a commit to Ms2ger/servo that referenced this issue Mar 1, 2016
@Ms2ger
Disable text-transform tests for now (see servo#9821).
6d3849f
Ms2ger added a commit to Ms2ger/servo that referenced this issue Mar 1, 2016
@Ms2ger
Disable text-transform tests for now (see servo#9821).
52a15f7
@KiChjang KiChjang added the A-testing label Mar 3, 2016
@gsnedders
Copy link
Contributor

@gsnedders gsnedders commented Apr 19, 2016

@Ms2ger can we have some link to something somewhere which says why they were disabled?
@Ms2ger
Copy link
Contributor Author

@Ms2ger Ms2ger commented Apr 20, 2016

The problem is that they have a higher-than-average number of non-ASCII characters, and our incremental parsing code is broken in a way that can turn a non-ASCII character into two U+FFFDs if the network boundary falls in the middle of a character.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-testing
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@Ms2ger @gsnedders @KiChjang
You can’t perform that action at this time.