Gather stability prefs into managing objects.

components/metrics/metrics_log.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.
 
 // This file defines a set of user experience metrics data recorded by
 // the MetricsService.  This is the unit of data that is sent to the server.
 
 #ifndef COMPONENTS_METRICS_METRICS_LOG_H_
 #define COMPONENTS_METRICS_METRICS_LOG_H_
 
 #include <stdint.h>
 
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/time/time.h"
 #include "components/metrics/metrics_service_client.h"
 #include "components/metrics/proto/chrome_user_metrics_extension.pb.h"
 
-class PrefRegistrySimple;
 class PrefService;
 
 namespace base {
 class HistogramSamples;
 }
 
 namespace variations {
 struct ActiveGroupId;
 }
 
@@ skipping 17 matching lines @@
   // Note: |this| instance does not take ownership of the |client|, but rather
   // stores a weak pointer to it. The caller should ensure that the |client| is
   // valid for the lifetime of this class.
   MetricsLog(const std::string& client_id,
              int session_id,
              LogType log_type,
              MetricsServiceClient* client,
              PrefService* local_state);
   virtual ~MetricsLog();
 
-  // Registers local state prefs used by this class.
-  static void RegisterPrefs(PrefRegistrySimple* registry);
-
   // Computes the MD5 hash of the given string, and returns the first 8 bytes of
   // the hash.
   static uint64_t Hash(const std::string& value);
 
   // Get the GMT buildtime for the current binary, expressed in seconds since
   // January 1, 1970 GMT.
   // The value is used to identify when a new build is run, so that previous
   // reliability stats, from other builds, can be abandoned.
   static int64_t GetBuildTime();
 
@@ skipping 93 matching lines @@
   bool HasEnvironment() const;
 
   // Write the default state of the enable metrics checkbox.
   void WriteMetricsEnableDefault(EnableMetricsDefault metrics_default,
                                  SystemProfileProto* system_profile);
 
   // Returns true if the stability metrics have already been filled in by a
   // call to RecordStabilityMetrics().
   bool HasStabilityMetrics() const;
 
-  // Within the stability group, write required attributes.
-  void WriteRequiredStabilityAttributes(PrefService* pref);
-
   // Within the stability group, write attributes that need to be updated asap
   // and can't be delayed until the user decides to restart chromium.
   // Delaying these stats would bias metrics away from happy long lived
   // chromium processes (ones that don't crash, and keep on running).
-  void WriteRealtimeStabilityAttributes(PrefService* pref,
-                                        base::TimeDelta incremental_uptime,
+  void WriteRealtimeStabilityAttributes(base::TimeDelta incremental_uptime,
                                         base::TimeDelta uptime);
 
   // closed_ is true when record has been packed up for sending, and should
   // no longer be written to.  It is only used for sanity checking.
   bool closed_;
 
   // The type of the log, i.e. initial or ongoing.
   const LogType log_type_;
 
   // Stores the protocol buffer representation for this log.
   ChromeUserMetricsExtension uma_proto_;
 
   // Used to interact with the embedder. Weak pointer; must outlive |this|
   // instance.
   MetricsServiceClient* const client_;
 
   // The time when the current log was created.
   const base::TimeTicks creation_time_;
 
   PrefService* local_state_;
 
   DISALLOW_COPY_AND_ASSIGN(MetricsLog);
 };
 
 }  // namespace metrics
 
 #endif  // COMPONENTS_METRICS_METRICS_LOG_H_