Bug 1213416 - Add a specific Extra to LOAD_URL telemetry in the Awesomescreen. r=liuche, a=sylvestre
authorMark Finkle <mfinkle@mozilla.com>
Fri, 09 Oct 2015 19:54:29 -0400
changeset 296385 36a1cee1aca8ee25483caab8681f104e93d7c15a
parent 296384 977151d42625244a85fa1fb8399914c61464f20d
child 296386 bcb8973e8e551aa3f969f035b08366c69212ed38
push id5245
push userraliiev@mozilla.com
push dateThu, 29 Oct 2015 11:30:51 +0000
treeherdermozilla-beta@dac831dc1bd0 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersliuche, sylvestre
bugs1213416
milestone43.0a2
Bug 1213416 - Add a specific Extra to LOAD_URL telemetry in the Awesomescreen. r=liuche, a=sylvestre
mobile/android/base/home/BrowserSearch.java
--- a/mobile/android/base/home/BrowserSearch.java
+++ b/mobile/android/base/home/BrowserSearch.java
@@ -294,20 +294,17 @@ public class BrowserSearch extends HomeF
                     return;
                 }
 
                 // Account for the search engine rows.
                 position -= getPrimaryEngineCount();
                 final Cursor c = mAdapter.getCursor(position);
                 final String url = c.getString(c.getColumnIndexOrThrow(URLColumns.URL));
 
-                // The "urlbar" and "frecency" sessions can be open at the same time. Use the LIST_ITEM
-                // method to set this LOAD_URL event apart from the case where the user commits what's in
-                // the url bar.
-                Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, TelemetryContract.Method.LIST_ITEM);
+                Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, TelemetryContract.Method.LIST_ITEM, "frecency");
 
                 // This item is a TwoLinePageRow, so we allow switch-to-tab.
                 mUrlOpenListener.onUrlOpen(url, EnumSet.of(OnUrlOpenListener.Flags.ALLOW_SWITCH_TO_TAB));
             }
         });
 
         mList.setOnItemLongClickListener(new AdapterView.OnItemLongClickListener() {
             @Override