GitLab

Turns out we don't really want to use the TabDelegate that
WebAppActivity uses, as it does asynchronous tab loading, and fires an
intent in order to launch the NTP, instead of just opening it in a tab.

This is slightly complicated by some weird initialization ordering
issues (why does CTA initialize the TabModels in initializeCompositor??)
and that we still want new tab link clicks to open in CCT (to avoid
breaking things that need multiple tabs alive at once, like oauth).

Bug: 984019
Change-Id: Iea98cdc0936b374eb985643af8dbb040feeef946
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742927Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#686457}