Convert all core module to use new static caching API

Im starting on node module now and will post patch as i move through them.

First stab at this process please review thoroughly to make sure the rest of these come in cleanly next week. -mf
#423806: Update user module to use drupal_static()
#423822: Update node module to use drupal_static()

Here's some more:
#480102: Update aggregator module to use drupal_static()
#480112: Update block module to use drupal_static()
#480122: Update blogapi module to use drupal_static()
#480412: Update book module to use drupal_static()
#480414: Update comment module to use drupal_static()
#480416: Update filter module to use drupal_static()
#480418: Update forum module to use drupal_static()
#480424: Update locale module to use drupal_static()
#480426: Update menu module to use drupal_static()
#480428: Update openid module to use drupal_static()
#480430: Update search module to use drupal_static()

And some more:
#481498: Update simpletest module to use drupal_static()
#481500: Update syslog module to use drupal_static()
#481502: Update system module to use drupal_static()
#481504: Update translation module to use drupal_static()
#481506: Update trigger module to use drupal_static()
#481508: Update update module to use drupal_static()

This was created a few days ago:
#447862: Convert field to the new static caching API

Most of these sub-issues are pretty quick fixes (except for field, locale, node, simpletest, and user).

There are still two static vars:

modules/simpletest/drupal_web_test_case.php:875:    static $available;
modules/system/system.queue.inc:63:    static $queues;

Both vars in defined within class definitions. I'm assuming you can't register those since there may be multiple instances of the respective classes. Still, it seems like the code is designed so that there won't ever be multiple instances of these two classes. Should these static vars be left alone? Please advice.

Rerolled some of the sub-issues.

Locale, search, and filter still need work.

The remaining issues are now locale, search and trigger. IMHO, locale and search are RTBC, while trigger needs only a tiny code-style fix. Once these three are in, this meta-issue can be closed...

Re-rolled #423822: Update node module to use drupal_static()

A search also reveals 39 uses in 19 files of 'static $var' in /modules and 72 in 29 files inside /includes as of today.

Are we trying to clean all of these up?

@agentrickard, some uses of local statics are legitimate and will not be converted. I forgot the exact criteria, but apparently drupal_static should only be used for variables that might conceivably be reset at some point.

Right, I read the docs on API. Have we sanity checked all the remaining statics?

I'm marking this fixed, if we have un-resettable statics, we can find them individually.