Implement nextHopProtocol in PerformanceResourceTiming and PerformanceNavigationTiming.

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

Index: third_party/WebKit/Source/core/timing/PerformanceResourceTimingTest.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceResourceTimingTest.cpp b/third_party/WebKit/Source/core/timing/PerformanceResourceTimingTest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..68c302a5ae953cd30d5be9389523fbcde44ec653
--- /dev/null
+++ b/third_party/WebKit/Source/core/timing/PerformanceResourceTimingTest.cpp
@@ -0,0 +1,57 @@
+// Copyright 2016 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.
+
+#include "core/timing/PerformanceResourceTiming.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace blink {
+
+class PerformanceResourceTimingTest : public ::testing::Test {
+ protected:
+  AtomicString produceNextHopProtocol(
+      const AtomicString& alpnNegotiatedProtocol,
+      const AtomicString& connectionInfo) {
+    return PerformanceResourceTiming::produceNextHopProtocol(
+        alpnNegotiatedProtocol, connectionInfo);
+  }
+};
+
+TEST_F(PerformanceResourceTimingTest,
+       TestFallbackToConnectionInfoWhenALPNUnknown) {
+  AtomicString connectionInfo = "http/1.1";
+  AtomicString alpnNegotiatedProtocol = "unknown";
+  EXPECT_EQ(produceNextHopProtocol(alpnNegotiatedProtocol, connectionInfo),
+            connectionInfo);
+}
+
+TEST_F(PerformanceResourceTimingTest,
+       TestFallbackToHTTPInfoWhenALPNAndConnectionInfoUnknown) {
+  AtomicString connectionInfo = "unknown";
+  AtomicString alpnNegotiatedProtocol = "unknown";
+  EXPECT_EQ(produceNextHopProtocol(alpnNegotiatedProtocol, connectionInfo),
+            "http/1.1");
+}
+
+TEST_F(PerformanceResourceTimingTest, TestReformatedWhenALPNIsQuicAndSpdy) {
+  AtomicString connectionInfo = "http/1.1";
+  AtomicString alpnNegotiatedProtocol = "quic/1+spdy/3";
+  EXPECT_EQ(produceNextHopProtocol(alpnNegotiatedProtocol, connectionInfo),
+            "http2/+quic/1");
+}
+
+TEST_F(PerformanceResourceTimingTest, TestReformatedWhenALPNIsQuic) {
+  AtomicString connectionInfo = "http/1.1";
+  AtomicString alpnNegotiatedProtocol = "quic/1+other/123";

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

what if this is just "quic/1"? Could we remove the "+something" assumption?

+  EXPECT_EQ(produceNextHopProtocol(alpnNegotiatedProtocol, connectionInfo),
+            "http2/+quic/1");
+}
+
+TEST_F(PerformanceResourceTimingTest, TestNoChangeWhenOtherwise) {
+  AtomicString connectionInfo = "http/1.1";
+  AtomicString alpnNegotiatedProtocol = "RandomProtocol";
+  EXPECT_EQ(produceNextHopProtocol(alpnNegotiatedProtocol, connectionInfo),
+            alpnNegotiatedProtocol);
+}
+}