From 581be33de1dc8189edb95d45bba5082b14273300 Mon Sep 17 00:00:00 2001
From: bretg <bgorsline@gmail.com>
Date: Wed, 8 Apr 2020 03:27:45 -0400
Subject: [PATCH] isSafariBrowser update  (#5077)

* isSafariBrowser-update

* can't restore navigator object

* update tests to reset navigator.userAgent value to initial value after each test that modifies the value

* fix stubs the navigator userAgent property using sinon

Co-authored-by: idettman <idettman@rubiconproject.com>
---
 src/utils.js            |  2 +-
 test/spec/utils_spec.js | 36 ++++++++++++++++++++++++++++++++++++
 2 files changed, 37 insertions(+), 1 deletion(-)

diff --git a/src/utils.js b/src/utils.js
index f43097e74b9..45822dd7088 100644
--- a/src/utils.js
+++ b/src/utils.js
@@ -817,7 +817,7 @@ export function inIframe() {
 }
 
 export function isSafariBrowser() {
-  return /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
+  return /^((?!chrome|android|crios|fxios).)*safari/i.test(navigator.userAgent);
 }
 
 export function replaceAuctionPrice(str, cpm) {
diff --git a/test/spec/utils_spec.js b/test/spec/utils_spec.js
index e7be6880444..9ba70810a08 100755
--- a/test/spec/utils_spec.js
+++ b/test/spec/utils_spec.js
@@ -922,4 +922,40 @@ describe('Utils', function () {
       });
     });
   });
+
+  describe('isSafariBrowser', function () {
+    let userAgentStub;
+    let userAgent;
+
+    before(function () {
+      userAgentStub = sinon.stub(navigator, 'userAgent').get(function () {
+        return userAgent;
+      });
+    });
+
+    after(function () {
+      userAgentStub.restore();
+    });
+
+    it('properly detects safari', function () {
+      userAgent = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_5) AppleWebKit/536.25 (KHTML, like Gecko) Version/6.0 Safari/536.25';
+      expect(utils.isSafariBrowser()).to.equal(true);
+    });
+    it('does not flag Chrome on MacOS', function () {
+      userAgent = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.163 Safari/537.36';
+      expect(utils.isSafariBrowser()).to.equal(false);
+    });
+    it('does not flag Chrome iOS', function () {
+      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';
+      expect(utils.isSafariBrowser()).to.equal(false);
+    });
+    it('does not flag Firefox iOS', function () {
+      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';
+      expect(utils.isSafariBrowser()).to.equal(false);
+    });
+    it('does not flag Windows Edge', function () {
+      userAgent = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.74 Safari/537.36 Edg/79.0.309.43';
+      expect(utils.isSafariBrowser()).to.equal(false);
+    });
+  });
 });