GitLab

When S13nServiceWorker is enabled but NetworkService is disabled,
NavigationURLLoaderImpl actually uses URLRequest when a navigation
is not intercepted. In that case we have a NavigationData and we can
populate it to NavigationHandle. Since some features like Previews
use NavigationData to store their data, it would be better to
populate it if possible.

This fixes following browser_tests:
- PreviewsNoScriptBrowserTest.NoScriptPreviewsEnabled
- PreviewsNoScriptBrowserTest.NoScriptPreviewsEnabledHttpRedirectToHttps

Bug: 866367
Change-Id: Idc85d272772be81dddde9e9e0bdf76f4fadfef8a
Reviewed-on: https://chromium-review.googlesource.com/1148094Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578963}