GitLab

This is a reland of 314cbf4c

Patchset 1 is the original change.

The main difference is that now the TabPersistentStore is created
with the actual index that TabWindowManagerImpl assigned to the
TabModelSelector, rather than the index that was "requested".

Other changes are to adapt tests.

Original change's description:
> Create glue-layer TabModelOrchestrators
>
> TabModelSelectorImpl does not create or destroy TabPersistentStore
> anymore. Instead, TabModelOrchestrators manage the lifetime of TabModelSelectorImpl and
> TabPersistentStore.
>
> Give TabModelSelectorImpl a Supplier<TabPersistentStore> instead of
> passing TabPersistentStore directly.
>
> This is a step to break the two-way dependency between
> TabModelSelectorImpl and TabPersistentStore.
>
> Bug: 1138561
> Change-Id: Ie2fa4a2735e509ff816b3e2cf5f8f6fffda7999c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2572639
> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
> Reviewed-by: David Trainor <dtrainor@chromium.org>
> Reviewed-by: Ella Ge <eirage@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#835819}

Bug: 1138561,1158259
Change-Id: Iac6eaaed149e853536813cd627d28236638657de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590427
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ella Ge <eirage@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843752}