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

Side by Side Diff: chrome/browser/resources/net_internals/source_entry.js

Issue 2621983004: Improve HttpStreamFactory NetLog events (Closed)
Patch Set: Address Bence comments and change back to auto Created 3 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | net/http/http_stream_factory_impl.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 var SourceEntry = (function() { 5 var SourceEntry = (function() {
6 'use strict'; 6 'use strict';
7 7
8 /** 8 /**
9 * A SourceEntry gathers all log entries with the same source. 9 * A SourceEntry gathers all log entries with the same source.
10 * 10 *
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 74
75 if (e.params == undefined) { 75 if (e.params == undefined) {
76 return; 76 return;
77 } 77 }
78 78
79 switch (e.source.type) { 79 switch (e.source.type) {
80 case EventSourceType.URL_REQUEST: 80 case EventSourceType.URL_REQUEST:
81 // TODO(ricea): Remove SOCKET_STREAM after M41 is released. 81 // TODO(ricea): Remove SOCKET_STREAM after M41 is released.
82 case EventSourceType.SOCKET_STREAM: 82 case EventSourceType.SOCKET_STREAM:
83 case EventSourceType.HTTP_STREAM_JOB: 83 case EventSourceType.HTTP_STREAM_JOB:
84 case EventSourceType.HTTP_STREAM_JOB_CONTROLLER:
84 this.description_ = e.params.url; 85 this.description_ = e.params.url;
85 break; 86 break;
86 // TODO(davidben): Remove CONNECT_JOB after M57 is released. 87 // TODO(davidben): Remove CONNECT_JOB after M57 is released.
87 case EventSourceType.CONNECT_JOB: 88 case EventSourceType.CONNECT_JOB:
88 case EventSourceType.TRANSPORT_CONNECT_JOB: 89 case EventSourceType.TRANSPORT_CONNECT_JOB:
89 case EventSourceType.SSL_CONNECT_JOB: 90 case EventSourceType.SSL_CONNECT_JOB:
90 case EventSourceType.SOCKS_CONNECT_JOB: 91 case EventSourceType.SOCKS_CONNECT_JOB:
91 case EventSourceType.HTTP_PROXY_CONNECT_JOB: 92 case EventSourceType.HTTP_PROXY_CONNECT_JOB:
92 case EventSourceType.WEB_SOCKET_TRANSPORT_CONNECT_JOB: 93 case EventSourceType.WEB_SOCKET_TRANSPORT_CONNECT_JOB:
93 this.description_ = e.params.group_name; 94 this.description_ = e.params.group_name;
(...skipping 252 matching lines...) Expand 10 before | Expand all | Expand 10 after
346 this.entries_, SourceTracker.getInstance().getPrivacyStripping(), 347 this.entries_, SourceTracker.getInstance().getPrivacyStripping(),
347 SourceTracker.getInstance().getUseRelativeTimes() ? 348 SourceTracker.getInstance().getUseRelativeTimes() ?
348 timeutil.getBaseTime() : 349 timeutil.getBaseTime() :
349 0, 350 0,
350 Constants.clientInfo.numericDate); 351 Constants.clientInfo.numericDate);
351 }, 352 },
352 }; 353 };
353 354
354 return SourceEntry; 355 return SourceEntry;
355 })(); 356 })();
OLDNEW
« no previous file with comments | « no previous file | net/http/http_stream_factory_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698