[Pkg-javascript-commits] [dojo] 53/149: Fixed various issues with sniff on WP 8.1 U1, fixes #18540

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository dojo.

commit 0c4df7a2ee80eb03b6e4d55996df76c24515387c
Author: Youcef Mammar <youcef.mammar at fr.ibm.com>
Date:   Thu Mar 26 11:24:51 2015 +0100

    Fixed various issues with sniff on WP 8.1 U1, fixes #18540
---
 sniff.js | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/sniff.js b/sniff.js
index d26d37e..500d54a 100644
--- a/sniff.js
+++ b/sniff.js
@@ -19,11 +19,14 @@ define(["./has"], function(has){
 		has.add("wp", parseFloat(dua.split("Windows Phone")[1]) || undefined);
 		has.add("msapp", parseFloat(dua.split("MSAppHost/")[1]) || undefined);
 		has.add("khtml", dav.indexOf("Konqueror") >= 0 ? tv : undefined);
-		has.add("webkit", parseFloat(dua.split("WebKit/")[1]) || undefined);
+		has.add("webkit", !has("wp") // NOTE: necessary since Windows Phone 8.1 Update 1, see #18540
+				&& parseFloat(dua.split("WebKit/")[1]) || undefined);
 		has.add("chrome", parseFloat(dua.split("Chrome/")[1]) || undefined);
 		has.add("android", !has("wp") // NOTE: necessary since Windows Phone 8.1 Update 1, see #18528
 				&& parseFloat(dua.split("Android ")[1]) || undefined);
-		has.add("safari", dav.indexOf("Safari") >= 0 && !has("chrome") && !has("android") ?
+		has.add("safari", dav.indexOf("Safari") >= 0 
+				&& !has("wp") // NOTE: necessary since Windows Phone 8.1 Update 1, see #18540
+				&& !has("chrome") && !has("android") ?
 			parseFloat(dav.split("Version/")[1]) : undefined);
 		has.add("mac", dav.indexOf("Macintosh") >= 0);
 		has.add("quirks", document.compatMode == "BackCompat");
@@ -49,7 +52,8 @@ define(["./has"], function(has){
 			}
 
 			// Mozilla and firefox
-			if(dua.indexOf("Gecko") >= 0 && !has("khtml") && !has("webkit") && !has("trident")){
+			if(dua.indexOf("Gecko") >= 0 && !has("wp") // NOTE: necessary since Windows Phone 8.1 Update 1
+					&& !has("khtml") && !has("webkit") && !has("trident")){
 				has.add("mozilla", tv);
 			}
 			if(has("mozilla")){

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/dojo.git