Adding UMA stats.

chrome/browser/ui/password_generation_status.cc

+// Copyright (c) 2011 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 "chrome/browser/ui/password_generation_status.h"
+
+#include "base/metrics/histogram.h"
+
+namespace password_generation {
+
+PasswordGenerationActions::PasswordGenerationActions()
+    : learn_more_visited(false),
+      password_accepted(false),
+      password_edited(false),
+      password_regenerated(false) {
+}
+
+PasswordGenerationActions::~PasswordGenerationActions() {
+}
+
+void UpdateUserActionsHistogram(PasswordGenerationActions actions) {
+  if (actions.password_accepted) {
+    if (actions.password_edited)
+      UMA_HISTOGRAM_ENUMERATION("PasswordGeneration.UserActions",
+                                ACCEPT_AFTER_EDITING, ACTION_ENUM_COUNT);

[Ilya Sherman, 2012/07/18 20:48:56]

nit: The "UMA_HISTOGRAM_ENUMERATION" block is repeated four times in this
method.  I would recommend having the if/else logic simply select which metric
to log, and moving the rest of this block out of the branched logic.

[zysxqn, 2012/07/18 22:46:17]

Done.

+    else
+      UMA_HISTOGRAM_ENUMERATION("PasswordGeneration.UserActions",
+                                ACCEPT_ORIGINAL_PASSWORD, ACTION_ENUM_COUNT);
+  } else if (actions.learn_more_visited) {
+    UMA_HISTOGRAM_ENUMERATION("PasswordGeneration.UserActions",
+                              LEARN_MORE, ACTION_ENUM_COUNT);
+  } else {
+    UMA_HISTOGRAM_ENUMERATION("PasswordGeneration.UserActions",
+                              IGNORE, ACTION_ENUM_COUNT);
+  }
+}
+
+void UpdateEventsHistogram(PasswordGenerationEvent event) {
+  UMA_HISTOGRAM_ENUMERATION("PasswordGeneration.Events",
+                            event, EVENT_ENUM_COUNT);
+}
+
+}  // namespace password_generation