Separate support for tab capture from chromium

vantezzen · Aug 10, 2022 · 452d4c4 · 452d4c4
1 parent c42a62d
commit 452d4c4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/src/background/index.ts b/src/background/index.ts
@@ -1,9 +1,9 @@
-import { isChromium } from "~shared/platform"
+import { supportsTabCapture } from "~shared/platform"
 
 import BackgroundManager from "./BackgroundManager"
 import setupKeyboardShortcutsListener from "./keyboardShortcuts"
 
-if (isChromium) {
+if (supportsTabCapture) {
   new BackgroundManager()
 }
 setupKeyboardShortcutsListener()

diff --git a/src/contents/index.tsx b/src/contents/index.tsx
@@ -9,7 +9,7 @@ import browser from "webextension-polyfill"
 
 import getState from "~shared/state"
 
-import { isChromium } from "../shared/platform"
+import { supportsTabCapture } from "../shared/platform"
 import setupFirefoxContent from "./lib/browserSetup/firefox"
 import setupBrowserContent from "./lib/browserSetup/shared"
 import Bar from "./lib/command-bar/Bar"
@@ -24,7 +24,7 @@ export const config: PlasmoContentScript = {
 const state = getState(StateEnvironment.Content)
 
 setupBrowserContent(state)
-if (!isChromium) {
+if (!supportsTabCapture) {
   setupFirefoxContent(state)
 }
 

diff --git a/src/shared/platform.ts b/src/shared/platform.ts
@@ -1 +1,2 @@
-export const isChromium = navigator.userAgent.includes('Chrome');
+export const isChromium = navigator.userAgent.includes("Chrome")
+export const supportsTabCapture = isChromium