Move isSafari() to @firebase/util

Author: hsubox76

packages/firestore/test/unit/util/async_queue.test.ts

@@ -24,7 +24,7 @@ import { getLogLevel, setLogLevel, LogLevel } from '../../../src/util/log';
 import { Deferred, Rejecter, Resolver } from '../../../src/util/promise';
 import { fail } from '../../../src/util/assert';
 import { IndexedDbTransactionError } from '../../../src/local/simple_db';
-import { isSafari } from '../../util/test_platform';
+import { isSafari } from '@firebase/util';
 
 use(chaiAsPromised);
 

packages/firestore/test/util/test_platform.ts

@@ -22,7 +22,6 @@ import { JsonProtoSerializer } from '../../src/remote/serializer';
 import { debugAssert, fail } from '../../src/util/assert';
 import { ConnectivityMonitor } from './../../src/remote/connectivity_monitor';
 import { NoopConnectivityMonitor } from './../../src/remote/connectivity_monitor_noop';
-import { isNode } from '@firebase/util';
 
 /* eslint-disable no-restricted-globals */
 
@@ -272,12 +271,3 @@ export class TestPlatform implements Platform {
     return this.basePlatform.randomBytes(nBytes);
   }
 }
-
-/** Returns true if we are running in Safari. */
-export function isSafari(): boolean {
-  return (
-    !isNode() &&
-    navigator.userAgent.includes('Safari') &&
-    !navigator.userAgent.includes('Chrome')
-  );
-}

packages/util/src/environment.ts

@@ -125,3 +125,12 @@ export function isUWP(): boolean {
 export function isNodeSdk(): boolean {
   return CONSTANTS.NODE_CLIENT === true || CONSTANTS.NODE_ADMIN === true;
 }
+
+/** Returns true if we are running in Safari. */
+export function isSafari(): boolean {
+  return (
+    !isNode() &&
+    navigator.userAgent.includes('Safari') &&
+    !navigator.userAgent.includes('Chrome')
+  );
+}