follow up loosely draft
authorGrigory Kruglov <gkruglov@mozilla.com>
Thu, 13 Jul 2017 18:05:24 -0400
changeset 608593 20667f490336267e3a1c080c25fe10250042de4e
parent 608592 040b2dd4287c8148f9e3823a160ad9ea2fe21a45
child 608594 aace539eb37e4e8be69aaaced7f9b92b7669acdf
push id68344
push userbmo:gkruglov@mozilla.com
push dateThu, 13 Jul 2017 22:07:34 +0000
milestone56.0a1
follow up loosely MozReview-Commit-ID: I0J1Zb7sVyt
mobile/android/base/java/org/mozilla/gecko/activitystream/ranking/HighlightsRanking.java
--- a/mobile/android/base/java/org/mozilla/gecko/activitystream/ranking/HighlightsRanking.java
+++ b/mobile/android/base/java/org/mozilla/gecko/activitystream/ranking/HighlightsRanking.java
@@ -22,17 +22,17 @@ import static java.util.Collections.sort
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.Action1;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.Action2;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.Func1;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.Func2;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.apply;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.apply2D;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.applyInPairs;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.filter;
-import static org.mozilla.gecko.activitystream.ranking.RankingUtils.mapCursor;
+import static org.mozilla.gecko.activitystream.ranking.RankingUtils.looselyMapCursor;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.mapWithLimit;
 import static org.mozilla.gecko.activitystream.ranking.RankingUtils.reduce;
 
 /**
  * HighlightsRanking.rank() takes a Cursor of highlight candidates and applies ranking to find a set
  * of good highlights. The result set is likely smaller than the cursor size.
  *
  * - First we calculate an initial score based on how frequent we visit the URL and domain.
@@ -89,17 +89,17 @@ public class HighlightsRanking {
         return createHighlightsList(highlights, limit);
     }
 
     /**
      * Extract features for every candidate. The heavy lifting is done in
      * HighlightCandidate.fromCursor().
      */
     @VisibleForTesting static List<HighlightCandidate> extractFeatures(Cursor cursor) {
-        return mapCursor(cursor, new Func1<Cursor, HighlightCandidate>() {
+        return looselyMapCursor(cursor, new Func1<Cursor, HighlightCandidate>() {
             @Override
             public HighlightCandidate call(Cursor cursor) {
                 return HighlightCandidate.fromCursor(cursor);
             }
         });
     }
 
     /**