Skip to content

Commit

Permalink
Remove codepath for desupported safari 3.1. Refs #5832, #9045, #11844…
Browse files Browse the repository at this point in the history
… !strict. Presumably doesn't fix #9045, which is awaiting a test case.

git-svn-id: http://svn.dojotoolkit.org/src/dojo/trunk@27941 560b804f-0ae3-0310-86f3-f6aa0a117693
  • Loading branch information
wkeese committed Feb 17, 2012
1 parent cf6024d commit b917903
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions selector/acme.js
Original file line number Diff line number Diff line change
Expand Up @@ -1066,26 +1066,14 @@ define([
// to make the determination (e.g. does it support QSA, does the query in
// question work in the native QSA impl, etc.).
var nua = navigator.userAgent;
// some versions of Safari provided QSA, but it was buggy and crash-prone.
// We need te detect the right "internal" webkit version to make this work.
var wk = "WebKit/";
var is525 = (
has("webkit") &&
(nua.indexOf(wk) > 0) &&
(parseFloat(nua.split(wk)[1]) > 528)
);

// IE QSA queries may incorrectly include comment nodes, so we throw the
// zipping function into "remove" comments mode instead of the normal "skip
// it" which every other QSA-clued browser enjoys
var noZip = has("ie") ? "commentStrip" : "nozip";

var qsa = "querySelectorAll";
var qsaAvail = (
!!getDoc()[qsa] &&
// see #5832
(!has("safari") || (has("safari") > 3.1) || is525 )
);
var qsaAvail = !!getDoc()[qsa];

//Don't bother with n+3 type of matches, IE complains if we modify those.
var infixSpaceRe = /n\+\d|([^ ])?([>~+])([^ =])?/g;
Expand Down

0 comments on commit b917903

Please sign in to comment.