commit b00366f14c729a20f11d6d09cf6b5a486caf9d12
author Shakti Sahu <shaktisahu@chromium.org> Wed Aug 14 19:45:06 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Aug 14 19:45:06 2024
tree 111ae8a04501605a059f1bbfe1157e307362f8e4
parent 1f1794f29da58c3cda259353fdf1b8687dbddd2c

Reland "[iOS TG Sync] Implement IsSaveableNavigation for iOS"

This is a reland of commit ecc4f43dd48973da67cab2062e41a102d69ee431

There was a problem with rebase which declared kChromeUINewTabURL
twice. The subsequent patchset fixes that.

Original change's description:
> [iOS TG Sync] Implement IsSaveableNavigation for iOS
>
> This CL adds various conditions to the navigation to determine
> whether a navigation should be synced. This is intended to be a
> mirror of the existing TabGroupSyncUtils::IsSaveableNavigation on
> desktop and android.
>
> The next CL https://chromium-review.googlesource.com/c/chromium/src/+/5788102
>  filters out unsupported schemes from syncing and replaces them
> with an NTP wherever required.
>
> This also required slight change in signature of
> FakeNavigationContext::SetWebState to accept a pointer instead
> of std::unique_ptr.
>
> Bug: 359726089
> Change-Id: Ic41cb3f4a8c5a6d24a4e99cf4b1f955e59e54f0f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5787832
> Reviewed-by: Min Qin <qinmin@chromium.org>
> Reviewed-by: Gauthier Ambard <gambard@chromium.org>
> Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1341761}

Bug: 359726089
Change-Id: I492f543cc9ed99f714de124494eb759943d41ff4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5788088
Reviewed-by: Ali Juma <ajuma@chromium.org>
Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1341875}