Updates to Server-Timing in accordance with with spec changes

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

 /*
  * Copyright (C) 2010 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 128 matching lines @@
   if (!RuntimeEnabledFeatures::PerformanceNavigationTiming2Enabled())
     return nullptr;
   if (!GetFrame())
     return nullptr;
   const DocumentLoader* document_loader =
       GetFrame()->Loader().GetDocumentLoader();
   DCHECK(document_loader);
   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() {
   DCHECK(GetFrame());
   if (!GetFrame()->GetDocument())
     return;
 
   if (HasObserverFor(PerformanceEntry::kLongTask)) {
     UseCounter::Count(&GetFrame()->LocalFrameRoot(),
                       WebFeature::kLongTaskObserver);
@@ skipping 97 matching lines @@
         culprit_dom_window->GetFrame()->DeprecatedLocalOwner();
     AddLongTaskTiming(
         start_time, end_time, attribution.first,
         GetFrameAttribute(frame_owner, HTMLNames::srcAttr, false),
         GetFrameAttribute(frame_owner, HTMLNames::idAttr, false),
         GetFrameAttribute(frame_owner, HTMLNames::nameAttr, true));
   }
 }
 
 }  // namespace blink