Commit 22fa94fe authored by clamy's avatar clamy Committed by Commit bot

PlzNavigate: Properly call WebContents::DidStartNavigatingToPendingEntry

This CL makes it so that NavigatorImpl::Navigate will call
DidStartNavigatingToPendingEntry after creating the NaviagtionRequest when
PlzNavigate is enabled. This makes the test
NavigationControllerTest.ReloadOriginalRequestURL pass with PlzNavigate
enabled.

BUG=439423

Review URL: https://codereview.chromium.org/1035953002

Cr-Commit-Position: refs/heads/master@{#322543}
parent 08f4e666
...@@ -277,6 +277,11 @@ bool NavigatorImpl::NavigateToEntry( ...@@ -277,6 +277,11 @@ bool NavigatorImpl::NavigateToEntry(
navigation_data_.reset(new NavigationMetricsData( navigation_data_.reset(new NavigationMetricsData(
navigation_start, entry.GetURL(), entry.restore_type())); navigation_start, entry.GetURL(), entry.restore_type()));
RequestNavigation(frame_tree_node, entry, reload_type, navigation_start); RequestNavigation(frame_tree_node, entry, reload_type, navigation_start);
// Notify observers about navigation.
if (delegate_)
delegate_->DidStartNavigationToPendingEntry(entry.GetURL(), reload_type);
return true; return true;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment