Use different source types for each ConnectJob subclass.

chrome/browser/resources/net_internals/source_entry.js

Index: chrome/browser/resources/net_internals/source_entry.js
diff --git a/chrome/browser/resources/net_internals/source_entry.js b/chrome/browser/resources/net_internals/source_entry.js
index e1e217b188f65a17de063dc3ad70594cd697936f..bca0e04b1103a339d665f16731377ca34995fb28 100644
--- a/chrome/browser/resources/net_internals/source_entry.js
+++ b/chrome/browser/resources/net_internals/source_entry.js
@@ -84,7 +84,13 @@ var SourceEntry = (function() {
         case EventSourceType.HTTP_STREAM_JOB:
           this.description_ = e.params.url;
           break;
+        // TODO(davidben): Remove CONNECT_JOB after M57 is released.
         case EventSourceType.CONNECT_JOB:

[mmenke, 2016/11/30 15:09:24]

This works, even if EventSourceType.CONNECT_JOB is undefined (And even when
everything but EventSourceType.CONNECT_JOB is undefined)?

[davidben, 2016/11/30 17:37:49]

Apparently. EventSourceType.SOCKET_STREAM is also undefined. JavaScript just
spits out the value |undefined| and I guess e.source.type is not that value. For
better or worse, we seem to rely on this everywhere. :-/

(I would probably suggest that, if we cared, we post-process EventSourceType and
friends to set all unknown keys to -1 or something.)

[mmenke, 2016/11/30 20:51:59]

But....then we'd have multiple "case -1:", instead of multiple "case unknowns:".
 Just to verify, you have checked this works with old logs?  Multiple cases with
the same value (even if it's unknown) seems even weirder than one case with an
unknown.

[davidben, 2016/11/30 20:58:38]

Yup, works fine. JavaScript is fine with duplicate switch-cases and picks the
first match.

+        case EventSourceType.TRANSPORT_CONNECT_JOB:
+        case EventSourceType.SSL_CONNECT_JOB:
+        case EventSourceType.SOCKS_CONNECT_JOB:
+        case EventSourceType.HTTP_PROXY_CONNECT_JOB:
+        case EventSourceType.WEB_SOCKET_TRANSPORT_CONNECT_JOB:
           this.description_ = e.params.group_name;
           break;
         case EventSourceType.HOST_RESOLVER_IMPL_JOB: