commit 70eadedd7764abef3d0cfdaba7de8eff7533b262
author Android Dialer <noreply@google.com> Mon Nov 13 10:52:54 2017 -0800
committer Zachary Heidepriem <zachh@google.com> Mon Nov 13 15:34:30 2017 -0800
tree 73b1533175f669cd55a44edde95f002987495193
parent 3066a0bfd12c218fa5f9527c5bea157e554d5f91

Add a new impression proto for same prefix call marked as spam.

Test: LoggerUtilsTest
PiperOrigin-RevId: 175555340
Change-Id: Ic38e4e60559d91d3471629a5ffff496081fb5fb9

java/com/android/dialer/contactphoto/ContactPhotoManagerImpl.java

diff --git a/java/com/android/dialer/contactphoto/ContactPhotoManagerImpl.java b/java/com/android/dialer/contactphoto/ContactPhotoManagerImpl.java
index a225072..5dbdf5e 100644
--- a/java/com/android/dialer/contactphoto/ContactPhotoManagerImpl.java
+++ b/java/com/android/dialer/contactphoto/ContactPhotoManagerImpl.java
@@ -931,7 +931,7 @@
 
     /**
      * Maximum number of photos to preload. If the cache size is 2Mb and the expected average size
-     * of a photo is 4kb, then this number should be 2Ma bugkb = 500.
+     * of a photo is 4kb, then this number should be 2Mb/4kb = 500.
      */
     private static final int MAX_PHOTOS_TO_PRELOAD = 100;
 

java/com/android/dialer/logging/dialer_impression.proto

diff --git a/java/com/android/dialer/logging/dialer_impression.proto b/java/com/android/dialer/logging/dialer_impression.proto
index cafcae3..3dd645e 100644
--- a/java/com/android/dialer/logging/dialer_impression.proto
+++ b/java/com/android/dialer/logging/dialer_impression.proto
@@ -121,11 +121,15 @@
 
     // From the service instead of an activity logs the number of times the
     // number was marked as Spam by the user (e.g from the feedback prompt)
-    SPAM_NOTIFICATION_SERVICE_ACTION_MARK_NUMBER_AS_SPAM = 1026;
+    SPAM_NOTIFICATION_SERVICE_ACTION_MARK_NUMBER_AS_SPAM = 1026
+
+        ;
 
     // From the service instead of an activity logs the number of times the
     // number was marked as Not Spam by the user (e.g from the feedback prompt)
-    SPAM_NOTIFICATION_SERVICE_ACTION_MARK_NUMBER_AS_NOT_SPAM = 1027;
+    SPAM_NOTIFICATION_SERVICE_ACTION_MARK_NUMBER_AS_NOT_SPAM = 1027
+
+        ;
 
     // User is in a active call i.e either incoming or outgoing
     // This is mainly so we can assign an impression event to a call event i.e
@@ -143,13 +147,17 @@
     INCOMING_NON_SPAM_CALL = 1030;
 
     // Spam notifications shown despite throttling
-    SPAM_NOTIFICATION_SHOWN_AFTER_THROTTLE = 1041;
+    SPAM_NOTIFICATION_SHOWN_AFTER_THROTTLE = 1041
+
+        ;
 
     // Spam notifications not shown due to throttling
     SPAM_NOTIFICATION_NOT_SHOWN_AFTER_THROTTLE = 1042;
 
     // Non spam notifications shown despite throttling
-    NON_SPAM_NOTIFICATION_SHOWN_AFTER_THROTTLE = 1043;
+    NON_SPAM_NOTIFICATION_SHOWN_AFTER_THROTTLE = 1043
+
+        ;
 
     // Non spam notifications not shown due to throttling
     NON_SPAM_NOTIFICATION_NOT_SHOWN_AFTER_THROTTLE = 1044;
@@ -581,5 +589,17 @@
     // Indicates that the number attempting to be assisted dialed had an
     // extension.
     ASSISTED_DIALING_CONSTRAINT_NUMBER_HAS_EXTENSION = 1289;
+
+    // User reports a same prefix call as spam from call history
+    REPORT_SAME_PREFIX_CALL_AS_SPAM_VIA_CALL_HISTORY =
+        1290
+
+        ;
+
+    // User reports a same prefix call as not spam from call history
+    REPORT_SAME_PREFIX_CALL_AS_NOT_SPAM_VIA_CALL_HISTORY =
+        1291
+
+        ;
   }
 }