Bug 1384021 - ActivityStream: Remove outdated constants. r?mcomella draft
authorSebastian Kaspari <s.kaspari@gmail.com>
Wed, 02 Aug 2017 19:01:19 +0200
changeset 641637 9f6a83ea09d0fabd8c692432d1e8b13c249c8a4b
parent 641636 5492d887d4292d2a4263a1e38f2007d017bcedcd
child 724865 8c1d65f4ae6b5daba10fe2bf7835a6f3c253ffff
push id72604
push users.kaspari@gmail.com
push dateMon, 07 Aug 2017 12:55:30 +0000
reviewersmcomella
bugs1384021, 1377287
milestone57.0a1
Bug 1384021 - ActivityStream: Remove outdated constants. r?mcomella While editing this file I noticed that those constants are unused now. This is leftover from bug 1377287. MozReview-Commit-ID: 2VQxfC5gq3g
mobile/android/base/java/org/mozilla/gecko/activitystream/ActivityStream.java
--- a/mobile/android/base/java/org/mozilla/gecko/activitystream/ActivityStream.java
+++ b/mobile/android/base/java/org/mozilla/gecko/activitystream/ActivityStream.java
@@ -20,41 +20,16 @@ import org.mozilla.gecko.preferences.Gec
 import org.mozilla.gecko.util.StringUtils;
 import org.mozilla.gecko.util.publicsuffix.PublicSuffix;
 
 import java.util.Arrays;
 import java.util.List;
 
 public class ActivityStream {
     /**
-     * List of undesired prefixes for labels based on a URL.
-     *
-     * This list is by no means complete and is based on those sources:
-     * - https://gist.github.com/nchapman/36502ad115e8825d522a66549971a3f0
-     * - https://github.com/mozilla/activity-stream/issues/1311
-     */
-    private static final List<String> UNDESIRED_LABEL_PREFIXES = Arrays.asList(
-            "index.",
-            "home."
-    );
-
-    /**
-     * Undesired labels for labels based on a URL.
-     *
-     * This list is by no means complete and is based on those sources:
-     * - https://gist.github.com/nchapman/36502ad115e8825d522a66549971a3f0
-     * - https://github.com/mozilla/activity-stream/issues/1311
-     */
-    private static final List<String> UNDESIRED_LABELS = Arrays.asList(
-            "render",
-            "login",
-            "edit"
-    );
-
-    /**
      * Is Activity Stream enabled?
      */
     public static boolean isEnabled(Context context) {
         // Fennec 57+ here we come!
         //   The only reason this method still exists is so that the old home panel code isn't
         //   suddenly unused and triggers all kinds of lint errors. However we should clean
         //   this up soon (Bug 1386725).
         return true;