Removed obsolete setting of framework id in executor.

Split from https://reviews.apache.org/r/19795/. This patch removes an
obsolete piece of code in the slave which sets the framework id in the
executor info. It has already been marked for deletion and significantly
simplifies the new reliable reregistration code.

Review: https://reviews.apache.org/r/19877