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

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

Issue 11583027: NetLog: Move the group_name to a CONNECT_JOB's first event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Add backward compatibility Created 8 years 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 | net/base/net_log_event_type_list.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/net_internals/source_entry.js
===================================================================
--- chrome/browser/resources/net_internals/source_entry.js (revision 174698)
+++ chrome/browser/resources/net_internals/source_entry.js (working copy)
@@ -202,10 +202,14 @@
return e;
}
if (this.entries_.length >= 2) {
- if (this.entries_[0].type == EventType.SOCKET_POOL_CONNECT_JOB ||
- this.entries_[1].type == EventType.UDP_CONNECT) {
+ // Needed for compatability with log dumps prior to M26.
+ // TODO(mmenke): Remove this.
+ if (this.entries_[0].type == EventType.SOCKET_POOL_CONNECT_JOB &&
+ this.entries_[0].params == undefined) {
return this.entries_[1];
}
+ if (this.entries_[1].type == EventType.UDP_CONNECT)
+ return this.entries_[1];
if (this.entries_[0].type == EventType.REQUEST_ALIVE &&
this.entries_[0].params == undefined) {
var start_index = 1;
« no previous file with comments | « no previous file | net/base/net_log_event_type_list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698