[Password Manager] Add UMA stats to track interaction with synced accounts.

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"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/pickle.h"
 #include "base/strings/string_util.h"
 #include "base/time/time.h"
 #include "components/autofill/core/common/password_form.h"
+#include "google_apis/gaia/gaia_auth_util.h"
+#include "google_apis/gaia/gaia_urls.h"
 #include "sql/connection.h"
 #include "sql/statement.h"
 #include "sql/transaction.h"
 
 using autofill::PasswordForm;
 
 namespace password_manager {
 
 static const int kCurrentVersionNumber = 6;
 static const int kCompatibleVersionNumber = 1;
@@ skipping 219 matching lines @@
     }
     if (!db_.Execute("CREATE INDEX logins_signon ON "
                      "logins (signon_realm)")) {
       NOTREACHED();
       return false;
     }
   }
   return true;
 }
 
-void LoginDatabase::ReportMetrics() {
+void LoginDatabase::ReportMetrics(const std::string& sync_username,
+                                  enum password_manager::SyncState sync_state) {

[Ilya Sherman, 2014/06/27 19:37:51]

nit: I don't think you need the "enum" keyword here.

[Garrett Casto, 2014/06/27 21:07:34]

Done.

   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"));
 
   if (!s.is_valid())
     return;
 
   int total_accounts = 0;
   int blacklisted_sites = 0;
@@ skipping 27 matching lines @@
     if (type == PasswordForm::TYPE_GENERATED) {
       UMA_HISTOGRAM_CUSTOM_COUNTS(
           "PasswordManager.TimesGeneratedPasswordUsed",
           usage_statement.ColumnInt(1), 0, 100, 10);
     } else {
       UMA_HISTOGRAM_CUSTOM_COUNTS(
           "PasswordManager.TimesPasswordUsed",
           usage_statement.ColumnInt(1), 0, 100, 10);
     }
   }
+
+  bool syncing_account_saved = false;
+  int times_syncing_account_used = 0;
+  if (!sync_username.empty()) {
+    sql::Statement sync_statement(db_.GetCachedStatement(
+        SQL_FROM_HERE,
+        "SELECT username_value, times_used FROM logins "
+        "WHERE signon_realm == ?"));
+    sync_statement.BindString(
+        0, GaiaUrls::GetInstance()->gaia_url().GetOrigin().spec());
+
+    if (!sync_statement.is_valid())
+      return;
+
+    while (sync_statement.Step()) {
+      std::string username = sync_statement.ColumnString(0);
+      // It's possible to have multiple stored credentials that all reference
+      // the same account (different spelling of the same name or different
+      // URL).
+      if (gaia::AreEmailsSame(sync_username, username)) {
+        syncing_account_saved = true;
+        times_syncing_account_used += sync_statement.ColumnInt(1);
+      }
+    }
+  }
+  UMA_HISTOGRAM_ENUMERATION("PasswordManager.SyncingAccountState",
+                            2 * sync_state + syncing_account_saved,
+                            2 * password_manager::NUM_SYNC_STATES);
+
+  if (syncing_account_saved) {
+    switch (sync_state) {
+      case password_manager::PASSWORD_SYNC_ENABLED:
+        UMA_HISTOGRAM_COUNTS_100(
+            "PasswordManager.PasswordSyncAccountTimesUsed",
+            times_syncing_account_used);
+        break;
+      case password_manager::PASSWORD_SYNC_DISABLED:
+        UMA_HISTOGRAM_COUNTS_100(
+            "PasswordManager.SyncAccountTimesUsed",
+            times_syncing_account_used);
+        break;
+      default:

[Ilya Sherman, 2014/06/27 19:37:51]

Optional nit: I generally prefer to enumerate all the cases explicitly, rather
than using a default stmt, so that if the enum is ever expanded, the compiler
will remind me to think about how the switch stmt should be updated.

[Garrett Casto, 2014/06/27 21:07:34]

Done.

+        NOTREACHED();
+    }
+  }
 }
 
 PasswordStoreChangeList LoginDatabase::AddLogin(const PasswordForm& form) {
   PasswordStoreChangeList list;
   std::string encrypted_password;
   if (EncryptedString(form.password_value, &encrypted_password) !=
           ENCRYPTION_RESULT_SUCCESS)
     return list;
 
   // You *must* change LoginTableColumns if this query changes.
@@ skipping 374 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