From 77a533f5b2a1640f52fc2ccbcf8c74c9aed94d6e Mon Sep 17 00:00:00 2001 From: Adam Powell Date: Fri, 16 Oct 2015 10:47:32 -0700 Subject: Fix ChooserTarget app scoring An initial sorting step before applying modifiers to the ChooserTarget scores provided by apps was backwards, causing subsequent target scores to be heavily penalized. Targets are then heavily influenced by the lowest score in the set relative to the targets from other apps. Bug 25013559 Change-Id: I39d5d7c601712fc6a19e694d5846d2c8d17a214f --- core/java/com/android/internal/app/ChooserActivity.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'core/java/com') diff --git a/core/java/com/android/internal/app/ChooserActivity.java b/core/java/com/android/internal/app/ChooserActivity.java index 9708cce..c6b340b 100644 --- a/core/java/com/android/internal/app/ChooserActivity.java +++ b/core/java/com/android/internal/app/ChooserActivity.java @@ -920,7 +920,7 @@ public class ChooserActivity extends ResolverActivity { @Override public int compare(ChooserTarget lhs, ChooserTarget rhs) { // Descending order - return (int) Math.signum(lhs.getScore() - rhs.getScore()); + return (int) Math.signum(rhs.getScore() - lhs.getScore()); } } -- cgit v1.1