commit 77a533f5b2a1640f52fc2ccbcf8c74c9aed94d6e
author Adam Powell <adamp@google.com> Fri Oct 16 10:47:32 2015 -0700
committer Adam Powell <adamp@google.com> Fri Oct 16 10:58:08 2015 -0700
tree 6ef6542a790805757aa1d5f407f98e5146172766
parent 9c76f61dea69e21eba3b89adbc2a00bcb522584e

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

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 @@
         @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());
         }
     }