Bug 221737

Summary: [LayoutTests] Convert http/tests/mime convert PHP to Python
Product: WebKit Reporter: Chris Gambrell <cgambrell>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, ews-watchlist, hi, jbedard, ryanhaddad, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=220749
Attachments:
Description Flags
Patch
none
Patch none

Chris Gambrell
Reported 2021-02-10 19:55:50 PST
Replacing PHP with equivalent Python CGI scripts
Attachments
Patch (14.96 KB, patch)
2021-02-12 08:02 PST, Chris Gambrell 		no flags
DetailsFormatted DiffDiff
Patch (14.98 KB, patch)
2021-02-12 15:20 PST, Chris Gambrell 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-02-10 19:56:57 PST
<rdar://problem/74219024>
Chris Gambrell
Comment 2 2021-02-12 08:02:41 PST
Created attachment 420123 [details] Patch
Jonathan Bedard
Comment 3 2021-02-12 09:02:25 PST
Comment on attachment 420123 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=420123&action=review > LayoutTests/http/tests/mime/quoted-charset.py:7 > +print('<p>Should say SUCCESS: SUóóåSS</p>') Should be a multi-line string. Also, you may need to dump this first sequence as bytes since this is testing a non-standard charset. > LayoutTests/http/tests/mime/resources/uppercase-mime-type.py:7 > +print('<html>') Should use a multi-line string here.
Chris Gambrell
Comment 4 2021-02-12 15:20:04 PST
Created attachment 420185 [details] Patch
Jonathan Bedard
Comment 5 2021-02-12 16:59:40 PST
Comment on attachment 420185 [details] Patch wk1 and wk2 tests are passing!
EWS
Comment 6 2021-02-12 17:48:28 PST
Committed r272827: <https://commits.webkit.org/r272827> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420185 [details].
Note You need to log in before you can comment on or make changes to this bug.