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::initiatorType() const {
   return m_initiatorType;
 }
 
+AtomicString PerformanceResourceTiming::produceNextHopProtocol(
+    const AtomicString& alpnNegotiatedProtocol,
+    const AtomicString& connectionInfo) {
+  // Fallback to connectionInfo when alpnNegotiatedProtocol is unknowm
+  AtomicString returnedProtocol = alpnNegotiatedProtocol == "unknown"
+                                      ? connectionInfo
+                                      : alpnNegotiatedProtocol;
+  // Fallback to "http/1.1" when connectionInfo is also unknown

[panicker, 2016/10/27 22:05:22]

we should probably log an error here, but let's atleast mention in the comment
that this is an "error" situation

+  returnedProtocol =
+      returnedProtocol == "unknown" ? "http/1.1" : returnedProtocol;
+
+  // If quic is used, it must be prefixed with http/2
+  if (returnedProtocol.contains("quic/")) {
+    AtomicString quicPart(
+        returnedProtocol.impl()->substring(0, returnedProtocol.find('+')));
+    returnedProtocol = "http2/+" + quicPart;

[Yoav Weiss, 2016/10/28 07:00:34]

Where is that behavior defined?

[Yoav Weiss, 2016/10/28 07:00:34]

Can you add a comment indicating where the "+" syntax is defined?
I couldn't find that definition in https://tools.ietf.org/html/rfc7301 where the
ALPN ID registry is defined. (and lacking both "http2" and "quic"...)

[panicker, 2016/10/28 15:50:23]

My understanding is that IETF has not decided yet whether there will be an ALPN
string for QUIC at all, 
so this is "best effort" suggestion by rch@ and igrigorik@ (added to CL).
We should update this to match IETF after that's decided.

[Yoav Weiss, 2016/11/03 08:10:23]

It would still be good to have some definition of said syntax. Also, since we'd
be exposing that, there is a non-zero compat risk here. We should run it by the
API owners as part of the I2S.

[Ryan Hamilton, 2016/11/03 14:30:45]

bnc: Can you take a look at this in light of your connection_info changes?

+  }
+  return returnedProtocol;
+}
+
+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 143 matching lines @@
 unsigned long long PerformanceResourceTiming::decodedBodySize() const {
   if (!m_allowTimingDetails)
     return 0;
 
   return m_decodedBodySize;
 }
 
 void PerformanceResourceTiming::buildJSONValue(V8ObjectBuilder& builder) const {
   PerformanceEntry::buildJSONValue(builder);
   builder.addString("initiatorType", initiatorType());
+  builder.addString("nextHopProtocol", nextHopProtocol());
   builder.addNumber("workerStart", workerStart());
   builder.addNumber("redirectStart", redirectStart());
   builder.addNumber("redirectEnd", redirectEnd());
   builder.addNumber("fetchStart", fetchStart());
   builder.addNumber("domainLookupStart", domainLookupStart());
   builder.addNumber("domainLookupEnd", domainLookupEnd());
   builder.addNumber("connectStart", connectStart());
   builder.addNumber("connectEnd", connectEnd());
   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