Bug 220096 – Remove non-inclusive language from tests in http/tests/xmlhttprequest

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Beth Dakin]

See results of 'report-non-inclusive-language --verbose' inside http/tests/xmlhttprequest below. I tried re-naming everything, but the tests broke (failure diff below), and I don't understand these tests enough to take the investigation much further. 

Script output:
access-control-basic-allow-list-request-headers.html: whitelist: 1
access-control-basic-allow-list-response-headers.html: whitelist: 1
resources/access-control-basic-whitelist-response-headers.cgi: whitelist: 2
Total: 4 (blacklist: 0, whitelist: 4, slave: 0)


Failure from simple re-name:
--- /Volumes/Data/worker/macOS-Mojave-Debug-WK1-Tests-EWS/build/layout-test-results/http/tests/xmlhttprequest/access-control-basic-allow-list-request-headers-expected.txt
+++ /Volumes/Data/worker/macOS-Mojave-Debug-WK1-Tests-EWS/build/layout-test-results/http/tests/xmlhttprequest/access-control-basic-allow-list-request-headers-actual.txt
@@ -1,6 +1,4 @@
-Accept: *
-Accept-Language: ru
-Content-Language: ru
-Content-Type: text/plain
+CONSOLE MESSAGE: Origin http://127.0.0.1:8000 is not allowed by Access-Control-Allow-Origin.
+CONSOLE MESSAGE: XMLHttpRequest cannot load http://localhost:8000/xmlhttprequest/resources/access-control-basic-allowlist-request-headers.cgi due to access control checks.
+Unexpected exception: NetworkError:  A network error occurred.
 
-


--- /Volumes/Data/worker/macOS-Mojave-Debug-WK1-Tests-EWS/build/layout-test-results/http/tests/xmlhttprequest/access-control-basic-allow-list-response-headers-expected.txt
+++ /Volumes/Data/worker/macOS-Mojave-Debug-WK1-Tests-EWS/build/layout-test-results/http/tests/xmlhttprequest/access-control-basic-allow-list-response-headers-actual.txt
@@ -1,9 +1,4 @@
-PASS: Response header cache-control allowed.
-PASS: Response header content-language allowed.
-PASS: Response header content-type allowed.
-PASS: Response header expires allowed.
-PASS: Response header last-modified allowed.
-PASS: Response header pragma allowed.
-PASS: Response header x-webkit denied.
-PASS: Non-allowlisted headers not passed to getAllResponseHeaders().
+CONSOLE MESSAGE: Origin http://127.0.0.1:8000 is not allowed by Access-Control-Allow-Origin.
+CONSOLE MESSAGE: XMLHttpRequest cannot load http://localhost:8000/xmlhttprequest/resources/access-control-basic-allowlist-response-headers.cgi due to access control checks.
+CONSOLE MESSAGE: NetworkError:  A network error occurred.

[Alexey Proskuryakov]

I don't remember anything anything tricky about these. Presumably you renamed the cgi files, not just changed the requests?

[Beth Dakin]

(In reply to Alexey Proskuryakov from comment #1)
> I don't remember anything anything tricky about these. Presumably you
> renamed the cgi files, not just changed the requests?

Yes, I did. I can try again and upload a patch here to try on the bots.

[Beth Dakin]

Created attachment 416689 [details]
Patch

Okay well now these tests do show as passing with these changes on my machine, so maybe I did something silly before.

[Beth Dakin]

Okay, now the tests are failing in the same way they were before. What do you think?

[Alexey Proskuryakov]

Comment on attachment 416689 [details]
Patch

I see the problem - cgi files need to have the Unix executable bit set, and they do not when testing the patch in EWS.

Applying the patch locally to an svn checkout with svn-apply, I see the executable bit preserved on rename, so it must be a webkit-patch limitation when dealing with git-svn.

Please land this manually from a pure svn checkout, or I can land it for you.

[Alexey Proskuryakov]

The way I found the problem was by downloading the full archive (via the "download layout test results" link in Buildbot), and checking Apache's error_log.txt file:

[Tue Dec 22 14:05:19.565341 2020] [cgi:error] [pid 1426] [client ::1:49939] AH01215: (13)Permission denied: exec of '/Volumes/Data/worker/macOS-Mojave-Release-WK1-Tests-EWS/build/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-allowlist-request-headers.cgi' failed: /Volumes/Data/worker/macOS-Mojave-Release-WK1-Tests-EWS/build/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-allowlist-request-headers.cgi, referer: http://127.0.0.1:8000/xmlhttprequest/access-control-basic-allow-list-request-headers.html

[Alexey Proskuryakov]

Committed http://trac.webkit.org/r271067

[Radar WebKit Bug Importer]

<rdar://problem/72603935>