GitLab

Before this CL, BVC is Toolbar's delegate, which is a
LocationBarControllerImpl's delegate. We would like for the
LocationBarCoordinator to own LBControllerImpl fully. For this, the
LocationBarCoordinator needs to become its delegate. However, the
delegate is mostly forwarding commands to BVC. So after this CL:

1. BVC is the delegate for LBCoordinator.
2. BVC calls Toolbar methods for Toolbar to react to events in LB.
3. LocationBarCoordinator is delegate for LBControllerImpl.
4. Remove old TODO and related ivar in BVC.

Bug: 785253, 244366
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I7a735ed60a54c040437a91ff225f0c6c31e0e799
Reviewed-on: https://chromium-review.googlesource.com/881165
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531886}