[ios] moving bookmark controller dismissal out of loadURLWithParams

Moving it to CRWWebStateObserver::didStartNavigation observer cb.  I think the only way BookmarkModalController is still up is when it is up and user triggers a load from an external app, like iGSA.

Bug: 907527
Change-Id: I73ad94294ef272a415b6c87a73422dc92695ee2c
Reviewed-on: https://chromium-review.googlesource.com/c/1421184
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: David Jean <djean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624759}
1 file changed