Updates to Server-Timing in accordance with with spec changes

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

Index: third_party/WebKit/Source/core/timing/Performance.cpp
diff --git a/third_party/WebKit/Source/core/timing/Performance.cpp b/third_party/WebKit/Source/core/timing/Performance.cpp
index a60c520b94f94b3757d865b804b8e0504dd77656..b817c6f2779ea378fb69940ad4adda4df1ca1270 100644
--- a/third_party/WebKit/Source/core/timing/Performance.cpp
+++ b/third_party/WebKit/Source/core/timing/Performance.cpp
@@ -146,7 +146,9 @@ PerformanceNavigationTiming* Performance::CreateNavigationTimingInstance() {
   ResourceTimingInfo* info = document_loader->GetNavigationTimingInfo();
   if (!info)
     return nullptr;
-  return new PerformanceNavigationTiming(GetFrame(), info, TimeOrigin());
+  PerformanceServerTimingVector serverTiming = ParseServerTiming(*info, true);

[Yoav Weiss, 2017/06/29 06:57:33]

It's hard to know what "true" here means from the caller site, which is why we
usually prefer enums for such flags. Can you replace the boolean with an enum?

+  return new PerformanceNavigationTiming(GetFrame(), info, TimeOrigin(),
+                                         serverTiming);
 }
 
 void Performance::UpdateLongTaskInstrumentation() {