corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1376024 - Whitelist more known Safe Browsing providers for telemetry purposes. r=dimi

Browse Source 
MozReview-Commit-ID: 6Dd1eDSJlzT
This commit is contained in:
Francois Marier
2017-06-23 17:00:45 -07:00
parent d877b17591
commit f37e04a87d
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
toolkit/components/url-classifier/nsUrlClassifierUtils.cpp
View File

@@ -292,11 +292,14 @@ nsUrlClassifierUtils::GetTelemetryProvider(const nsACString& aTableName,
nsACString& aProvider) nsACString& aProvider)
{ {
GetProvider(aTableName, aProvider); GetProvider(aTableName, aProvider);
// Filter out build-in providers: mozilla, google, google4 // Whitelist known providers to avoid reporting on private ones.
// Empty provider is filtered as "other" // An empty provider is treated as "other"
if (!NS_LITERAL_CSTRING("mozilla").Equals(aProvider) && if (!NS_LITERAL_CSTRING("mozilla").Equals(aProvider) &&
!NS_LITERAL_CSTRING("google").Equals(aProvider) && !NS_LITERAL_CSTRING("google").Equals(aProvider) &&
!NS_LITERAL_CSTRING("google4").Equals(aProvider) && !NS_LITERAL_CSTRING("google4").Equals(aProvider) &&
!NS_LITERAL_CSTRING("baidu").Equals(aProvider) &&
!NS_LITERAL_CSTRING("mozcn").Equals(aProvider) &&
!NS_LITERAL_CSTRING("yandex").Equals(aProvider) &&
!NS_LITERAL_CSTRING(TESTING_TABLE_PROVIDER_NAME).Equals(aProvider)) { !NS_LITERAL_CSTRING(TESTING_TABLE_PROVIDER_NAME).Equals(aProvider)) {
aProvider.Assign(NS_LITERAL_CSTRING("other")); aProvider.Assign(NS_LITERAL_CSTRING("other"));
} }