Send UMA stability stats in a separate UMA log on startup.

chrome/browser/metrics/metrics_service_unittest.cc

 // 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.
 
 #include <ctype.h>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/scoped_testing_local_state.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "components/variations/metrics_util.h"
 #include "content/public/common/process_type.h"
+#include "content/public/common/webplugininfo.h"
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
+class TestMetricsService : public MetricsService {
+ public:
+  TestMetricsService() {}
+  virtual ~TestMetricsService() {}
+
+  const MetricsLogManager& log_manager() const {
+    return log_manager_;
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(TestMetricsService);
+};
+
 class MetricsServiceTest : public testing::Test {
  public:
   MetricsServiceTest()
       : ui_thread_(content::BrowserThread::UI, &message_loop_),
         testing_local_state_(TestingBrowserProcess::GetGlobal()) {
   }
 
+  virtual ~MetricsServiceTest() {
+    MetricsService::SetExecutionPhase(MetricsService::UNINITIALIZED_PHASE);
+  }
+
   PrefService* GetLocalState() {
     return testing_local_state_.Get();
   }
 
   // Returns true if there is a synthetic trial in the given vector that matches
   // the given trial name and trial group; returns false otherwise.
   bool HasSyntheticTrial(
       const std::vector<chrome_variations::ActiveGroupId>& synthetic_trials,
       const std::string& trial_name,
       const std::string& trial_group) {
@@ skipping 93 matching lines @@
         switches::kResetVariationState);
 
     MetricsService::SetExecutionPhase(MetricsService::UNINITIALIZED_PHASE);
     MetricsService service;
     service.GetLowEntropySource();
 
     EXPECT_TRUE(GetLocalState()->GetString(kCachePrefName).empty());
   }
 }
 
+TEST_F(MetricsServiceTest, InitialStabilityLogAfterCleanShutDown) {
+  base::FieldTrialList field_trial_list(NULL);
+  base::FieldTrialList::CreateFieldTrial("UMAStability", "SeparateLog");
+
+  GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, true);
+
+  TestMetricsService service;
+  service.InitializeMetricsRecordingState(MetricsService::REPORTING_ENABLED);
+  // No initial stability log should be generated.
+  EXPECT_FALSE(service.log_manager().has_unsent_logs());
+  EXPECT_FALSE(service.log_manager().has_staged_log());
+}
+
+TEST_F(MetricsServiceTest, InitialStabilityLogAfterCrash) {
+  base::FieldTrialList field_trial_list(NULL);
+  base::FieldTrialList::CreateFieldTrial("UMAStability", "SeparateLog");
+
+  GetLocalState()->ClearPref(prefs::kStabilityExitedCleanly);
+
+  // Set up prefs to simulate restarting after a crash.
+
+  // Save an existing system profile to prefs, to correspond to what would be
+  // saved from a previous session.
+  MetricsLog log("client", 1);
+  log.RecordEnvironment(std::vector<content::WebPluginInfo>(),
+                        GoogleUpdateMetrics(),
+                        std::vector<chrome_variations::ActiveGroupId>());
+
+  // Record stability build time and version from previous session, so that
+  // stability metrics (including exited cleanly flag) won't be cleared.
+  GetLocalState()->SetInt64(prefs::kStabilityStatsBuildTime,
+                            MetricsLog::GetBuildTime());
+  GetLocalState()->SetString(prefs::kStabilityStatsVersion,
+                             MetricsLog::GetVersionString());
+
+  GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, false);
+
+  TestMetricsService service;
+  service.InitializeMetricsRecordingState(MetricsService::REPORTING_ENABLED);
+  // The initial stability log should be generated and persisted in unsent logs.
+  EXPECT_TRUE(service.log_manager().has_unsent_logs());

[Ilya Sherman, 2013/12/12 09:03:38]

Can you check the contents of the unsent logs as well?

[Alexei Svitkine (slow), 2013/12/12 16:09:27]

Done.

+  EXPECT_FALSE(service.log_manager().has_staged_log());
+}
+
 // Crashes on at least Mac and Linux.  http://crbug.com/320433
 TEST_F(MetricsServiceTest, DISABLED_RegisterSyntheticTrial) {
   MetricsService service;
 
   // Add two synthetic trials and confirm that they show up in the list.
   SyntheticTrialGroup trial1(metrics::HashName("TestTrial1"),
                              metrics::HashName("Group1"),
                              base::TimeTicks::Now());
   service.RegisterSyntheticFieldTrial(trial1);
 
@@ skipping 33 matching lines @@
   service.log_manager_.FinishCurrentLog();
   service.log_manager_.BeginLoggingWithLog(new MetricsLog("clientID", 1),
                                            MetricsLog::ONGOING_LOG);
   service.GetCurrentSyntheticFieldTrials(&synthetic_trials);
   EXPECT_EQ(3U, synthetic_trials.size());
   EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial1", "Group2"));
   EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial2", "Group2"));
   EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial3", "Group3"));
   service.log_manager_.FinishCurrentLog();
 }