Refactor submit handler to process nodes all at once (rather than per row)

I looked through the code, and _taxonomy_access_node_access_update() is called in the following functions:

1. taxonomy_access_admin_delete_role()
2. taxonomy_access_admin_form_submit()
3. taxonomy_access_grant_update()
4. taxonomy_access_defaults_update()
5. hook_taxonomy()

Numbers 3 and 4 are all called only by the submit handler itself (#2), or by other functions that are in turn called by the submit handler. Therefore, this is what I propose:

1. Modify _taxonomy_access_cache_affected_nodes() to append more affected nodes to an existing list
2. Replace calls to _taxonomy_access_node_access_update() in 3 and 4 above with _taxonomy_access_cache_affected_nodes().
3. Place the following just before return in the submit handler for 'add' and 'save all' cases:
   

   _taxonomy_access_node_access_update(_taxonomy_access_cache_affected_nodes());
   

4. Refactor the 'delete' case in the submit handler to use the cache where it currently calls _taxonomy_access_node_access_update() and then call that once with the cache as above.

Attached patch implements the solution described in #1. This needs to be tested thoroughly once #739732: Nodes not updated on vocabulary default add and #739874: Some nodes not updated on role disable are fixed.

The patch needs to be rerolled against CVS; the first hunk fails. (The other 7 succeed with fuzz + offsets).

Here's the patch rerolled against CVS.

Tested adding, deleting, and changing both terms and defaults; the list of nodes updated seems appropriate in all three cases.

Committed to 6.x-1.x-dev:
http://drupal.org/cvs?commit=341362