Reland "[omnibox]: Initialize HistoryURLProvider earlier."

This is a reland of 6e200e4f600e2eb0258e244177aa9439ff0b6bb4

Original change's description:
> [omnibox]: Initialize HistoryURLProvider earlier.
> 
> This CL moves the HistoryURlProvider's initialization after the SearchProvider's
> but before the ZeroSuggestProvider's and ClipboardURLProvider's.
> 
> For threading reasons, CL 1384637 moved the HistoryURlProvider's initialization
> last in the AutocompleteController's constructor. However, the zero suggest and
> clipboard URL providers take a reference of the history URL provider when
> initialized. To avoid setting these references to NULL, this CL moves the
> history URL provider's initialization before the other two's but keeps
> it after the search provider's for the reasons described in the other CL.
> 
> Bug: 932257
> Change-Id: I1085ad89e13f29ea29264ce3fa1a2d33197f3b17
> Reviewed-on: https://chromium-review.googlesource.com/c/1474383
> Reviewed-by: Mark Pearson <mpearson@chromium.org>
> Commit-Queue: manuk hovanesian <manukh@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#632682}

Bug: 932257
Change-Id: If1dd04d838bc78f8f323280b4023fea551a80b3a
Reviewed-on: https://chromium-review.googlesource.com/c/1475249
Reviewed-by: manuk hovanesian <manukh@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#632793}
1 file changed