commit 0eb79fe5eaf01396b77a7ba15ada94ce99e2b58f
author Nancy Chen <nancychen@google.com> Wed Aug 19 19:33:38 2015 -0700
committer The Android Automerger <android-build@google.com> Thu Aug 20 12:56:00 2015 -0700
tree 76fe72e05ed1ec555c8c55ac20b171b2193c6bfc
parent ba6bdb5279ce83885f6f8e95dffe1dce3263cf65

Only mark as read voicemails with the "is_read" flag set.

The vvm sync service was querying for all voicemails with the "dirty"
bit set instead of voicemails that had actually been read and then
marking the voicemail as "read" locally. This resulted in voicemails
being marked as read that had not been listened to by the user.

Bug: 23351142
Change-Id: I24500fca0e835f643b3c635d498f51947359a481

src/com/android/phone/vvm/omtp/sync/VoicemailsQueryHelper.java

diff --git a/src/com/android/phone/vvm/omtp/sync/VoicemailsQueryHelper.java b/src/com/android/phone/vvm/omtp/sync/VoicemailsQueryHelper.java
index b86351c..8e2c76a 100644
--- a/src/com/android/phone/vvm/omtp/sync/VoicemailsQueryHelper.java
+++ b/src/com/android/phone/vvm/omtp/sync/VoicemailsQueryHelper.java
@@ -48,7 +48,8 @@
     public static final int IS_READ = 2;
     public static final int DELETED = 3;
 
-    final static String READ_SELECTION = Voicemails.DIRTY + "=1 AND " + Voicemails.DELETED + "!=1";
+    final static String READ_SELECTION = Voicemails.DIRTY + "=1 AND "
+                + Voicemails.DELETED + "!=1 AND " + Voicemails.IS_READ + "=1";
     final static String DELETED_SELECTION = Voicemails.DELETED + "=1";
 
     private Context mContext;