Add support for collecting non-Chrome crash stats in Chrome OS

chrome/browser/metrics/metrics_log_unittest.cc

 // Copyright (c) 2006-2010 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 <string>
 
 #include "base/string_util.h"
 #include "base/time.h"
 #include "chrome/browser/metrics/metrics_log.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/test/testing_profile.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::TimeDelta;
 
 namespace {
   class MetricsLogTest : public testing::Test {
   };
 };
 
 
 // Since buildtime is highly variable, this function will scan an output log and
 // replace it with a consistent number.
 static void NormalizeBuildtime(std::string* xml_encoded) {
   std::string prefix = "buildtime=\"";
   const char postfix = '\"';
   size_t offset = xml_encoded->find(prefix);
-  ASSERT_GT(offset, 0u);
+  ASSERT_NE(std::string::npos, offset);
   offset += prefix.size();
   size_t postfix_position = xml_encoded->find(postfix, offset);
-  ASSERT_GT(postfix_position, offset);
+  ASSERT_NE(std::string::npos, postfix_position);
   for (size_t i = offset; i < postfix_position; ++i) {
     char digit = xml_encoded->at(i);
     ASSERT_GE(digit, '0');
     ASSERT_LE(digit, '9');
   }
 
   // Insert a single fake buildtime.
   xml_encoded->replace(offset, postfix_position - offset, "123246");
 }
 
@@ skipping 144 matching lines @@
 
   std::string encoded;
   // Leave room for the NUL terminator.
   ASSERT_TRUE(log.GetEncodedLog(WriteInto(&encoded, size + 1), size));
   TrimWhitespaceASCII(encoded, TRIM_ALL, &encoded);
   NormalizeBuildtime(&encoded);
   NormalizeBuildtime(&expected_output);
 
   ASSERT_EQ(expected_output, encoded);
 }
+
+TEST(MetricsLogTest, ChromeOSStabilityData) {
+  NoTimeMetricsLog log("bogus client ID", 0);
+  TestingProfile profile;
+  PrefService* pref = profile.GetPrefs();
+
+  pref->SetInteger(prefs::kStabilityChildProcessCrashCount, 11);
+  pref->SetInteger(prefs::kStabilityOtherUserCrashCount, 10);
+  pref->SetInteger(prefs::kStabilityKernelCrashCount, 11);

[jar (doing other things), 2011/01/14 02:31:47]

nit: four unique integers would probably be better (you repeated 11).

[kmixter1, 2011/01/15 22:21:07]

I didn't even notice that - 4 different was my intention. Done.

+  pref->SetInteger(prefs::kStabilitySystemUncleanShutdownCount, 12);
+  std::string expected_output = StringPrintf(
+      "<log clientid=\"bogus client ID\" buildtime=\"123456789\" "
+          "appversion=\"%s\">\n"
+      "<stability stuff>\n", MetricsLog::GetVersionString().c_str());
+  // Expect 3 warnings about not yet being able to send the
+  // Chrome OS stability stats.
+  log.WriteStabilityElement(profile.GetPrefs());
+  log.CloseLog();
+
+  int size = log.GetEncodedLogSize();
+  ASSERT_GT(size, 0);
+
+  EXPECT_EQ(0, pref->GetInteger(prefs::kStabilityChildProcessCrashCount));
+  EXPECT_EQ(0, pref->GetInteger(prefs::kStabilityOtherUserCrashCount));
+  EXPECT_EQ(0, pref->GetInteger(prefs::kStabilityKernelCrashCount));
+  EXPECT_EQ(0, pref->GetInteger(prefs::kStabilitySystemUncleanShutdownCount));
+
+  std::string encoded;
+  // Leave room for the NUL terminator.
+  ASSERT_TRUE(log.GetEncodedLog(WriteInto(&encoded, size + 1), size));

[jar (doing other things), 2011/01/14 02:31:47]

nit: I'd rather not see an ASSERT with side effects.  Better IMO is assert a
return value, but have the action outside the assert.  (you could argue these
are test-harness asserts... but I think readability is better when you can see
action code, and then look at tested assertions separately).

[kmixter1, 2011/01/15 22:21:07]

Done.

+
+  // Check that we can find childprocesscrashcount, but not
+  // any of the ChromeOS ones that we are not emitting until log
+  // servers can handle them.

[jar (doing other things), 2011/01/14 02:31:47]

:-) Nice!

[kmixter1, 2011/01/15 22:21:07]

Done.

+  EXPECT_NE(std::string::npos,
+            encoded.find(" childprocesscrashcount=\"11\""));
+  EXPECT_EQ(std::string::npos,
+            encoded.find(" otherusercrashcount="));
+  EXPECT_EQ(std::string::npos,
+            encoded.find(" kernelcrashcount="));
+  EXPECT_EQ(std::string::npos,
+            encoded.find(" systemuncleanshutdowns="));
+}
+
 #endif  // OS_CHROMEOS
 
 // Make sure our ID hashes are the same as what we see on the server side.
 TEST(MetricsLogTest, CreateHash) {
   static const struct {
     std::string input;
     std::string output;
   } cases[] = {
     {"Back", "0x0557fa923dcee4d0"},
     {"Forward", "0x67d2f6740a8eaebf"},
     {"NewTab", "0x290eb683f96572f1"},
   };
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); i++) {
     std::string hash_string = MetricsLog::CreateHash(cases[i].input);
 
     // Convert to hex string
     // We're only checking the first 8 bytes, because that's what
     // the metrics server uses.
     std::string hash_hex = "0x";
     for (size_t j = 0; j < 8; j++) {
       base::StringAppendF(&hash_hex, "%02x",
                           static_cast<uint8>(hash_string.data()[j]));
     }
     EXPECT_EQ(cases[i].output, hash_hex);
   }
 };