Skip to content

Commit 146cfbe

Browse files
committed
Merge pull request cocos2d#1521 from dingpinglv/Iss3825_SimpleAudioEngine
Fixed #3825: update the multiple audio white list
2 parents d3de35e + 8f2e618 commit 146cfbe

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

cocos2d/audio/SimpleAudioEngine.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1960,7 +1960,7 @@ cc.AudioEngine.getInstance = function () {
19601960
if (cc.Browser.supportWebAudio) {
19611961
this._instance = new cc.WebAudioEngine();
19621962
} else {
1963-
if (cc.Browser.multipleAudioWhiteList.indexOf(cc.Browser.type) == -1)
1963+
if (cc.Browser.multipleAudioWhiteList.indexOf(cc.Browser.type) !== -1)
19641964
this._instance = new cc.SimpleAudioEngine();
19651965
else
19661966
this._instance = new cc.SimpleAudioEngineForMobile();

cocos2d/core/platform/miniFramework.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,20 +63,23 @@ cc.Browser.webglWhiteList = ["baidubrowser", "opera", "firefox", "chrome", "safa
6363
* Browsers that multiple audio support well
6464
* @type {Array}
6565
*/
66-
cc.Browser.multipleAudioWhiteList = ["baidubrowser", "opera", "firefox", "chrome", "safari"];
66+
cc.Browser.multipleAudioWhiteList = ["baidubrowser", "opera", "firefox", "chrome", "safari", "ucbrowser", "qqbrowser", "mqqbrowser"];
6767

6868
(function () {
6969
var ua = navigator.userAgent;
7070
cc.Browser.ua = ua.toLowerCase();
7171
cc.Browser.platform = navigator.platform.toLowerCase();
7272
cc.Browser.isMobile = (cc.Browser.ua.indexOf('mobile') != -1 || cc.Browser.ua.indexOf('android') != -1);
7373
cc.Browser.type = (function () {
74-
var browserTypes = cc.Browser.ua.match(/micromessenger|qqbrowser|mqqbrowser|ucbrowser|360browser|baidubrowser|maxthon|ie|opera|firefox/) || cc.Browser.ua.match(/chrome|safari/);
74+
var browserTypes = cc.Browser.ua.match(/micromessenger|qqbrowser|mqqbrowser|ucbrowser|360browser|baiduboxapp|baidubrowser|maxthon|ie|opera|miuibrowser|firefox/)
75+
|| cc.Browser.ua.match(/chrome|safari/);
76+
7577
if (browserTypes && browserTypes.length > 0) {
7678
var el = browserTypes[0];
7779
if (el == 'micromessenger') {
7880
return 'wechat';
79-
}
81+
}else if( el === "safari" && (cc.Browser.ua.match(/android.*applewebkit/) != null))
82+
return "androidbrowser";
8083
return el;
8184
}
8285
return "unknow";

0 commit comments

Comments
 (0)