Neal, thanks for looking at this.

Size:  Yeah.  :-\  I don't know what to do about this.  There were a lot of bugs, so there are a lot of diffs.  :(

Whitespace:  I checked, and the extra whitespace is definitely in trunk's Lib/test/test_minidom.py; a recent change added it.  This patch removes it.  (I could revert the whitespace anyway, just to reduce the size of the patch.  Let me know if I should do this.)

Doc:  Oops!  I will fix this today and upload a new patch.

pass:  Thanks, I'll change as suggested.  (Just to explain myself-- I used "if name == 'nodeValue': pass; else: ..." because 'nodeValue' is the special case, and the special behavior is "do nothing".  But I'm not attached to that wording.  I'll change it.)