Permalink
Browse files

totally redid cookies test. thx @tauren. fixes 191

  • Loading branch information...
1 parent e8cc0f6 commit 33f00fbbeb12e92bf24711ea386e722cce6f60cc @paulirish paulirish committed Dec 12, 2011
Showing with 13 additions and 31 deletions.
  1. +13 −31 feature-detects/cookies.js
@@ -1,33 +1,15 @@
-// cookies?
-// both delicious and useful.
+// by tauren
+// https://github.com/Modernizr/Modernizr/issues/191
+
+Modernizr.addTest('cookies', function () {
+ // Quick test if browser has cookieEnabled host property
+ if (navigator.cookieEnabled) return true;
+ // Create cookie
+ document.cookie = "cookietest=1";
+ var ret = document.cookie.indexOf("cookietest=") != -1;
+ // Delete cookie
+ document.cookie = "cookietest=1; expires=Thu, 01-Jan-1970 00:00:01 GMT";
+ return ret;
+});
-(function(){
-
-// FIXME: we could do this with less code.
-function createCookie(name,value) {
- document.cookie = name+"="+value+"; path=/";
-}
-
-function readCookie(name) {
- var nameEQ = name + "=";
- var ca = document.cookie.split(';');
- for(var i=0;i < ca.length;i++) {
- var c = ca[i];
- while (c.charAt(0)==' ') c = c.substring(1,c.length);
- if (c.indexOf(nameEQ) == 0) return c.substring(nameEQ.length,c.length);
- }
- return null;
-}
-
-function eraseCookie(name) {
- createCookie(name,"",-1);
-}
-
-
-var random = '' + Math.round(Math.random() * 1e8);
-createCookie('Modernizr', random);
-Modernizr.cookies = readCookie('Modernizr') == random;
-eraseCookie('Modernizr');
-
-})();

0 comments on commit 33f00fb

Please sign in to comment.