Bug 1320730 - Ensure homepanel telemetry session is set when panel initially loaded r?sebastian
We currently only start a session when swiping between homepanels. We should also send
it when showing the homepanel normally.
MozReview-Commit-ID: 8Kk2RDCbDDc
--- a/mobile/android/base/java/org/mozilla/gecko/home/HomePager.java
+++ b/mobile/android/base/java/org/mozilla/gecko/home/HomePager.java
@@ -456,16 +456,22 @@ public class HomePager extends ViewPager
// UI thread to give the pager time to commit the new page selection
// internally and load the right initial panel.
ThreadUtils.getUiHandler().post(new Runnable() {
@Override
public void run() {
adapter.setCanLoadHint(true);
}
});
+
+ // We need to fire telemetry on the initial load: we will subsequently send telemetry whenever
+ // the user switches between homepanels, but the first load doesn't involve any switching hence
+ // we need to send telemetry now:
+ final String panelType = ((HomeAdapter) getAdapter()).getPanelIdAtPosition(mDefaultPageIndex);
+ startNewPanelTelemetrySession(panelType);
}
@Override
public void setOnPanelChangeListener(OnPanelChangeListener listener) {
mPanelChangedListener = listener;
}
@Override