evaluate -> execute_script when return not needed.

execute does not fetch the return value. It is therefore faster and should be used when the return is not needed.

The two other occurrences on the page were factored out in https://github.com/gitlabhq/gitlabhq/pull/7837