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

Unified Diff: chrome/browser/resources/net_internals/log_view_painter.js

Issue 2853383003: Prune some unused log entries. (Closed)
Patch Set: . Created 3 years, 8 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
« no previous file with comments | « no previous file | chrome/test/data/webui/net_internals/log_view_painter.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/net_internals/log_view_painter.js
diff --git a/chrome/browser/resources/net_internals/log_view_painter.js b/chrome/browser/resources/net_internals/log_view_painter.js
index ce2ef342f588ff8d1529bd5aa89c2adfcfa88917..54cc6d8ec4ab21c4dcfe4a4389d2e7063644fb44 100644
--- a/chrome/browser/resources/net_internals/log_view_painter.js
+++ b/chrome/browser/resources/net_internals/log_view_painter.js
@@ -291,9 +291,6 @@ var stripPrivacyInfo;
case EventType.CERT_CT_COMPLIANCE_CHECKED:
case EventType.EV_CERT_CT_COMPLIANCE_CHECKED:
return writeParamsForCheckedCertificates;
-
- case EventType.SSL_VERSION_FALLBACK:
- return writeParamsForSSLVersionFallback;
}
return null;
}
@@ -406,30 +403,6 @@ var stripPrivacyInfo;
}
/**
- * Converts an SSL version number to a textual representation.
- * For instance, SSLVersionNumberToName(0x0301) returns 'TLS 1.0'.
- */
- function SSLVersionNumberToName(version) {
- if ((version & 0xFFFF) != version) {
- // If the version number is more than 2 bytes long something is wrong.
- // Print it as hex.
- return 'SSL 0x' + version.toString(16);
- }
-
- // See if it is a known TLS name.
- var kTLSNames = {0x0301: 'TLS 1.0', 0x0302: 'TLS 1.1', 0x0303: 'TLS 1.2'};
- var name = kTLSNames[version];
- if (name)
- return name;
-
- // Otherwise label it as an SSL version.
- var major = (version & 0xFF00) >> 8;
- var minor = version & 0x00FF;
-
- return 'SSL ' + major + '.' + minor;
- }
-
- /**
* TODO(eroman): get rid of this, as it is only used by 1 callsite.
*
* Indent |lines| by |start|.
@@ -657,26 +630,6 @@ var stripPrivacyInfo;
entry.params.certificate, out, consumedParams, 'certificate');
}
- /**
- * Outputs the SSL version fallback parameters of |entry| to |out|.
- */
- function writeParamsForSSLVersionFallback(entry, out, consumedParams) {
- var params = entry.params;
-
- if (typeof params.version_before != 'number' ||
- typeof params.version_after != 'number') {
- // Unrecognized params.
- return;
- }
-
- var line = SSLVersionNumberToName(params.version_before) + ' ==> ' +
- SSLVersionNumberToName(params.version_after);
- out.writeArrowIndentedLines([line]);
-
- consumedParams.version_before = true;
- consumedParams.version_after = true;
- }
-
function writeParamsForProxyConfigChanged(entry, out, consumedParams) {
var params = entry.params;
« no previous file with comments | « no previous file | chrome/test/data/webui/net_internals/log_view_painter.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698