commit ea99ea1dc01c5bbf058c18abe50008147e5b7ae9
author clamy <clamy@chromium.org> Mon May 28 13:54:23 2018
committer Commit Bot <commit-bot@chromium.org> Mon May 28 13:54:23 2018
tree 14d733a30c842f9d1d7f516de19f2cbb41c7493b
parent 6cf25da95bee042ec69811e16116fdadb0ea600d

Move NavigationRequest creation to NavigationController: 2/3

This CL moves the NavigationRequest creation to the NavigationController. The
Navigator will now take a NavigationRequest as an argument to Navigate instead
of the pending NavigationEntry.

Summary of the changes:
1) Navigator
 - NavigatorImpl::NavigateToEntry and NavigatorImpl::NavigateToPendingEntry are
   removed in favor of Navigate which takes as argument a NavigationRequest. The
   NavigationController is now responsible for creating the NavigationRequest and
   passing it to Navigator::Navigate to have the navigation start.
 - NavigatorImpl::RequestNavigation, which currently creates the
   NavigationRequest is removed. Instead, the NavigationRequest is created in
   NavigationControllerImpl::CreateNavigationRequest.

2) NavigationController
 - HandleRendererDebugURL is called from NavigateWithoutEntry and handles
   navigation to a renderer debug URL. Navigations to a renderer debug URL do not
   require a NavigationRequest. They cannot commit, hence why we can't have a
   committed NavigationEntry for a renderer debug URL.

The next CLs will cleanup the internals of NavigationController.

Bug: 803365
Change-Id: I8b01384324ccfdca451baf1e645ac7d0869e8d47
Reviewed-on: https://chromium-review.googlesource.com/957735
Commit-Queue: Camille Lamy <clamy@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#562250}