Defer checking for Chrome

Author: erikeldridge

common/api-review/util.api.md

@@ -264,11 +264,6 @@ export function isBrowser(): boolean;
 // @public (undocumented)
 export function isBrowserExtension(): boolean;
 
-// Warning: (ae-missing-release-tag) "isChrome" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)
-//
-// @public (undocumented)
-export function isChrome(): boolean;
-
 // Warning: (ae-missing-release-tag) "isCloudflareWorker" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)
 //
 // @public

packages/util/src/environment.ts

@@ -173,12 +173,6 @@ export function isSafari(): boolean {
   );
 }
 
-export function isChrome(): boolean {
-  return (
-    !isNode() && !!navigator.userAgent && navigator.userAgent.includes('Chrome')
-  );
-}
-
 /**
  * This method checks if indexedDB is supported by current browser/service worker context
  * @return true if indexedDB is supported by current browser/service worker context

packages/vertexai/src/methods/chrome-adapter.test.ts

@@ -25,7 +25,6 @@ import {
   LanguageModelCreateOptions
 } from '../types/language-model';
 import { stub } from 'sinon';
-import * as util from '@firebase/util';
 import { GenerateContentRequest } from '../types';
 
 use(sinonChai);
@@ -41,16 +40,6 @@ describe('ChromeAdapter', () => {
         })
       ).to.be.false;
     });
-    it('returns false if browser is not Chrome', async () => {
-      const chromeStub = stub(util, 'isChrome').returns(false);
-      const adapter = new ChromeAdapter(undefined, 'prefer_on_device');
-      expect(
-        await adapter.isAvailable({
-          contents: []
-        })
-      ).to.be.false;
-      chromeStub.restore();
-    });
     it('returns false if AI API is undefined', async () => {
       const adapter = new ChromeAdapter(undefined, 'prefer_on_device');
       expect(
@@ -175,7 +164,6 @@ describe('ChromeAdapter', () => {
       ).to.be.false;
     });
     it('returns true if model is readily available', async () => {
-      const chromeStub = stub(util, 'isChrome').returns(true);
       const languageModelProvider = {
         availability: () => Promise.resolve(Availability.available)
       } as LanguageModel;
@@ -188,7 +176,6 @@ describe('ChromeAdapter', () => {
           contents: [{ role: 'user', parts: [{ text: 'hi' }] }]
         })
       ).to.be.true;
-      chromeStub.restore();
     });
     it('returns false and triggers download when model is available after download', async () => {
       const languageModelProvider = {

packages/vertexai/src/methods/chrome-adapter.ts

@@ -30,7 +30,6 @@ import {
   LanguageModelMessageRole,
   LanguageModelMessageContent
 } from '../types/language-model';
-import { isChrome } from '@firebase/util';
 
 /**
  * Defines an inference "backend" that uses Chrome's on-device model,
@@ -66,10 +65,6 @@ export class ChromeAdapter {
     if (this.mode === 'only_in_cloud') {
       return false;
     }
-    // Returns false because only Chrome's experimental Prompt API is supported.
-    if (!isChrome()) {
-      return false;
-    }
     // Returns false if the on-device inference API is undefined.;
     if (!this.languageModelProvider) {
       return false;