Bug 1247146 - Send firstrun panel telemetry only for unique panel visits. r=mfinkle draft
authorChenxia Liu <liuche@mozilla.com>
Fri, 12 Feb 2016 16:21:42 -0800
changeset 330786 ef029201d3e2ef06e033487549634832ccd5bc29
parent 330785 c30ba54620a4aa056de988a0ea864a7313330f1a
child 514252 0b879b3aadceac67f8b37c10589596b9dea2d96c
push id10841
push usercliu@mozilla.com
push dateSat, 13 Feb 2016 00:21:59 +0000
reviewersmfinkle
bugs1247146
milestone47.0a1
Bug 1247146 - Send firstrun panel telemetry only for unique panel visits. r=mfinkle MozReview-Commit-ID: CbLVODQhNL6
mobile/android/base/java/org/mozilla/gecko/firstrun/FirstrunPager.java
--- a/mobile/android/base/java/org/mozilla/gecko/firstrun/FirstrunPager.java
+++ b/mobile/android/base/java/org/mozilla/gecko/firstrun/FirstrunPager.java
@@ -91,25 +91,30 @@ public class FirstrunPager extends ViewP
             @Override
             public void finish() {
                 if (onFinishListener != null) {
                     onFinishListener.onFinish();
                 }
             }
         };
         addOnPageChangeListener(new OnPageChangeListener() {
+            private int maxPanel = 0;
+
             @Override
             public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
                 mDecor.onPageScrolled(position, positionOffset, positionOffsetPixels);
             }
 
             @Override
             public void onPageSelected(int i) {
                 mDecor.onPageSelected(i);
-                Telemetry.sendUIEvent(TelemetryContract.Event.SHOW, TelemetryContract.Method.PANEL, "onboarding." + i + "." + panels.get(i).getTag());
+                if (i > maxPanel) {
+                    Telemetry.sendUIEvent(TelemetryContract.Event.SHOW, TelemetryContract.Method.PANEL, "onboarding." + i + "." + panels.get(i).getTag());
+                    maxPanel = i;
+                }
             }
 
             @Override
             public void onPageScrollStateChanged(int i) {}
         });
 
         animateLoad();