NEW 186538
Add ByteString as a string that adheres to WebIDL ByteString constraints 
https://bugs.webkit.org/show_bug.cgi?id=186538
Summary Add ByteString as a string that adheres to WebIDL ByteString constraints
youenn fablet
Reported 2018-06-11 13:33:37 PDT
This will allow to tighten/optimize the code a bit.
Attachments
Patch (35.76 KB, patch)
2018-06-11 13:57 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Fixing compilation (35.77 KB, patch)
2018-06-11 16:47 PDT, youenn fablet 		ews-watchlist: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from ews112 for mac-sierra (3.44 MB, application/zip)
2018-06-11 19:15 PDT, EWS Watchlist 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2018-06-11 13:57:37 PDT
Created attachment 342462 [details] Patch
Chris Dumez
Comment 2 2018-06-11 14:07:37 PDT
Comment on attachment 342462 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=342462&action=review > Source/WebCore/ChangeLog:8 > + Modify IDL conversion to instantiate WTF::ByteString instead of WTF::String. Is it really worth adding yet one more string type? This is not required for correct behavior since the JS bindings already dealt with ByteString correctly. I am personally not convinced this is an improvement and find it surprising with add Web IDL concepts in WTF.
youenn fablet
Comment 3 2018-06-11 16:47:16 PDT
Created attachment 342486 [details] Fixing compilation
EWS Watchlist
Comment 4 2018-06-11 19:15:18 PDT
Comment on attachment 342486 [details] Fixing compilation Attachment 342486 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/8140083 New failing tests: imported/w3c/web-platform-tests/XMLHttpRequest/overridemimetype-blob.html
EWS Watchlist
Comment 5 2018-06-11 19:15:20 PDT
Created attachment 342502 [details] Archive of layout-test-results from ews112 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews112 Port: mac-sierra Platform: Mac OS X 10.12.6
Ahmad Saleem
Comment 6 2022-09-03 06:38:51 PDT
Is this needed anymore? Unfortunately, it didn't had any test case in the attached patch so I couldn't check. :-( Just trying to close old bugs etc., so don't mind my bumps. Thanks!
Note You need to log in before you can comment on or make changes to this bug.