11622 : OutlierDetection should use Ticker, not TimeProvider

Author: vimanikag

binder/src/main/java/io/grpc/binder/internal/PingTracker.java

@@ -104,10 +104,9 @@ private synchronized void fail(Status status) {
     }
 
     private synchronized void success() {
-      long currentTickerTimeNanos = ticker.read();
       if (!done) {
         done = true;
-        executor.execute(() -> callback.onSuccess(currentTickerTimeNanos - startTimeNanos));
+        executor.execute(() -> callback.onSuccess(ticker.read() - startTimeNanos));
       }
     }
   }

netty/src/main/java/io/grpc/netty/AbstractNettyHandler.java

@@ -200,7 +200,7 @@ public void updateWindow() throws Http2Exception {
       setPinging(false);
 
       long currentTickerTimeNanos = ticker.read();
-      long elapsedTime = (currentTickerTimeNanos - lastPingTime);
+      long elapsedTime = Math.max(0L,(currentTickerTimeNanos - lastPingTime));
       if (elapsedTime == 0) {
         elapsedTime = 1;
       }

util/src/main/java/io/grpc/util/OutlierDetectionLoadBalancer.java

@@ -149,7 +149,7 @@ public Status acceptResolvedAddresses(ResolvedAddresses resolvedAddresses) {
         initialDelayNanos = config.intervalNanos;
       } else {
         long currentTickerTimeNanos = ticker.read();
-        long elapsedTimeNanos = currentTickerTimeNanos - detectionTimerStartNanos;
+        long elapsedTimeNanos = Math.max(0L,(currentTickerTimeNanos - detectionTimerStartNanos));
         // If a timer has started earlier we cancel it and use the difference between the start
         // time and now as the interval.
         initialDelayNanos = Math.max(0L,