Don't add change owners as reviewers on replace

The old logic here was pretty convoluted to follow. We
no longer use the have variables, instead the code uses
Set<Account.Id> haveApprovals. So drop those from the
code and simplify the way this is written.

In the process add a new conditional that avoids adding
the change owner as a reviewer when they are the author
or the committer of the new commit. In either case the
owner is already going to be part of the change.

Change-Id: I471b2934bcd2446ab4101636363ca2020a78a866
2 files changed