GitLab

When Navigate() calls ShowSingletonTabOverwritingNTP(), the std::move(*params) is executed before params->browser is passed as an argument.
This becomes a problem when we use BackupRefPtr for NavigateParams::browser. Because BackupRefPtr will clear when moved so ShowSingletonTabOverwritingNTP will receive a nullptr.

Bug: 1080832
Change-Id: Id26f41e128b9c79d2506ac82e3655d1fdb58f1b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2567279Reviewed-by: Bartek Nowierski <bartekn@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Keishi Hattori <keishi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834109}