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

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: nits 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..99835fa786c36e3ed543cf5b590adb43f6c756ee 100644
--- a/components/safe_browsing/password_protection/password_protection_request.cc
+++ b/components/safe_browsing/password_protection/password_protection_request.cc
@@ -148,6 +148,16 @@ 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:
« no previous file with comments | « components/safe_browsing/csd.proto ('k') | components/safe_browsing/password_protection/password_protection_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698