[Password Generation] Breakout UMA stats for generated passwords

components/password_manager/core/browser/login_database.cc

 // 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.
 
 #include "components/password_manager/core/browser/login_database.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
@@ skipping 306 matching lines @@
       return false;
     }
   }
   return true;
 }
 
 void LoginDatabase::ReportMetrics(const std::string& sync_username,
                                   bool custom_passphrase_sync_enabled) {
   sql::Statement s(db_.GetCachedStatement(
       SQL_FROM_HERE,
-      "SELECT signon_realm, blacklisted_by_user, COUNT(username_value) "
-      "FROM logins GROUP BY signon_realm, blacklisted_by_user"));
+      "SELECT signon_realm, password_type, blacklisted_by_user,"
+      "COUNT(username_value) FROM logins GROUP BY "
+      "signon_realm, password_type, blacklisted_by_user"));
 
   if (!s.is_valid())
     return;
 
   std::string custom_passphrase = "WithoutCustomPassphrase";
   if (custom_passphrase_sync_enabled) {
     custom_passphrase = "WithCustomPassphrase";
   }
 
-  int total_accounts = 0;
+  int total_user_created_accounts = 0;
+  int total_generated_accounts = 0;
   int blacklisted_sites = 0;
   while (s.Step()) {
-    int blacklisted = s.ColumnInt(1);
-    int accounts_per_site = s.ColumnInt(2);
+    PasswordForm::Type password_type =
+        static_cast<PasswordForm::Type>(s.ColumnInt(1));
+    int blacklisted = s.ColumnInt(2);
+    int accounts_per_site = s.ColumnInt(3);
     if (blacklisted) {
       ++blacklisted_sites;
+    } else if (password_type == PasswordForm::TYPE_GENERATED) {
+      total_generated_accounts += accounts_per_site;
+      LogAccountStat(
+          base::StringPrintf("PasswordManager.AccountsPerSite.AutoGenerated.%s",
+                             custom_passphrase.c_str()),
+          accounts_per_site);
     } else {
       total_accounts += accounts_per_site;

[Ilya Sherman, 2014/11/18 22:37:10]

You might want to update the variable name here and below as well ;)

[Garrett Casto, 2014/11/18 22:54:49]

What I get for sending a response while compiling. Done.

-      LogAccountStat(base::StringPrintf("PasswordManager.AccountsPerSite.%s",
-                                        custom_passphrase.c_str()),
-                     accounts_per_site);
+      LogAccountStat(
+          base::StringPrintf("PasswordManager.AccountsPerSite.UserCreated.%s",
+                             custom_passphrase.c_str()),
+          accounts_per_site);
     }
   }
-  LogAccountStat(base::StringPrintf("PasswordManager.TotalAccounts.%s",
-                                    custom_passphrase.c_str()),
-                 total_accounts);
+  LogAccountStat(
+      base::StringPrintf("PasswordManager.TotalAccounts.UserCreated.%s",
+                         custom_passphrase.c_str()),
+      total_accounts);
+  LogAccountStat(
+      base::StringPrintf("PasswordManager.TotalAccounts.AutoGenerated.%s",
+                         custom_passphrase.c_str()),
+      total_generated_accounts);
   LogAccountStat(base::StringPrintf("PasswordManager.BlacklistedSites.%s",
                                     custom_passphrase.c_str()),
                  blacklisted_sites);
 
   sql::Statement usage_statement(db_.GetCachedStatement(
       SQL_FROM_HERE,
       "SELECT password_type, times_used FROM logins"));
 
   if (!usage_statement.is_valid())
     return;
 
   while (usage_statement.Step()) {
     PasswordForm::Type type = static_cast<PasswordForm::Type>(
         usage_statement.ColumnInt(0));
 
     if (type == PasswordForm::TYPE_GENERATED) {
-      LogTimesUsedStat(
-          base::StringPrintf("PasswordManager.TimesGeneratedPasswordUsed.%s",
-                             custom_passphrase.c_str()),
-          usage_statement.ColumnInt(1));
+      LogTimesUsedStat(base::StringPrintf(
+                           "PasswordManager.TimesPasswordUsed.AutoGenerated.%s",
+                           custom_passphrase.c_str()),
+                       usage_statement.ColumnInt(1));
     } else {
       LogTimesUsedStat(
-          base::StringPrintf("PasswordManager.TimesPasswordUsed.%s",
+          base::StringPrintf("PasswordManager.TimesPasswordUsed.UserCreated.%s",
                              custom_passphrase.c_str()),
           usage_statement.ColumnInt(1));
     }
   }
 
   bool syncing_account_saved = false;
   if (!sync_username.empty()) {
     sql::Statement sync_statement(db_.GetCachedStatement(
         SQL_FROM_HERE,
         "SELECT username_value FROM logins "
@@ skipping 426 matching lines @@
 
 bool LoginDatabase::DeleteAndRecreateDatabaseFile() {
   DCHECK(db_.is_open());
   meta_table_.Reset();
   db_.Close();
   sql::Connection::Delete(db_path_);
   return Init(db_path_);
 }
 
 }  // namespace password_manager