Bug 1329138 - Part 1: Add telemetry probe while removing a bookmark folder. r=nechen
authorJing-wei Wu <topwu.tw@gmail.com>
Wed, 24 May 2017 10:14:07 +0800
changeset 409613 773925ed0edd801db7c655ad70c443b16e819452
parent 409612 794a1edad2b4ba97a9c79ad3d6c24833e50dd91a
child 409614 f0cc36791dde4152d7bbeede9359f10ae1f1d661
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnechen
bugs1329138
milestone55.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 1329138 - Part 1: Add telemetry probe while removing a bookmark folder. r=nechen MozReview-Commit-ID: 5NYMp3uBdNw
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
@@ -466,17 +466,19 @@ public abstract class HomeFragment exten
                     .buildAndShow();
         }
 
         private void removeBookmark(ContentResolver cr) {
             SavedReaderViewHelper rch = SavedReaderViewHelper.getSavedReaderViewHelper(context);
             final boolean isReaderViewPage = rch.isURLCached(info.url);
 
             final String extra;
-            if (isReaderViewPage) {
+            if (info.isFolder) {
+                extra = "bookmark_folder";
+            } else if (isReaderViewPage) {
                 extra = "bookmark_reader";
             } else {
                 extra = "bookmark";
             }
 
             Telemetry.sendUIEvent(TelemetryContract.Event.UNSAVE, TelemetryContract.Method.CONTEXT_MENU, extra);
             db.removeBookmarkWithId(cr, info.bookmarkId);