Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(663)

Unified Diff: components/safe_browsing/password_protection/password_protection_request.cc

Issue 2949243004: Distinguish G Suite accounts from regular gmail/googlemail accounts (Closed)
Patch Set: address lpz's comments Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/safe_browsing/password_protection/password_protection_request.cc
diff --git a/components/safe_browsing/password_protection/password_protection_request.cc b/components/safe_browsing/password_protection/password_protection_request.cc
index 552def6263195cd341bf0648963be43e253df740..19602595d3458d93d3830fe99d91caa425a8ac2a 100644
--- a/components/safe_browsing/password_protection/password_protection_request.cc
+++ b/components/safe_browsing/password_protection/password_protection_request.cc
@@ -148,6 +148,15 @@ void PasswordProtectionRequest::FillRequestProto() {
request_proto_->mutable_password_reuse_event();
reuse_event->set_is_chrome_signin_password(
saved_domain_ == std::string(password_manager::kSyncPasswordDomain));
+ if (reuse_event->is_chrome_signin_password()) {
+ reuse_event->set_sync_account_type(
+ password_protection_service_->GetSyncAccountType());
+ UMA_HISTOGRAM_ENUMERATION(
+ "PasswordProtection.PasswordReuseSyncAccountType",
+ reuse_event->sync_account_type(),
+ LoginReputationClientRequest::PasswordReuseEvent::
+ SyncAccountType_MAX + 1);
+ }
break;
}
default:

Powered by Google App Engine
This is Rietveld 408576698