Bug 1358333 - Fix APPLICATION_REPUTATION_ALLOWLIST_MATCH and APPLICATION_REPUTATION_BLOCKLIST_MATCH telemetry r=francois
authorThomas Nguyen <tnguyen@mozilla.com>
Fri, 21 Apr 2017 11:18:34 +0800
changeset 566625 fee151a11c16a1a6995770aa4c3400b31d2faaf0
parent 566624 fe2fff73428147c36b75189670a36cee6ff07b39
child 566626 fd25c2c275fceb51347b7a108c6ce17214dbf70c
push id55296
push userzbraniecki@mozilla.com
push dateFri, 21 Apr 2017 23:53:52 +0000
reviewersfrancois
bugs1358333
milestone55.0a1
Bug 1358333 - Fix APPLICATION_REPUTATION_ALLOWLIST_MATCH and APPLICATION_REPUTATION_BLOCKLIST_MATCH telemetry r=francois MozReview-Commit-ID: DbI0uNdEwZ8
toolkit/components/downloads/ApplicationReputation.cpp
--- a/toolkit/components/downloads/ApplicationReputation.cpp
+++ b/toolkit/components/downloads/ApplicationReputation.cpp
@@ -153,18 +153,18 @@ LookupTablesInPrefs(const nsACString& ta
     found = true;
 
     if (!shouldRecordTelemetry) {
       return found;
     }
 
     // We are checking if the table found is V2 or V4 to record telemetry
     // Both V2 and V4 begin with "goog" but V4 ends with "-proto"
-    if (StringBeginsWith(prefToken, NS_LITERAL_CSTRING("goog"))) {
-      if (StringEndsWith(prefToken, NS_LITERAL_CSTRING("-proto"))) {
+    if (StringBeginsWith(table, NS_LITERAL_CSTRING("goog"))) {
+      if (StringEndsWith(table, NS_LITERAL_CSTRING("-proto"))) {
         telemetryInfo |= TelemetryMatchInfo::eV4Match;
       } else {
         telemetryInfo |= TelemetryMatchInfo::eV2Match;
       }
     }
   }
 
   // Record telemetry for matching allow list and block list