Domain Reliability: Add sample_rate field to Beacon

components/domain_reliability/config.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_DOMAIN_RELIABILITY_CONFIG_H_
 #define COMPONENTS_DOMAIN_RELIABILITY_CONFIG_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 14 matching lines @@
  public:
   DomainReliabilityConfig();
   ~DomainReliabilityConfig();
 
   // Uses the JSONValueConverter to parse the JSON for a config into a struct.
   static scoped_ptr<const DomainReliabilityConfig> FromJSON(
       const base::StringPiece& json);
 
   bool IsValid() const;
 
-  bool DecideIfShouldReportRequest(bool success) const;
+  bool DecideIfShouldReportRequest(bool success, double* sample_rate_out)

[jkarlin, 2015/12/07 14:57:27]

I would prefer you added:

double GetSampleRate(bool success) const;

instead of adding the out parameter to DecideIfShouldReportRequest.

[Deprecated (see juliatuttle), 2015/12/07 20:17:57]

Done.

+      const;
 
   // Registers with the JSONValueConverter so it will know how to convert the
   // JSON for a config into the struct.
   static void RegisterJSONConverter(
       base::JSONValueConverter<DomainReliabilityConfig>* converter);
 
   GURL origin;
   bool include_subdomains;
   ScopedVector<GURL> collectors;
 
   double success_sample_rate;
   double failure_sample_rate;
   ScopedVector<std::string> path_prefixes;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DomainReliabilityConfig);
 };
 
 }  // namespace domain_reliability
 
 #endif  // COMPONENTS_DOMAIN_RELIABILITY_CONFIG_H_