Add the spdy protocol to the information displayed about a spdy session in net-internals.

chrome/browser/resources/net_internals/spdy_view.js

 // Copyright (c) 2012 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.
 
 /**
  * This view displays a summary of the state of each SPDY sessions, and
  * has links to display them in the events tab.
  */
 var SpdyView = (function() {
   'use strict';
@@ skipping 121 matching lines @@
   };
 
   /**
    * Creates a table printer to print out the state of list of SPDY sessions.
    */
   function createSessionTablePrinter(spdySessions) {
     var tablePrinter = new TablePrinter();
     tablePrinter.addHeaderCell('Host');
     tablePrinter.addHeaderCell('Proxy');
     tablePrinter.addHeaderCell('ID');
+    tablePrinter.addHeaderCell('Protocol Negotiatied');
     tablePrinter.addHeaderCell('Active streams');
     tablePrinter.addHeaderCell('Unclaimed pushed');
     tablePrinter.addHeaderCell('Max');
     tablePrinter.addHeaderCell('Initiated');
     tablePrinter.addHeaderCell('Pushed');
     tablePrinter.addHeaderCell('Pushed and claimed');
     tablePrinter.addHeaderCell('Abandoned');
     tablePrinter.addHeaderCell('Received frames');
     tablePrinter.addHeaderCell('Secure');
     tablePrinter.addHeaderCell('Sent settings');
     tablePrinter.addHeaderCell('Received settings');
     tablePrinter.addHeaderCell('Error');
 
     for (var i = 0; i < spdySessions.length; i++) {
       var session = spdySessions[i];
       tablePrinter.addRow();
 
       tablePrinter.addCell(session.host_port_pair);
       tablePrinter.addCell(session.proxy);
 
       var idCell = tablePrinter.addCell(session.source_id);
       idCell.link = '#events&q=id:' + session.source_id;
 
+      tablePrinter.addCell(session.protocol_negotiated);

[eroman, 2012/03/07 19:27:32]

note: since this is a new field, when loading older logs it will end up printing
"undefined". That is probably OK, but you might want to enclose this in a
condition and add a comment to remove it in the future (say +2 chrome major
versions).

[Ryan Hamilton, 2012/03/07 19:46:22]

I don't mind undefined, but I'm happy to add a conditional.  Can you point me at
an example I could use?  I'm pretty new to this part of the code base.

       tablePrinter.addCell(session.active_streams);
       tablePrinter.addCell(session.unclaimed_pushed_streams);
       tablePrinter.addCell(session.max_concurrent_streams);
       tablePrinter.addCell(session.streams_initiated_count);
       tablePrinter.addCell(session.streams_pushed_count);
       tablePrinter.addCell(session.streams_pushed_and_claimed_count);
       tablePrinter.addCell(session.streams_abandoned_count);
       tablePrinter.addCell(session.frames_received);
       tablePrinter.addCell(session.is_secure);
       tablePrinter.addCell(session.sent_settings);
@@ skipping 18 matching lines @@
       tablePrinter.addRow();
 
       tablePrinter.addCell(entry.host_port_pair);
       tablePrinter.addCell(entry.alternate_protocol);
     }
     return tablePrinter;
   }
 
   return SpdyView;
 })();