Add experimental feature info to certificate reports

components/network_time/network_time_tracker.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_NETWORK_TIME_NETWORK_TIME_TRACKER_H_
 #define COMPONENTS_NETWORK_TIME_NETWORK_TIME_TRACKER_H_
 
 #include <stdint.h>
 #include <memory>
 
@@ skipping 28 matching lines @@
 
 // Clock resolution is platform dependent.
 #if defined(OS_WIN)
 const int64_t kTicksResolutionMs = base::Time::kMinLowResolutionThresholdMs;
 #else
 const int64_t kTicksResolutionMs = 1;  // Assume 1ms for non-windows platforms.
 #endif
 
 // Variations Service feature that enables network time service querying.
 extern const base::Feature kNetworkTimeServiceQuerying;
+// Parameter name for the network time experiment that controls whether
+// time queries can be issued on-demand.

[jwd, 2016/11/03 15:15:34]

Seeing as I've asked you to reduce the magicness of what params get reported,
could you add a comment somewhere that makes sense (maybe here?) that new
parameters need to be added in cert_report_helper for them to be included in
cert reports?

+extern const char kVariationsServiceEnableFetchesOnDemand[];
 
 // A class that receives network time updates and can provide the network time
 // for a corresponding local time. This class is not thread safe.
 class NetworkTimeTracker : public net::URLFetcherDelegate {
  public:
   // Describes the result of a GetNetworkTime() call, describing whether
   // network time was available and if not, why not.
   enum NetworkTimeResult {
     // Network time is available.
     NETWORK_TIME_AVAILABLE,
@@ skipping 142 matching lines @@
   std::vector<base::Closure> fetch_completion_callbacks_;
 
   base::ThreadChecker thread_checker_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkTimeTracker);
 };
 
 }  // namespace network_time
 
 #endif  // COMPONENTS_NETWORK_TIME_NETWORK_TIME_TRACKER_H_