diff options
author | Sudheer Shanka <sudheersai@google.com> | 2019-02-11 00:23:29 -0800 |
---|---|---|
committer | Sudheer Shanka <sudheersai@google.com> | 2019-02-11 00:23:29 -0800 |
commit | f69fc9c44581d7e474f2ddf55dae6fcce83f1cd5 (patch) | |
tree | 9f9e6b1915f08183bff9742502781a3e79edbc6c /apct-tests/perftests/multiuser | |
parent | 0671c44a85bf38db54445a909979382d521318cc (diff) |
Don't change the order of test results.
Test: atest apct-tests/perftests/multiuser/src/android/multiuser/UserLifecycleTests.java
Change-Id: Ie21030581ae81509eaa5b3ec31d5e6978dcb23d4
Diffstat (limited to 'apct-tests/perftests/multiuser')
-rw-r--r-- | apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java b/apct-tests/perftests/multiuser/src/android/multiuser/BenchmarkResults.java index e417ca791c45..c1362dc3a6a7 100644 --- 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() { |