metrics: Use absolute interval-based perf collection

chrome/browser/metrics/perf_provider_chromeos.h

 // Copyright (c) 2012 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 CHROME_BROWSER_METRICS_PERF_PROVIDER_CHROMEOS_H_
 #define CHROME_BROWSER_METRICS_PERF_PROVIDER_CHROMEOS_H_
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/threading/non_thread_safe.h"
 #include "base/time/time.h"
 #include "base/timer/timer.h"
-#include "components/metrics/proto/perf_data.pb.h"
+#include "components/metrics/proto/sampled_profile.pb.h"
 
 namespace metrics {
 
 class WindowedIncognitoObserver;
 
 // Provides access to ChromeOS perf data. perf aka "perf events" is a
 // performance profiling infrastructure built into the linux kernel. For more
 // information, see: https://perf.wiki.kernel.org/index.php/Main_Page.
 class PerfProvider : public base::NonThreadSafe {
  public:
   PerfProvider();
   ~PerfProvider();
 
   // Writes collected perf data protobufs to |perf_data|. Clears all the stored
   // perf data. Returns true if it wrote to |perf_data|.
-  bool GetPerfData(std::vector<PerfDataProto>* perf_data);
+  bool GetPerfData(std::vector<SampledProfile>* perf_data);
 
  private:
   // Starts an internal timer to start collecting perf data. The timer is set to
-  // trigger |interval| after this function call.
-  void ScheduleCollection(const base::TimeDelta& interval);
+  // trigger at a random point in either the current or the next profiling
+  // interval.
+  void ScheduleCollection();
 
   // Collects perf data if it has not been consumed by calling GetPerfData()
   // (see above).
   void CollectIfNecessary();
 
   // Collects perf data by calling CollectIfNecessary() and reschedules it to be
   // collected again.
   void CollectIfNecessaryAndReschedule();
 
   // Parses a protobuf from the |data| passed in only if the
   // |incognito_observer| indicates that no incognito window had been opened
   // during the profile collection period.
   void ParseProtoIfValid(
       scoped_ptr<WindowedIncognitoObserver> incognito_observer,
       const std::vector<uint8>& data);
 
-  // Vector of perf data protobufs.
-  std::vector<PerfDataProto> cached_perf_data_;
+  // Vector of SampledProfile protobufs containing perf profiles.
+  std::vector<SampledProfile> cached_perf_data_;
 
   // For scheduling collection of perf data.
   base::OneShotTimer<PerfProvider> timer_;
 
   // To pass around the "this" pointer across threads safely.
   base::WeakPtrFactory<PerfProvider> weak_factory_;
 
+  // Record of the last login time.
+  base::Time login_time_;

[Ilya Sherman, 2014/05/16 21:48:57]

Did you mean TimeTicks?

[Simon Que, 2014/05/17 03:25:07]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(PerfProvider);
 };
 
 }  // namespace metrics
 
 #endif  // CHROME_BROWSER_METRICS_PERF_PROVIDER_CHROMEOS_H_