updated iOS private notes for localStorage. also a stronger test.

This was initiated due to a commit i saw from lindsey simon: https://github.com/andris9/jStorage/pull/18/files
Modernizr · Oct 21, 2011 · a93625c · a93625c · paulirish · Oct 21, 2011
1 parent 1e82aef
commit a93625c
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/modernizr.js b/modernizr.js
@@ -711,19 +711,22 @@ window.Modernizr = (function( window, document, undefined ) {
     //   `('localStorage' in window) && ` test first because otherwise Firefox will
     //   throw http://bugzil.la/365772 if cookies are disabled
 
-    // Also in iOS5 Private Browsing mode, attepting to use localStorage
+    // Also in iOS5 Private Browsing mode, attepting to use localStorage.setItem
     // will throw the exception:
     //   QUOTA_EXCEEDED_ERRROR DOM Exception 22.
+    // Peculiarly, getItem and removeItem calls do not throw.
 
     // Because we are forced to try/catch this, we'll go aggressive.
 
-    // FWIW: IE8 Compat mode supports these features completely:
+    // Just FWIW: IE8 Compat mode supports these features completely:
     //   http://www.quirksmode.org/dom/html5.html
     // But IE8 doesn't support either with local files
 
     tests['localstorage'] = function() {
         try {
-            return !!localStorage.getItem('getItem');
+            localStorage.setItem(mod, mod);
+            localStorage.removeItem(mod);
+            return true;
         } catch(e) {
             return false;
         }