From a98e9d5b0650b327e499771b9aa024a98deadef6 Mon Sep 17 00:00:00 2001 From: uabdullah Date: Tue, 19 Dec 2017 14:37:09 -0800 Subject: Print array map for debugging purposes. A small bug that only printed the last value of the array map and not all of it. Bug: 64882313 Test: N/A PiperOrigin-RevId: 179605815 Change-Id: I7230508f47e6603470d8067ef5233ae3c9af8911 --- .../dialer/voicemail/listui/NewVoicemailAdapter.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/java/com/android/dialer/voicemail/listui/NewVoicemailAdapter.java b/java/com/android/dialer/voicemail/listui/NewVoicemailAdapter.java index a4848c8f0..315bf1cf0 100644 --- a/java/com/android/dialer/voicemail/listui/NewVoicemailAdapter.java +++ b/java/com/android/dialer/voicemail/listui/NewVoicemailAdapter.java @@ -190,7 +190,7 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. printHashSet(); // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. - printHashMap(); + printArrayMap(); if (viewHolder instanceof NewVoicemailHeaderViewHolder) { LogUtil.i( @@ -241,7 +241,7 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter newVoicemailViewHolderArrayMap.remove(newVoicemailViewHolder.getViewHolderId()); printHashSet(); - printHashMap(); + printArrayMap(); } newVoicemailViewHolder.reset(); @@ -265,7 +265,7 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. printHashSet(); // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. - printHashMap(); + printArrayMap(); // If the viewholder is playing the voicemail, keep updating its media player view (seekbar, // duration etc.) @@ -298,12 +298,12 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. printHashSet(); // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. - printHashMap(); + printArrayMap(); } - private void printHashMap() { + private void printArrayMap() { LogUtil.i( - "NewVoicemailAdapter.printHashMap", + "NewVoicemailAdapter.printArrayMap", "hashMapSize: %d, currentlyExpandedViewHolderId:%d", newVoicemailViewHolderArrayMap.size(), currentlyExpandedViewHolderId); @@ -311,9 +311,9 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter if (!newVoicemailViewHolderArrayMap.isEmpty()) { String ids = ""; for (int id : newVoicemailViewHolderArrayMap.keySet()) { - ids = id + String.valueOf(id) + " "; + ids = ids + id + " "; } - LogUtil.i("NewVoicemailAdapter.printHashMap", "ids are " + ids); + LogUtil.i("NewVoicemailAdapter.printArrayMap", "ids are " + ids); } } @@ -801,7 +801,7 @@ final class NewVoicemailAdapter extends RecyclerView.Adapter currentlyExpandedViewHolderId); // TODO(uabdullah): a bug Remove logging, temporarily here for debugging. printHashSet(); - printHashMap(); + printArrayMap(); return null; } } -- cgit v1.2.3 From cc8942a206ad7d6c779547c0ec84c35988a6beaa Mon Sep 17 00:00:00 2001 From: uabdullah Date: Tue, 19 Dec 2017 15:53:56 -0800 Subject: Populate voicemails in simulator Adds the logic for only adding voicemails (no call log) in the simulator menu. Also adds the logic for adding a fast voicemail, which only adds 1 voicemail. Bug: 64882313 Test: N/A PiperOrigin-RevId: 179615214 Change-Id: I5a55df44d94b6123eaed6bc56439472b1ab81151 --- .../dialer/simulator/impl/SimulatorMainMenu.java | 30 ++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/java/com/android/dialer/simulator/impl/SimulatorMainMenu.java b/java/com/android/dialer/simulator/impl/SimulatorMainMenu.java index 6ec30f64b..c48e2836e 100644 --- a/java/com/android/dialer/simulator/impl/SimulatorMainMenu.java +++ b/java/com/android/dialer/simulator/impl/SimulatorMainMenu.java @@ -44,8 +44,12 @@ final class SimulatorMainMenu { "Notifications", SimulatorNotifications.getActionProvider(activity.getApplicationContext())) .addItem("Populate database", () -> populateDatabase(activity.getApplicationContext())) + .addItem("Populate voicemail", () -> populateVoicemail(activity.getApplicationContext())) .addItem( "Fast populate database", () -> fastPopulateDatabase(activity.getApplicationContext())) + .addItem( + "Fast populate voicemail database", + () -> populateVoicemailFast(activity.getApplicationContext())) .addItem("Clean database", () -> cleanDatabase(activity.getApplicationContext())) .addItem("clear preferred SIM", () -> clearPreferredSim(activity.getApplicationContext())) .addItem("Sync voicemail", () -> syncVoicemail(activity.getApplicationContext())) @@ -65,6 +69,32 @@ final class SimulatorMainMenu { .executeSerial(new PopulateDatabaseWorkerInput(context, false)); } + private static void populateVoicemail(@NonNull Context context) { + DialerExecutorComponent.get(context) + .dialerExecutorFactory() + .createNonUiTaskBuilder(new PopulateVoicemailWorker()) + .build() + .executeSerial(new PopulateDatabaseWorkerInput(context, false)); + } + + private static void populateVoicemailFast(@NonNull Context context) { + DialerExecutorComponent.get(context) + .dialerExecutorFactory() + .createNonUiTaskBuilder(new PopulateVoicemailWorker()) + .build() + .executeSerial(new PopulateDatabaseWorkerInput(context, true)); + } + + private static class PopulateVoicemailWorker + implements Worker { + @Nullable + @Override + public Void doInBackground(PopulateDatabaseWorkerInput input) { + VoicemailPopulator.populateVoicemail(input.context, input.fastMode); + return null; + } + } + private static void fastPopulateDatabase(@NonNull Context context) { DialerExecutorComponent.get(context) .dialerExecutorFactory() -- cgit v1.2.3