New ClassAds provides an iterator via the begin() and end() methods for traversing all of the attributes in the ad. The iterator doesn't handle any chained ad, but no calls to NextName() deal with chained ads.

NextName() returns new-allocated memory that the caller must free. All of the existing callsites don't appear to do so.

Developer time: several hours to convert the callsites, could be done by a student.

Progress: All callsites now use NextNameOriginal().