Metrics log modification to handle external components.

components/metrics/metrics_service.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 service that collects information about the user
 // experience in order to help improve future versions of the app.
 
 #ifndef COMPONENTS_METRICS_METRICS_SERVICE_H_
 #define COMPONENTS_METRICS_METRICS_SERVICE_H_
 
@@ skipping 133 matching lines @@
   void EnableReporting();
   void DisableReporting();
 
   // Returns the client ID for this client, or the empty string if metrics
   // recording is not currently running.
   std::string GetClientId();
 
   // Returns the install date of the application, in seconds since the epoch.
   int64 GetInstallDate();
 
+  // Returns the date at which the current metrics client ID was created as
+  // an int64 containing seconds since the epoch8.

[Ilya Sherman, 2015/03/23 21:59:29]

nit: What's an "epoch8"?

[Olivier, 2015/03/24 16:28:02]

Done.

+  int64 GetMetricsReportingEnabledDate();
+
   // Returns the preferred entropy provider used to seed persistent activities
   // based on whether or not metrics reporting will be permitted on this client.
   //
   // If metrics reporting is enabled, this method returns an entropy provider
   // that has a high source of entropy, partially based on the client ID.
   // Otherwise, it returns an entropy provider that is based on a low entropy
   // source.
   scoped_ptr<const base::FieldTrial::EntropyProvider> CreateEntropyProvider();
 
   // At startup, prefs needs to be called with a list of all the pref names and
@@ skipping 72 matching lines @@
 
   // Check if this install was cloned or imaged from another machine. If a
   // clone is detected, reset the client id and low entropy source. This
   // should not be called more than once.
   void CheckForClonedInstall(
       scoped_refptr<base::SingleThreadTaskRunner> task_runner);
 
   // Clears the stability metrics that are saved in local state.
   void ClearSavedStabilityMetrics();
 
+  // Sets the connection type callback used to pass to the scheduler.
+  void SetConnectionTypeCallback(
+      base::Callback<void(bool*)> is_cellular_callback);

[Ilya Sherman, 2015/03/23 21:59:29]

Why are these three lines included in the diff?

[Olivier, 2015/03/24 16:28:02]

Done.

+
+  // Pushes a log that has been generated by an external component.
+  void PushExternalLog(const std::string& log);
+
  protected:
   // Exposed for testing.
   MetricsLogManager* log_manager() { return &log_manager_; }
 
  private:
   // The MetricsService has a lifecycle that is stored as a state.
   // See metrics_service.cc for description of this lifecycle.
   enum State {
     INITIALIZED,          // Constructor was called.
     INIT_TASK_SCHEDULED,  // Waiting for deferred init tasks to finish.
@@ skipping 232 matching lines @@
   // Weak pointers factory used for saving state. All weak pointers managed by
   // this factory are invalidated in ScheduleNextStateSave.
   base::WeakPtrFactory<MetricsService> state_saver_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MetricsService);
 };
 
 }  // namespace metrics
 
 #endif  // COMPONENTS_METRICS_METRICS_SERVICE_H_