commit f26c7930deef471858d0f6c043e1ffc64ef1e3db
author Hyunyoung Song <hyunyoungs@google.com> Sat Jun 06 14:44:27 2020 -0700
committer Hyunyoung Song <hyunyoungs@google.com> Sun Jun 07 22:03:06 2020 -0700
tree 48b3de02d154e2914684bce066c6d3bf61059608
parent f6858198b9c4bcd54a9fc614badded2a32cdf37e

Fix double logging for wallpaper settings icon activation

Bug: 137957099

Change-Id: Id95fa026c1e6f177ffbb03e99cc18ef03e9a37b5

src/com/android/launcher3/BaseDraggingActivity.java

diff --git a/src/com/android/launcher3/BaseDraggingActivity.java b/src/com/android/launcher3/BaseDraggingActivity.java
index d1d5e26..268b910 100644
--- a/src/com/android/launcher3/BaseDraggingActivity.java
+++ b/src/com/android/launcher3/BaseDraggingActivity.java
@@ -187,8 +187,9 @@
                         sourceContainer);
             }
             getUserEventDispatcher().logAppLaunch(v, intent, user);
-            getStatsLogManager().log(LAUNCHER_APP_LAUNCH_TAP, item == null ? null
-                    : item.buildProto());
+            if (item != null) {
+                getStatsLogManager().log(LAUNCHER_APP_LAUNCH_TAP, item.buildProto());
+            }
             return true;
         } catch (NullPointerException|ActivityNotFoundException|SecurityException e) {
             Toast.makeText(this, R.string.activity_not_found, Toast.LENGTH_SHORT).show();