Currently, G1EvacFailureObjectsSet::iterate(ObjectClosure* closure) is not a const method, but it looks like a const method. It will be a surprise for reader, we should refine it in some way.

I.e. you would not expect that an iteration method is destructive, clearing out the set it just iterated over.