commit 94be2c1b3b86fff9463f7b543b63f6f543179097
author Wei Jia <wjia@google.com> Mon Jul 13 13:18:56 2015 -0700
committer Wei Jia <wjia@google.com> Mon Jul 13 13:18:56 2015 -0700
tree d0bd5ceedde2db32fbd093966a386bdb9b5b676c
parent d56e346bb734a50595a096909b3b6d4354b4646a

MediaSyncTest: use float for playback rate tolerance to keep consistent with others.

Bug: 22069191
Change-Id: I370c67aa4e0165d13d3497203624b7af014df252

tests/tests/media/src/android/media/cts/MediaSyncTest.java

diff --git a/tests/tests/media/src/android/media/cts/MediaSyncTest.java b/tests/tests/media/src/android/media/cts/MediaSyncTest.java
index 41d8d89..b334040 100644
--- a/tests/tests/media/src/android/media/cts/MediaSyncTest.java
+++ b/tests/tests/media/src/android/media/cts/MediaSyncTest.java
@@ -56,7 +56,7 @@
     private static final String LOG_TAG = "MediaSyncTest";
 
     private final long NO_TIMESTAMP = -1;
-    private final int PLAYBACK_RATE_TOLERANCE_PERCENT = 2;
+    private final float FLOAT_PLAYBACK_RATE_TOLERANCE = .02f;
     private final long TIME_MEASUREMENT_TOLERANCE_US = 20000;
     final int INPUT_RESOURCE_ID =
             R.raw.video_480x360_mp4_h264_1350kbps_30fps_aac_stereo_192kbps_44100hz;
@@ -501,9 +501,9 @@
                     mediaDurationUs,
                     playTimeUs * playbackRate,
                     // sync.getTolerance() is MediaSync's tolerance of the playback rate, whereas
-                    // PLAYBACK_RATE_TOLERANCE_PERCENT / 100 is our test's tolerance.
+                    // FLOAT_PLAYBACK_RATE_TOLERANCE is our test's tolerance.
                     // We need to add both to get an upperbound for allowable error.
-                    mediaDurationUs * (sync.getTolerance() + PLAYBACK_RATE_TOLERANCE_PERCENT / 100.)
+                    mediaDurationUs * (sync.getTolerance() + FLOAT_PLAYBACK_RATE_TOLERANCE)
                             + TIME_MEASUREMENT_TOLERANCE_US);
         }