Bug 1243558 - Add telemetry to screenshots click handler. r=sebastian draft
authorMichael Comella <michael.l.comella@gmail.com>
Mon, 29 Feb 2016 17:16:37 -0800
changeset 335642 e88fabd64fe4f5354a5321d023ff1d2f61152c25
parent 335641 e64e95e5f7701fdc7561ce23b60f465701de96e1
child 335643 416f2bd05eb2dfdd9e89d687348c0e085db1dbdc
push id11840
push usermichael.l.comella@gmail.com
push dateTue, 01 Mar 2016 01:35:09 +0000
reviewerssebastian
bugs1243558
milestone47.0a1
Bug 1243558 - Add telemetry to screenshots click handler. r=sebastian MozReview-Commit-ID: 1B9nBOrx6TT
mobile/android/base/java/org/mozilla/gecko/home/BookmarksListView.java
--- a/mobile/android/base/java/org/mozilla/gecko/home/BookmarksListView.java
+++ b/mobile/android/base/java/org/mozilla/gecko/home/BookmarksListView.java
@@ -81,16 +81,18 @@ public class BookmarksListView extends H
         final Cursor cursor = adapter.getCursor();
         if (cursor == null) {
             return;
         }
 
         cursor.moveToPosition(position);
 
         if (adapter.getOpenFolderType() == BookmarksListAdapter.FolderType.SCREENSHOTS) {
+            Telemetry.sendUIEvent(TelemetryContract.Event.LOAD_URL, TelemetryContract.Method.LIST_ITEM, "bookmarks-screenshot");
+
             final String fileUrl = "file://" + cursor.getString(cursor.getColumnIndex(BrowserContract.UrlAnnotations.VALUE));
             getOnUrlOpenListener().onUrlOpen(fileUrl, EnumSet.of(OnUrlOpenListener.Flags.ALLOW_SWITCH_TO_TAB));
             return;
         }
 
         int type = cursor.getInt(cursor.getColumnIndexOrThrow(Bookmarks.TYPE));
         if (type == Bookmarks.TYPE_FOLDER) {
             // If we're clicking on a folder, update adapter to move to that folder