Enhanced BrowserClient class

Author: seifibrahim32

pkgs/http/lib/src/browser_client.dart

@@ -29,6 +29,23 @@ BaseClient createClient() {
   return BrowserClient();
 }
 
+/// Caching mode used by the [BrowserClient].
+///
+/// Sets the request cache value of the browser Fetch API.
+/// [`Request.cache`](https://developer.mozilla.org/en-US/docs/Web/API/Request/cache) property.
+enum CacheMode {
+  defaultType('default'),
+  reload('reload'),
+  noStore('no-store'),
+  noCache('no-cache'),
+  forceCache('force-cache'),
+  onlyIfCached('only-if-cached');
+
+  final String cacheType;
+
+  const CacheMode(this.cacheType);
+}
+
 /// A `package:web`-based HTTP client that runs in the browser and is backed by
 /// [`window.fetch`](https://fetch.spec.whatwg.org/).
 ///
@@ -44,7 +61,7 @@ BaseClient createClient() {
 class BrowserClient extends BaseClient {
   final _abortController = AbortController();
 
-  final String? _cacheMode;
+  final CacheMode _cacheMode;
 
   /// Create a new browser-based HTTP Client.
   ///
@@ -53,12 +70,10 @@ class BrowserClient extends BaseClient {
   ///
   /// For example:
   /// ```dart
-  /// final client = BrowserClient(cacheMode: 'reload');
+  /// final client = BrowserClient(cacheMode: CacheMode.reload);
   /// ```
-  ///
-  /// Defaults to `default`.
-  /// https://developer.mozilla.org/en-US/docs/Web/API/Request/cache
-  BrowserClient([String? cacheMode]) : _cacheMode = cacheMode ?? 'default';
+  BrowserClient({CacheMode cacheMode = CacheMode.defaultType})
+      : _cacheMode = cacheMode;
 
   /// Whether to send credentials such as cookies or authorization headers for
   /// cross-site requests.
@@ -84,7 +99,7 @@ class BrowserClient extends BaseClient {
             RequestInit(
               method: request.method,
               body: bodyBytes.isNotEmpty ? bodyBytes.toJS : null,
-              cache: _cacheMode!,
+              cache: _cacheMode.cacheType,
               credentials: withCredentials ? 'include' : 'same-origin',
               headers: {
                 if (request.contentLength case final contentLength?)
@@ -199,18 +214,4 @@ Stream<List<int>> _readBody(BaseRequest request, Response response) async* {
 @JS()
 extension type _IterableHeaders._(JSObject _) implements JSObject {
   external void forEach(JSFunction fn);
-}
-
-/// Caching modes for any http request in a browser.
-enum CacheMode {
-  defaultType('default'),
-  reload('reload'),
-  noStore('no-store'),
-  noCache('no-cache'),
-  forceCache('force-cache'),
-  onlyIfCached('only-if-cached');
-
-  final String cacheType;
-
-  const CacheMode(this.cacheType);
-}
+}