Store a copy of the face to avoid returning temporaries
The HierarchicFaceIterator now stores a face directly which is returned by reference when the iterator is dereferenced. As the EntityPointer got removed and we store a copy we don't need any RawPointerFace and StorableFace anymore.