Bug 1320730 - Ensure homepanel telemetry session is set when panel initially loaded r?sebastian draft
authorAndrzej Hunt <ahunt@mozilla.com>
Tue, 07 Feb 2017 13:05:34 -0800
changeset 480127 d87f0d6500ebc916190fcb2972d5894aaf4cf759
parent 479958 e677ba018b22558fef1d07b74d416fd3a28a5dc3
child 480128 b4ac0fdf9cc26b4b590c5a147486c8f4bd8fe14b
push id44455
push userahunt@mozilla.com
push dateTue, 07 Feb 2017 21:11:22 +0000
reviewerssebastian
bugs1320730
milestone54.0a1
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
mobile/android/base/java/org/mozilla/gecko/home/HomePager.java
--- 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