Kill the killswitch

Looks good to me but it would be good to have a JavaScript wizard confirm this too.

Looks good to me. I'm all for getting rid of the killswitch.

However, we'll need to be sure to add this removal to the module/theme upgrade documentation.

Also, kkaefer, what's going on with the document.cookie stuff in there? Are we moving from Drupal.settings over to a cookie-based system?

Other than the cookie question, this looks good to me.

The cookie is created without explicit domain, that means the cookie will be created under the current domain in the URL. It looks good, but there's an issue here that I would like to mention now that this snippet is being review.

The cookie domain in settings.php can be set to a higher level of the domain being used for the site, and that allows sharing the cookies between subdomains of the same domain. But this cookie "has_js" cannot be shared because it will exist a different version for each subdomain.

If the global $cookie_domain was in Drupal.settings, then it could be used here to create this cookie under the same exact scope the rest of the cookies are created by Drupal.

Code looks good and I tried it and everything works as expected

Yep, it is time, sorry IE4. It's nice that the introduction of behaviors made this patch so much smaller that it would've been in D5 days. Second this RTBC.

Great, thanks for all the reviews folks! I've committed this to HEAD. Marking needs work, as this needs to be reflected in the upgrade documentation, as I'm pretty sure checking for Drupal.jsEnabled is considered a best practice in contrib js files.

Also, markus_petrux's comment at #4 sounds like it's worthy of further discussion. I didn't block this patch for that since kkaefer is right, and it's just moving existing lines of code around, but Markus, could you start another issue to discuss moving this to the Drupal.settings array?

@webchick #8: #445098: Provide a consistent method to generate cookies

I hope that helps.

I'm assigning this to myself to add it to the module/theme upgrade guides

I've added information to:
* Module upgrade guide http://drupal.org/node/224333#no-jsenabled
* Theme upgrade guide (same text) http://drupal.org/update/theme/6/7#no-jsenabled
* Categorical module upgrade page http://drupal.org/node/394070 (I marked it as "unknown" as to Coder warning/upgrade status).

If someone could review the text there, that would probably be good... if the text on those pages is OK, it is probably time (I think) to mark this issue as fixed.

Maybe the bot will ignore it this time...

Looks good