Remove obsolete methods and member variables from HitIdHelper

Now that all clients have been migrated it is safe to drop all
methods which try to modify the internal state of HitIdHelper after
initialization. The `m_id` member variable can then be dropped too.

Also update methods to pass `std::string` arguments by reference.


Former-commit-id: 22f74dfd