From 3182adeefaad8a8ad8d93624f8acf8eccbf34bce Mon Sep 17 00:00:00 2001 From: Michael Lange Date: Wed, 24 Jun 2020 20:30:52 -0700 Subject: [PATCH] Remove special-casing of Safari to force use of a PollLogger --- ui/app/utils/classes/stream-logger.js | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/ui/app/utils/classes/stream-logger.js b/ui/app/utils/classes/stream-logger.js index 7105c09f3..aedd18f47 100644 --- a/ui/app/utils/classes/stream-logger.js +++ b/ui/app/utils/classes/stream-logger.js @@ -89,16 +89,5 @@ export default class StreamLogger extends EmberObject.extend(AbstractLogger) { } StreamLogger.reopenClass({ - isSupported: !!window.ReadableStream && !isSafari(), + isSupported: !!window.ReadableStream, }); - -// Fetch streaming doesn't work in Safari yet despite all the primitives being in place. -// Bug: https://bugs.webkit.org/show_bug.cgi?id=185924 -// Until this is fixed, Safari needs to be explicitly targeted for poll-based logging. -function isSafari() { - const oldSafariTest = /constructor/i.test(window.HTMLElement); - const newSafariTest = (function(p) { - return p.toString() === '[object SafariRemoteNotification]'; - })(!window['safari'] || (typeof window.safari !== 'undefined' && window.safari.pushNotification)); - return oldSafariTest || newSafariTest; -}