diff --git a/ReactAndroid/src/main/java/com/facebook/react/modules/core/JavaTimerManager.java b/ReactAndroid/src/main/java/com/facebook/react/modules/core/JavaTimerManager.java index 1766d089a1c..37548d4c46a 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/modules/core/JavaTimerManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/modules/core/JavaTimerManager.java @@ -373,4 +373,23 @@ public class JavaTimerManager { } }); } + + /** + * Returns a bool representing whether there are any active timers that will be fired within a + * certain period of time. Disregards repeating timers (setInterval). Used for testing to + * determine if RN is idle. + * + * @param rangeMs The time range, in ms, to check + * @return True if there are pending timers within the given range; false otherwise + */ + /* package */ boolean hasActiveTimersInRange(long rangeMs) { + synchronized (mTimerGuard) { + for (Timer timer : mTimers) { + if (!timer.mRepeat && timer.mInterval < rangeMs) { + return true; + } + } + } + return false; + } } diff --git a/ReactAndroid/src/main/java/com/facebook/react/modules/core/TimingModule.java b/ReactAndroid/src/main/java/com/facebook/react/modules/core/TimingModule.java index dbbbf843b38..8ac7f07606a 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/modules/core/TimingModule.java +++ b/ReactAndroid/src/main/java/com/facebook/react/modules/core/TimingModule.java @@ -11,6 +11,7 @@ import com.facebook.fbreact.specs.NativeTimingSpec; import com.facebook.react.bridge.LifecycleEventListener; import com.facebook.react.bridge.ReactApplicationContext; import com.facebook.react.bridge.WritableArray; +import com.facebook.react.common.annotations.VisibleForTesting; import com.facebook.react.devsupport.interfaces.DevSupportManager; import com.facebook.react.jstasks.HeadlessJsTaskContext; import com.facebook.react.jstasks.HeadlessJsTaskEventListener; @@ -134,4 +135,9 @@ public final class TimingModule extends NativeTimingSpec headlessJsTaskContext.removeTaskEventListener(this); mJavaTimerManager.onInstanceDestroy(); } + + @VisibleForTesting + public boolean hasActiveTimersInRange(long rangeMs) { + return mJavaTimerManager.hasActiveTimersInRange(rangeMs); + } } diff --git a/ReactAndroid/src/test/java/com/facebook/react/modules/BUCK b/ReactAndroid/src/test/java/com/facebook/react/modules/BUCK index 4cd9e7446e9..3b5337ebbb6 100644 --- a/ReactAndroid/src/test/java/com/facebook/react/modules/BUCK +++ b/ReactAndroid/src/test/java/com/facebook/react/modules/BUCK @@ -28,6 +28,7 @@ rn_robolectric_test( react_native_target("java/com/facebook/react/common/network:network"), react_native_target("java/com/facebook/react/devsupport:interfaces"), react_native_target("java/com/facebook/react/jstasks:jstasks"), + react_native_target("java/com/facebook/react/module/annotations:annotations"), react_native_target("java/com/facebook/react/modules/blob:blob"), react_native_target("java/com/facebook/react/modules/camera:camera"), react_native_target("java/com/facebook/react/modules/clipboard:clipboard"), diff --git a/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.java b/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.java index 1656a6a23ab..7a98a0f2ba8 100644 --- a/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.java +++ b/ReactAndroid/src/test/java/com/facebook/react/modules/timing/TimingModuleTest.java @@ -7,6 +7,7 @@ package com.facebook.react.modules.timing; +import static org.fest.assertions.api.Assertions.assertThat; import static org.mockito.Mockito.*; import com.facebook.react.bridge.Arguments; @@ -254,6 +255,19 @@ public class TimingModuleTest { verify(mJSTimersMock).callIdleCallbacks(SystemClock.currentTimeMillis()); } + @Test + public void testActiveTimersInRange() { + mTimingModule.onHostResume(); + assertThat(mTimingModule.hasActiveTimersInRange(100)).isFalse(); + + mTimingModule.createTimer(41, 1, 0, true); + assertThat(mTimingModule.hasActiveTimersInRange(100)).isFalse(); // Repeating + + mTimingModule.createTimer(42, 150, 0, false); + assertThat(mTimingModule.hasActiveTimersInRange(100)).isFalse(); // Out of range + assertThat(mTimingModule.hasActiveTimersInRange(200)).isTrue(); // In range + } + private static class PostFrameIdleCallbackHandler implements Answer { private ChoreographerCompat.FrameCallback mFrameCallback;