Remove undeeded overrides of IdentityManager::Observer::OnAddAccountToCookieComplete

... from AccountInvestigator and iOS' AccountConsistencyService.

This is part of the work to change GCMS::AddAccountToCookie to take
a completion callback instead of relying on a callback observer
(namely GCMS::Observer::OnAddAccountToCookieCompleted) to notify its
completion.

In these two cases particularly:

- AccountInvestigator's override of
IdentityManager::Observer::OnAddAccountToCookieComplete had an empty
body.

- AccountConsistencyService also listens to OnAccountsInCookieUpdated()
and takes the same action on that event. That callback is fired in all
circumstances that OnAddAccountToCookieCompleted() is fired.

BUG=932180

Change-Id: I47a744319c0c36500b2a368c3b540df5dbeeae9a
Reviewed-on: https://chromium-review.googlesource.com/c/1474034
Reviewed-by: David Roger <droger@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#633081}
4 files changed