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

Unified Diff: chrome/test/data/webui/net_internals/log_view_painter.js

Issue 7553009: Add some browser tests for net-internals (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fix comments Created 9 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: chrome/test/data/webui/net_internals/log_view_painter.js
===================================================================
--- chrome/test/data/webui/net_internals/log_view_painter.js (revision 0)
+++ chrome/test/data/webui/net_internals/log_view_painter.js (revision 0)
@@ -0,0 +1,81 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * Tests the behavior of stripCookiesAndLoginInfo.
+ */
+netInternalsTest.test('NetInternalsLogViewPainterStripInfo', function() {
+ // Each entry in |expectations| is a list consisting of a header element
+ // before and after applying the filter. If the second entry is null, the
+ // element should be unmodified.
+ var expectations = [
+ ['set-cookie: blah', 'set-cookie: [value was stripped]'],
+ ['set-cookie2: blah', 'set-cookie2: [value was stripped]'],
+ ['cookie: blah', 'cookie: [value was stripped]'],
+ ['authorization: NTLM blah', 'authorization: NTLM [value was stripped]'],
+
+ ['proxy-authorization: Basic blah',
+ 'proxy-authorization: Basic [value was stripped]'],
+
+ ['WWW-Authenticate: Basic realm="Something, or another"', null],
+
+ ['WWW-Authenticate: Negotiate blah-token-blah',
+ 'WWW-Authenticate: Negotiate [value was stripped]'],
+
+ ['WWW-Authenticate: NTLM asdllk2j3l423lk4j23l4kj',
+ 'WWW-Authenticate: NTLM [value was stripped]'],
+
+ ['WWW-Authenticate: Kerberos , Negotiate asdfasdfasdfasfa', null],
+ ['WWW-Authenticate: Kerberos, Negotiate asdfasdfasdfasfa', null],
+ ['WWW-Authenticate: Digest , Negotiate asdfasdfasdfasfa', null],
+ ['WWW-Authenticate: Digest realm="Foo realm", Negotiate asdf', null],
+ ['WWW-Authenticate: Kerberos,Digest,Basic', null],
+ ['WWW-Authenticate: Digest realm="asdfasdf", nonce=5, qop="auth"', null],
+ ['WWW-Authenticate: Basic realm=foo,foo=bar , Digest ', null],
+ ['Proxy-Authenticate: Basic realm="Something, or another"', null],
+
+ ['Proxy-Authenticate: Negotiate blah-token-blah',
+ 'Proxy-Authenticate: Negotiate [value was stripped]'],
+
+ ['Proxy-Authenticate: NTLM asdllk2j3l423lk4j23l4kj',
+ 'Proxy-Authenticate: NTLM [value was stripped]'],
+
+ ['Proxy-Authenticate: Kerberos , Negotiate asdfasdfa', null],
+ ['Proxy-Authenticate: Kerberos, Negotiate asdfasdfa', null],
+ ['Proxy-Authenticate: Digest , Negotiate asdfasdfa', null],
+ ['Proxy-Authenticate: Digest realm="Foo realm", Negotiate asdfasdfa', null],
+ ['Proxy-Authenticate: Kerberos,Digest,Basic', null],
+ ['Proxy-Authenticate: Digest realm="asdfasdf", nonce=5, qop="auth"', null],
+ ['Proxy-Authenticate: Basic realm=foo,foo=bar , Digest ', null]
+ ];
+
+ for (var i = 0; i < expectations.length; ++i) {
+ var expectation = expectations[i];
+ // Position within params.headers where the authentication information goes.
+ for (var position = 0; position < 3; ++position) {
+ var entry = {
+ 'params': {
+ 'headers': [
+ 'Host: clients1.google.com',
+ 'Connection: keep-alive',
+ 'User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64)'],
+ 'line': 'GET / HTTP/1.1\r\n'},
+ 'phase': 0,
+ 'source': {'id': 329, 'type': 1},
+ 'time': '22468349',
+ 'type': 104};
+
+ entry.params.headers[position] = expectation[0];
+ var stripped = stripCookiesAndLoginInfo(entry);
+ expectTrue(stripped !== entry,
Sheridan Rawlins 2011/08/03 22:21:16 Does expectNotEquals do what you need for this? It
mmenke 2011/08/04 00:21:15 Oh, it does. I assumed it just used ==. Switched
+ expectation[0] + ': not duplicated.');
+ if (expectation[1] == null) {
+ expectEquals(stripped.params.headers[position], expectation[0]);
+ } else {
+ expectEquals(stripped.params.headers[position], expectation[1]);
+ }
+ }
+ }
+ netInternalsTest.testDone();
+});
Property changes on: chrome\test\data\webui\net_internals\log_view_painter.js
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698