Merge "Don't change the order of test results."

2019-02-22 18:31:07 +00:00
parent dd6eb81b19 f69fc9c445
commit c7dc355feb
1 changed files with 5 additions and 3 deletions
--- a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java
+++ b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java
@@ -60,11 +60,13 @@ public class BenchmarkResults {
        if (size == 0) {
            return 0f;
        }
-        Collections.sort(mResults);
+
+        final ArrayList<Long> resultsCopy = new ArrayList<>(mResults);
+        Collections.sort(resultsCopy);
        final int idx = size / 2;
        return size % 2 == 0
-                ? (double) (mResults.get(idx) + mResults.get(idx - 1)) / 2
-                : mResults.get(idx);
+                ? (double) (resultsCopy.get(idx) + resultsCopy.get(idx - 1)) / 2
+                : resultsCopy.get(idx);
    }

    private double standardDeviation() {