Allow native Jitsi SDK on dev and staging Brave Talk hosts (#8754)

brave · Feb 9, 2024 · 1ab30ba · 1ab30ba
1 parent 0d7f871
commit 1ab30ba
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 17 deletions.
diff --git a/Sources/Brave/Frontend/Browser/DomainUserScript.swift b/Sources/Brave/Frontend/Browser/DomainUserScript.swift
@@ -43,10 +43,9 @@ enum DomainUserScript: CaseIterable {
                   "search-dev-local.brave.com"])
 #if canImport(BraveTalk)
     case .braveTalkHelper:
-      return Set(["talk.brave.com", "beta.talk.brave.com",
-                 "talk.bravesoftware.com", "beta.talk.bravesoftware.com",
-                 "dev.talk.brave.software", "beta.talk.brave.software",
-                 "talk.brave.software"])
+      return Set(["talk.brave.com",
+                  "talk.bravesoftware.com",
+                  "talk.brave.software"])
 #endif
     case .bravePlaylistFolderSharingHelper:
       return Set(["playlist.bravesoftware.com", "playlist.brave.com"])

diff --git a/...d/UserContent/UserScripts/Scripts_Dynamic/Scripts/DomainSpecific/Paged/BraveTalkScript.js b/...d/UserContent/UserScripts/Scripts_Dynamic/Scripts/DomainSpecific/Paged/BraveTalkScript.js
@@ -29,14 +29,12 @@ window.__firefox__.includeOnce("BraveTalkScript", function($) {
     });
   });
 
-  if (document.location.host === "talk.brave.com") {
-    const postRoom = $((event) => {
-      if (event.target.tagName !== undefined && event.target.tagName.toLowerCase() == "iframe") {
-        launchNativeBraveTalk(event.target.src);
-        window.removeEventListener("DOMNodeInserted", postRoom)
-      }
-    });
-    window.addEventListener("DOMNodeInserted", postRoom);
-  }
+  const postRoom = $((event) => {
+    if (event.target.tagName !== undefined && event.target.tagName.toLowerCase() == "iframe") {
+      launchNativeBraveTalk(event.target.src);
+      window.removeEventListener("DOMNodeInserted", postRoom)
+    }
+  });
+  window.addEventListener("DOMNodeInserted", postRoom);
 });
 
diff --git a/Tests/ClientTests/DomainUserScriptTests.swift b/Tests/ClientTests/DomainUserScriptTests.swift
@@ -38,11 +38,7 @@ class DomainUserScriptTests: XCTestCase {
   func testBraveTalkAPIAvailability() throws {
     let goodURLs = [
       URL(string: "https://talk.brave.com"),
-      URL(string: "https://beta.talk.brave.com"),
       URL(string: "https://talk.bravesoftware.com"),
-      URL(string: "https://beta.talk.bravesoftware.com"),
-      URL(string: "https://dev.talk.brave.software"),
-      URL(string: "https://beta.talk.brave.software"),
       URL(string: "https://talk.brave.com/account"),
     ].compactMap { $0 }