Question: utils.isSafariBrowser includes iOS-Chrome and iOS-FireFox

[r-ishigami]

Type of issue

Question

Description

utils.isSafariBrowser regular expressions include iOS x Chrome and iOS x FireFox .
Is it a specification?

Prebid.js/src/utils.js

Lines 817 to 819 in 7c45626

	export function isSafariBrowser() {
	return /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
	}

• iPhone Chrome UserAgent

Mozilla/5.0 (iPhone; CPU iPhone OS 13_3 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) CriOS/80.0.3987.95 Mobile/15E148 Safari/604.1

• iPhone FireFox UserAgent

Mozilla/5.0 (iPhone; CPU OS 13_3_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) FxiOS/23.0  Mobile/15E148 Safari/605.1.15

Proposal

export function isSafariBrowser() {
  return /^((?!chrome|android|crios|fxios).)*safari/i.test(navigator.userAgent);
}

Relation

#3095

[bretg]

Thanks @r-ishigami - PR opened, you're welcome to review.

[r-ishigami]

Thanks, @bretg :-) Reviewed it.