Use spec compliant content-type extraction in more places and enable a <stylesheet> quirk#28321
Conversation
|
Heads up! This PR modifies the following files:
|
|
|
I fixed warning issues |
|
@bors-servo r+ |
|
📌 Commit bd05f87 has been approved by |
Fix some Content-Type tests <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix #___ (GitHub issue number if applicable) <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because ___ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
|
💔 Test failed - status-taskcluster |
|
Looks like the results from https://community-tc.services.mozilla.com/tasks/V0zbGTN_THaRLbhb3i1onA/runs/0/logs/public/filtered-wpt-errorsummary.log need to be updated. The mac ones are duplicates or can be ignored. |
|
I rebased and updated the test logs. |
|
@bors-servo try |
Fix some Content-Type tests <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix #___ (GitHub issue number if applicable) <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because ___ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
|
💔 Test failed - status-taskcluster |
|
Seems some test expectations need to be updated 👀 Details |
|
@bors-servo try |
|
🔨 Triggering try run (#12771341700) for Linux WPT |
|
Test results for linux-wpt-layout-2020 from try job (#12771341700): Flaky unexpected result (17)
Stable unexpected results that are known to be intermittent (11)
Stable unexpected results (9)
|
|
|
|
I took a look at the failures in I looked through (one of) Gecko's MIME parsers, which is modeled off of the mimesniff implementation, and they have a funky addition where the presence of invalid tokens in the value will cause the value to be retained and treated as a quoted value instead when re-serializing. Overall, my instinct is to accept the current set of expected failures because the set of new passes is significantly larger. |
|
Okay. Sounds good. I'll update the results and land this then. |
0df5c28 to
40acd3a
Compare
40acd3a to
cbbb634
Compare
|
🔨 Triggering try run (#15108744906) for Linux (WPT) |
|
Test results for linux-wpt from try job (#15108744906): Flaky unexpected result (20)
Stable unexpected results that are known to be intermittent (15)
Stable unexpected results (2)
|
|
|
…a <stylesheet> quirk Signed-off-by: Martin Robinson <[email protected]>
cbbb634 to
2ab86fa
Compare
This changes includes two semi-related things:
types and charsets for
XMLHttpRequest.<stylesheet>parsing quirk involving mime types.Testing: There are tests for these changes.