Chromium Code Reviews Chromium Code Reviews
Issue 2563083002:
[DevTools] show warning if 'Set-Cookie' header length is > 4096 symbols (Closed)
| Index: third_party/WebKit/LayoutTests/http/tests/inspector/network/warning-for-long-cookie.html |
| diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/network/warning-for-long-cookie.html b/third_party/WebKit/LayoutTests/http/tests/inspector/network/warning-for-long-cookie.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..88d7239ba7038f6e7890998d9d885704e29c80a1 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/http/tests/inspector/network/warning-for-long-cookie.html |
| @@ -0,0 +1,31 @@ |
| +<html> |
| +<head> |
| +<script src="../inspector-test.js"></script> |
| +<script src="../console-test.js"></script> |
| +<script src="../network-test.js"></script> |
| +<script> |
| + |
| +function fetchCookie(length) |
| +{ |
| + var xhr = new XMLHttpRequest(); |
| + xhr.open('GET', `http://localhost:8000/inspector/network/resources/set-long-cookie.php?length=${length}`, true); |
|
allada
2016/12/10 09:48:22
We are trying to stay away from using localhost in
kozy
2016/12/16 03:15:02
Done.
|
| + xhr.send(); |
| +} |
| + |
| +function test() |
| +{ |
| + InspectorTest.addConsoleViewSniffer(dumpMessages); |
|
allada
2016/12/10 09:48:22
I'm not against this, however the test will timeou
kozy
2016/12/16 03:15:02
Done.
|
| + InspectorTest.evaluateInPage("fetchCookie(4097)"); |
| + |
| + function dumpMessages() |
| + { |
| + InspectorTest.dumpConsoleMessages(); |
| + InspectorTest.completeTest(); |
| + } |
| +} |
| +</script> |
| +</head> |
| +<body onload="runTest()"> |
| +Checks that we show warning message for long cookie. |
| +</body> |
| +</html> |
