Bug 1245576 - Add missing unsave.1 UI Telemetry for removing bookmarks and readinglist items from Home Panels r=margaret
authorMark Finkle <mfinkle@mozilla.com>
Thu, 04 Feb 2016 02:45:02 -0500
changeset 283013 148870861645edf0bf62c0bd961247f125a70e40
parent 283012 df69b3262b511333a5d8da8f02123fcac9cc95c2
child 283014 fee0a6a0235f189fda1e21deb884cdcabed1942a
push id29973
push userphilringnalda@gmail.com
push dateFri, 05 Feb 2016 03:25:00 +0000
treeherdermozilla-central@5e024441510f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmargaret
bugs1245576
milestone47.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1245576 - Add missing unsave.1 UI Telemetry for removing bookmarks and readinglist items from Home Panels r=margaret
mobile/android/base/java/org/mozilla/gecko/home/HomeFragment.java
--- a/mobile/android/base/java/org/mozilla/gecko/home/HomeFragment.java
+++ b/mobile/android/base/java/org/mozilla/gecko/home/HomeFragment.java
@@ -389,24 +389,26 @@ public abstract class HomeFragment exten
                 mDB.unpinSite(cr, mPosition);
                 if (mDB.hideSuggestedSite(mUrl)) {
                     cr.notifyChange(SuggestedSites.CONTENT_URI, null);
                 }
             }
 
             switch(mType) {
                 case BOOKMARKS:
+                    Telemetry.sendUIEvent(TelemetryContract.Event.UNSAVE, TelemetryContract.Method.CONTEXT_MENU, "bookmark");
                     mDB.removeBookmarksWithURL(cr, mUrl);
                     break;
 
                 case HISTORY:
                     mDB.removeHistoryEntry(cr, mUrl);
                     break;
 
                 case READING_LIST:
+                    Telemetry.sendUIEvent(TelemetryContract.Event.UNSAVE, TelemetryContract.Method.CONTEXT_MENU, "reading_list");
                     mDB.getReadingListAccessor().removeReadingListItemWithURL(cr, mUrl);
                     GeckoAppShell.sendEventToGecko(GeckoEvent.createBroadcastEvent("Reader:Removed", mUrl));
                     break;
 
                 default:
                     Log.e(LOGTAG, "Can't remove item type " + mType.toString());
                     break;
             }