Add quotation marks around probability value in Alternative Services.

net/spdy/spdy_alt_svc_wire_format.cc

 // Copyright (c) 2015 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 "net/spdy/spdy_alt_svc_wire_format.h"
 
 #include <limits>
 #include <string>
 
 #include "base/logging.h"
@@ skipping 129 matching lines @@
       }
       if (parameter_name.compare("v") == 0) {
         if (!ParsePositiveInteger16(parameter_value_begin, c, &version)) {
           return false;
         }
       } else if (parameter_name.compare("ma") == 0) {
         if (!ParsePositiveInteger32(parameter_value_begin, c, &max_age)) {
           return false;
         }
       } else if (parameter_name.compare("p") == 0) {
-        if (!ParseProbability(parameter_value_begin, c, &probability)) {
+        // Probability value is enclosed in quotation marks.
+        if (*parameter_value_begin != '"' || *(c - 1) != '"') {
+          return false;
+        }
+        if (!ParseProbability(parameter_value_begin + 1, c - 1, &probability)) {
           return false;
         }
       }
     }
     altsvc_vector->push_back(AlternativeService(protocol_id, host, port,
                                                 version, max_age, probability));
     for (; c != value.end() && (*c == ' ' || *c == '\t' || *c == ','); ++c) {
     }
   }
   return true;
@@ skipping 52 matching lines @@
       value.push_back(c);
     }
     base::StringAppendF(&value, ":%d\"", altsvc.port);
     if (altsvc.version != 0) {
       base::StringAppendF(&value, "; v=%d", altsvc.version);
     }
     if (altsvc.max_age != 86400) {
       base::StringAppendF(&value, "; ma=%d", altsvc.max_age);
     }
     if (altsvc.probability != 1.0) {
-      base::StringAppendF(&value, "; p=%.2f", altsvc.probability);
+      base::StringAppendF(&value, "; p=\"%.2f\"", altsvc.probability);
     }
   }
   return value;
 }
 
 // static
 void SpdyAltSvcWireFormat::SkipWhiteSpace(StringPiece::const_iterator* c,
                                           StringPiece::const_iterator end) {
   for (; *c != end && (**c == ' ' || **c == '\t'); ++*c) {
   }
@@ skipping 109 matching lines @@
   ++c;
   double place_value = 0.1;
   for (; c != end && isdigit(*c); ++c) {
     *probability += place_value * (*c - '0');
     place_value *= 0.1;
   }
   return (c == end && *probability <= 1.0);
 }
 
 }  // namespace net