Bug 1315594 - Add missing break r?sebastian draft
authorAndrzej Hunt <ahunt@mozilla.com>
Mon, 07 Nov 2016 13:39:49 +0100
changeset 434816 4a0cb41f6fe590d20ca1b7d4d9667e1dbff5b1d5
parent 434815 5d57a8220958517fd5860d8c261e68aaaf51a433
child 536124 129edd27d2c925303d80eeb889e137a68a10d8a4
push id34835
push userahunt@mozilla.com
push dateMon, 07 Nov 2016 12:40:59 +0000
reviewerssebastian
bugs1315594
milestone52.0a1
Bug 1315594 - Add missing break r?sebastian I'm not quite sure when this slipped in - and I probably didn't test carefully enough. MozReview-Commit-ID: Ki1VC693NHQ
mobile/android/base/java/org/mozilla/gecko/home/activitystream/menu/ActivityStreamContextMenu.java
--- a/mobile/android/base/java/org/mozilla/gecko/home/activitystream/menu/ActivityStreamContextMenu.java
+++ b/mobile/android/base/java/org/mozilla/gecko/home/activitystream/menu/ActivityStreamContextMenu.java
@@ -193,16 +193,17 @@ public abstract class ActivityStreamCont
                 ThreadUtils.postToBackgroundThread(new Runnable() {
                     @Override
                     public void run() {
                         BrowserDB.from(context)
                                 .removeHistoryEntry(context.getContentResolver(),
                                         url);
                     }
                 });
+                break;
 
             default:
                 throw new IllegalArgumentException("Menu item with ID=" + item.getItemId() + " not handled");
         }
 
         dismiss();
         return true;
     }