Attend to review comments

Author: brizental

glean/src/core/upload/index.ts

@@ -98,7 +98,7 @@ class PingUploader implements PingsDatabaseObserver {
     platform: Platform,
     private readonly pingsDatabase = Context.pingsDatabase,
     private readonly policy = new Policy(),
-    private readonly rateLimiter = new RateLimiter(RATE_LIMITER_INTERVAL, MAX_PINGS_PER_INTERVAL)
+    private readonly rateLimiter = new RateLimiter(RATE_LIMITER_INTERVAL_MS, MAX_PINGS_PER_INTERVAL)
   ) {
     this.processing = [];
     // Initialize the ping uploader with either the platform defaults or a custom

glean/src/core/upload/rate_limiter.ts

@@ -13,9 +13,6 @@ export const enum RateLimiterState {
   // The RateLimiter has not reached the maximum count, but it is also not incrementing.
   Stopped,
   // The RateLimiter has reached the maximum count for the current interval.
-  //
-  // This variant contains the remaining time (in milliseconds)
-  // until the rate limiter is not throttled anymore.
   Throttled,
 }
 
@@ -62,7 +59,7 @@ class RateLimiter {
   /**
    * The rate limiter should reset if
    *
-   * 1. It has never started;
+   * 1. It has never started i.e. `started` is still `undefined`;
    * 2. It has been started more than the interval time ago;
    * 3. Something goes wrong while trying to calculate the elapsed time since the last reset.
    *