Implement nextHopProtocol in PerformanceResourceTiming and PerformanceNavigationTiming.

third_party/WebKit/Source/core/timing/PerformanceResourceTiming.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2012 Intel Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 44 matching lines @@
     double lastRedirectEndTime,
     bool allowTimingDetails,
     bool allowRedirectDetails)
     : PerformanceEntry(
           info.initialURL().getString(),
           "resource",
           monotonicTimeToDOMHighResTimeStamp(timeOrigin, startTime),
           monotonicTimeToDOMHighResTimeStamp(timeOrigin,
                                              info.loadFinishTime())),
       m_initiatorType(info.initiatorType()),
+      m_nextHopProtocol(info.finalResponse().alpnNegotiatedProtocol()),
+      m_connectionInfo(info.finalResponse().connectionInfo()),
       m_timeOrigin(timeOrigin),
       m_timing(info.finalResponse().resourceLoadTiming()),
       m_lastRedirectEndTime(lastRedirectEndTime),
       m_finishTime(info.loadFinishTime()),
       m_transferSize(info.transferSize()),
       m_encodedBodySize(info.finalResponse().encodedBodyLength()),
       m_decodedBodySize(info.finalResponse().decodedBodyLength()),
       m_didReuseConnection(info.finalResponse().connectionReused()),
       m_allowTimingDetails(allowTimingDetails),
       m_allowRedirectDetails(allowRedirectDetails) {}
 
 PerformanceResourceTiming::~PerformanceResourceTiming() {}
 
+AtomicString PerformanceResourceTiming::produceNextHopProtocol(
+    const AtomicString& alpnNegotiatedProtocol,
+    const AtomicString& connectionInfo) {
+  AtomicString returnedProtocol = alpnNegotiatedProtocol == "unknown"
+                                      ? connectionInfo
+                                      : alpnNegotiatedProtocol;
+  returnedProtocol =
+      returnedProtocol == "unknown" ? "http/1.1" : returnedProtocol;

[panicker, 2016/10/24 18:23:31]

I don't think we can assume http1 version here.

[sunjian, 2016/10/25 00:10:55]

Talked off line. Agreed to set it to be "http/1.1" when both
alpnNegotiatedProtocol and connectionInfo happen to be unknown.

+  returnedProtocol =
+      returnedProtocol == "quic/1+spdy/3" ? "http2/+quic/35" : returnedProtocol;

[panicker, 2016/10/24 18:23:31]

cannot assume quic version

[sunjian, 2016/10/25 00:10:55]

Done.

+  return returnedProtocol;
+}
+
 AtomicString PerformanceResourceTiming::initiatorType() const {
   return m_initiatorType;
 }
 
+AtomicString PerformanceResourceTiming::nextHopProtocol() const {
+  return PerformanceResourceTiming::produceNextHopProtocol(m_nextHopProtocol,
+                                                           m_connectionInfo);
+}
+
 double PerformanceResourceTiming::workerStart() const {
   if (!m_timing || m_timing->workerStart() == 0.0)
     return 0.0;
 
   return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin,
                                             m_timing->workerStart());
 }
 
 double PerformanceResourceTiming::workerReady() const {
   if (!m_timing || m_timing->workerReady() == 0.0)
@@ skipping 161 matching lines @@
   builder.addNumber("secureConnectionStart", secureConnectionStart());
   builder.addNumber("requestStart", requestStart());
   builder.addNumber("responseStart", responseStart());
   builder.addNumber("responseEnd", responseEnd());
   builder.addNumber("transferSize", transferSize());
   builder.addNumber("encodedBodySize", encodedBodySize());
   builder.addNumber("decodedBodySize", decodedBodySize());
 }
 
 }  // namespace blink