Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(179)

Unified Diff: third_party/WebKit/LayoutTests/inspector/curl-command.html

Issue 2182213006: [Devtools] Fix multi-line curl copy response when multi-part-form (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed windows curl with multi-line Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/inspector/curl-command.html
diff --git a/third_party/WebKit/LayoutTests/inspector/curl-command.html b/third_party/WebKit/LayoutTests/inspector/curl-command.html
index 294e08754f02e1ff735ed5f5b0781e125bf7705f..0e9a42343b5ff2ce509012a84d062f305de9c776 100644
--- a/third_party/WebKit/LayoutTests/inspector/curl-command.html
+++ b/third_party/WebKit/LayoutTests/inspector/curl-command.html
@@ -27,15 +27,18 @@ var test = function()
function dumpRequest(headers, data, opt_url)
{
- InspectorTest.addResult(logView._generateCurlCommand(newRequest(headers, data, opt_url), WebInspector.isWin() ? "win" : "unix"));
+ InspectorTest.addResult("Windows: " + logView._generateCurlCommand(newRequest(headers, data, opt_url), "win"));
+ InspectorTest.addResult("Unix: " + logView._generateCurlCommand(newRequest(headers, data, opt_url), "unix"));
}
dumpRequest({})
dumpRequest({}, "123");
dumpRequest({"Content-Type": "application/x-www-form-urlencoded"}, "1&b");
dumpRequest({"Content-Type": "application/json"}, "{\"a\":1}");
- dumpRequest({"Content-Type": "application/binary"}, "1234\r\n\x30\x30\2\3\4\5\'\"!"); // expected content length: 15
- dumpRequest({"Content-Type": "application/binary"}, "1234\r\n\1\x30\x30\2\3\4\5\'\"!"); // expected content length: 16
+ dumpRequest({"Content-Type": "application/binary"}, "1234\r\n\x30\x30\2\3\4\5\'\"!");
+ dumpRequest({"Content-Type": "application/binary"}, "1234\r\n\1\x30\x30\2\3\4\5\'\"!");
+ dumpRequest({"Content-Type": "application/binary"}, "%OS%\r\n%%OS%%\r\n\"\\\"'$&!"); // Ensure %OS% for windows is not env evaled
+ dumpRequest({"Content-Type": "application/binary"}, "!@#$%^&*()_+~`1234567890-=[]{};':\",./\r<>?\r\nqwer\nt\n\nyuiopasdfghjklmnbvcxzQWERTYUIOPLKJHGFDSAZXCVBNM");
dumpRequest({"Content-Type": "application/binary"}, "\x7F\x80\x90\xFF\u0009\u0700");
dumpRequest({}, "", "http://labs.ft.com/?querystring=[]{}"); // Character range symbols must be escaped (http://crbug.com/265449).
dumpRequest({"Content-Type": "application/binary"}, "%PATH%$PATH");

Powered by Google App Engine
This is Rietveld 408576698