Comment #15 on issue 22181 by [email protected]: Can't log in to CNET w/Chrome http://code.google.com/p/chromium/issues/detail?id=22181
Ahh...just talked to dglazkov. There are really 3 issues here: He pointed out that they're looking to see if "Storage" is undefined not window.sessionStorage. And this is just plain wrong. It's completely valid for|(typeof Storage != "undefined")| and window.sessionStorage to not work (for example, if a UA implemented just LocalStorage and not SessionStorage). We should definitely talk to CNET about checking window.sessionStorage directly (maybe in addition to seeing if Storage is undefined). Whether we return undefined or null is completely separate, but still worth considering. For Chrome3, I think we should take all traces of it out of the build. I'm happy to write a CL against the branch to do so, if that's the direction we want to go. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs -~----------~----~----~----~------~----~------~--~---
