Chromium Code Reviews| Index: components/safe_browsing/base_ping_manager.cc |
| diff --git a/components/safe_browsing/base_ping_manager.cc b/components/safe_browsing/base_ping_manager.cc |
| index 30fdf1c079bce65fe32a1bbccd98f8e7a48f927b..d397c2acd70956799f76c4d8538b082ed834c3f2 100644 |
| --- a/components/safe_browsing/base_ping_manager.cc |
| +++ b/components/safe_browsing/base_ping_manager.cc |
| @@ -201,7 +201,9 @@ GURL BasePingManager::SafeBrowsingHitUrl( |
| hit_report.threat_type == SB_THREAT_TYPE_URL_UNWANTED || |
| hit_report.threat_type == SB_THREAT_TYPE_BINARY_MALWARE_URL || |
| hit_report.threat_type == SB_THREAT_TYPE_CLIENT_SIDE_PHISHING_URL || |
| - hit_report.threat_type == SB_THREAT_TYPE_CLIENT_SIDE_MALWARE_URL); |
| + hit_report.threat_type == SB_THREAT_TYPE_CLIENT_SIDE_MALWARE_URL || |
| + hit_report.threat_type == |
| + SB_THREAT_TYPE_PASSWORD_PROTECTION_PHISHING_URL); |
| std::string url = ProtocolManagerHelper::ComposeUrl( |
| url_prefix_, "report", client_name_, version_, std::string(), |
| hit_report.extended_reporting_level); |
| @@ -226,6 +228,9 @@ GURL BasePingManager::SafeBrowsingHitUrl( |
| case SB_THREAT_TYPE_CLIENT_SIDE_MALWARE_URL: |
| threat_list = "malcsdhit"; |
| break; |
| + case SB_THREAT_TYPE_PASSWORD_PROTECTION_PHISHING_URL: |
| + threat_list = "phishpphit"; |
|
Nathan Parker
2017/05/30 18:26:57
Need to ensure backend knows of this string.
Jialiu Lin
2017/05/30 19:12:48
I've already confirmed with weiningy@, this will n
|
| + break; |
| default: |
| NOTREACHED(); |
| } |
| @@ -247,6 +252,9 @@ GURL BasePingManager::SafeBrowsingHitUrl( |
| case safe_browsing::ThreatSource::CLIENT_SIDE_DETECTION: |
| threat_source = "csd"; |
| break; |
| + case safe_browsing::ThreatSource::PASSWORD_PROTECTION_SERVICE: |
| + threat_source = "pps"; |
| + break; |
| case safe_browsing::ThreatSource::UNKNOWN: |
| NOTREACHED(); |
| } |