Bug 1364202 - increase timeout time for waiting playback status changed. draft
authorAlastor Wu <alwu@mozilla.com>
Wed, 17 May 2017 11:33:07 +0800
changeset 579223 76d510b17e7b7b0696742790fc338ee2dcc4a9cb
parent 579221 dbb5f0b4e11562d3b2c376b75ae8f17d15b598fd
child 628927 5ece6037737e73fe13f819d0fc564a23ca6dce04
push id59166
push useralwu@mozilla.com
push dateWed, 17 May 2017 03:33:24 +0000
bugs1364202
milestone55.0a1
Bug 1364202 - increase timeout time for waiting playback status changed. Because the emulator is super slow, it would need longer time to wait for the tab's playback status changed. MozReview-Commit-ID: CLUimz6pF26
mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/MediaPlaybackTest.java
--- a/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/MediaPlaybackTest.java
+++ b/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/MediaPlaybackTest.java
@@ -25,16 +25,17 @@ import android.service.notification.Stat
 import com.robotium.solo.Condition;
 
 abstract class MediaPlaybackTest extends BaseTest {
     private Context mContext;
     private int mPrevIcon = 0;
     private JavascriptBridge mJs;
 
     private static final int UI_CHANGED_WAIT_MS = 6000;
+    private static final int MEDIA_PLAYBACK_CHANGED_WAIT_MS = 15000;
 
     protected final void info(String msg) {
         mAsserter.dumpLog(msg);
     }
 
     protected final Context getContext() {
         if (mContext == null) {
             mContext = getInstrumentation().getTargetContext();
@@ -101,30 +102,30 @@ abstract class MediaPlaybackTest extends
         if (tab.isAudioPlaying() == isTabPlaying) {
             return;
         }
         waitForCondition(new Condition() {
             @Override
             public boolean isSatisfied() {
                 return tab.isAudioPlaying() == isTabPlaying;
             }
-        }, MAX_WAIT_MS);
+        }, MEDIA_PLAYBACK_CHANGED_WAIT_MS);
     }
 
     private final void waitUntilTabMediaPlaybackChanged(final Tab tab,
                                                         final boolean isTabPlaying) {
         if (tab.isMediaPlaying() == isTabPlaying) {
             return;
         }
         waitForCondition(new Condition() {
             @Override
             public boolean isSatisfied() {
                 return tab.isMediaPlaying() == isTabPlaying;
             }
-        }, MAX_WAIT_MS);
+        }, MEDIA_PLAYBACK_CHANGED_WAIT_MS);
     }
 
     /**
      * These methods are used to check Tab's playing related attributes.
      * isMediaPlaying : is any media playing (might be audible or non-audbile)
      * isAudioPlaying : is any audible media playing
      */
     protected final void checkTabMediaPlayingState(final Tab tab,