Bug 1247146 - Send firstrun panel telemetry only for unique panel visits. r=mfinkle
MozReview-Commit-ID: CbLVODQhNL6
--- 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();