Enable ServerTiming support

third_party/WebKit/Source/platform/network/HTTPParsers.cpp

 /*
  * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2011 Apple 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:
@@ skipping 853 matching lines @@
 
 bool parseContentRangeHeaderFor206(const String& contentRange,
                                    int64_t* firstBytePosition,
                                    int64_t* lastBytePosition,
                                    int64_t* instanceLength) {
   return net::HttpUtil::ParseContentRangeHeaderFor206(
       StringUTF8Adaptor(contentRange).asStringPiece(), firstBytePosition,
       lastBytePosition, instanceLength);
 }
 
+template <typename CharType>
+inline bool isNotServerTimingHeaderDelimiter(CharType c) {
+  return c != '=' && c != ';' && c != ',';
+}
+
+const LChar* parseServerTimingToken(const LChar* begin,
+                                    const LChar* end,
+                                    String& result) {
+  const LChar* position = begin;
+  skipWhile<LChar, isNotServerTimingHeaderDelimiter>(position, end);
+  result = String(begin, position - begin).stripWhiteSpace();
+  return position;
+}
+
+String checkDoubleQuotedString(const String& value) {
+  if (value.length() < 2 || value[0] != '"' ||
+      value[value.length() - 1] != '"') {
+    return value;
+  }
+
+  StringBuilder out;
+  unsigned
+      pos = 1,
+      len = value.length() -
+            1;  // begin after the opening DQUOTE, end before the closing DQUOTE

[Yoav Weiss, 2017/04/06 20:19:55]

Nit: formatting is weird. Move the comment to the line above. Also capitalize
and add a period.

+  while (pos < len) {
+    if (value[pos] == '\\')
+      pos++;  // skip right past the backslash

[Yoav Weiss, 2017/04/06 20:19:55]

Can you move the comments to above the `while` so that they describe what this
whole section is doing, rather than line by line?

+    if (pos < len)
+      out.append(value[pos++]);  // but include everything else
+  }
+
+  return out.toString();
+}
+
+std::unique_ptr<ServerTimingHeaderVector> parseServerTimingHeader(
+    const String& headerValue) {
+  std::unique_ptr<ServerTimingHeaderVector> headers =
+      WTF::makeUnique<ServerTimingHeaderVector>();
+
+  if (!headerValue.isNull()) {
+    DCHECK(headerValue.is8Bit());
+
+    const LChar* position = headerValue.characters8();
+    const LChar* end = position + headerValue.length();
+    while (position < end) {
+      String metric, value, description = "";
+      position = parseServerTimingToken(position, end, metric);
+      if (*position == '=') {
+        position = parseServerTimingToken(position + 1, end, value);
+      }
+      if (*position == ';') {
+        position = parseServerTimingToken(position + 1, end, description);
+      }
+      position++;
+
+      headers->push_back(WTF::makeUnique<ServerTimingHeader>(
+          metric, value.toDouble(), checkDoubleQuotedString(description)));
+    }
+  }
+  return headers;
+}
+
 }  // namespace blink