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

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

Issue 2896053002: Don't send saved password domain (Closed)
Patch Set: add is_chrome_signin_password Created 3 years, 7 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 e4c7183089a8d73383533279190072f57d714598..e14b57a31495f7aa00108f479e7ab4814f6658b5 100644
--- a/components/safe_browsing/password_protection/password_protection_request.cc
+++ b/components/safe_browsing/password_protection/password_protection_request.cc
@@ -123,13 +123,7 @@ void PasswordProtectionRequest::FillRequestProto() {
break;
}
case LoginReputationClientRequest::PASSWORD_REUSE_EVENT: {
- if (password_protection_service_->IsExtendedReporting() &&
- !password_protection_service_->IsIncognito()) {
- LoginReputationClientRequest::PasswordReuseEvent* password_reuse =
- request_proto_->mutable_password_reuse_event();
- password_reuse->add_password_reused_original_origins(saved_domain_);
- }
- // TODO(jialiul): Fill more password_reuse information.
+ // TODO(jialiul): Fill more password reuse related information when ready.
break;
}
default:

Powered by Google App Engine
This is Rietveld 408576698