PlzNavigate: Invoke didStartProvisionalLoad() when the renderer initiates a navigation in startLoad(

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 3457a0fe969b0ebd384f2bd29725fd824ca3cdc7..83970ec108ca4250b9734d01d321ee3aa9eeda9e 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3448,7 +3448,9 @@ void RenderFrameImpl::didCreateDataSource(blink::WebLocalFrame* frame,
           content_initiated));
 }
 
-void RenderFrameImpl::didStartProvisionalLoad(blink::WebLocalFrame* frame) {
+void RenderFrameImpl::didStartProvisionalLoad(
+    blink::WebLocalFrame* frame,
+    const blink::WebURLRequest& request) {
   DCHECK_EQ(frame_, frame);
   WebDataSource* ds = frame->provisionalDataSource();
 
@@ -6190,6 +6192,8 @@ void RenderFrameImpl::BeginNavigation(const NavigationPolicyInfo& info) {
   CHECK(IsBrowserSideNavigationEnabled());
   browser_side_navigation_pending_ = true;
 
+  didStartProvisionalLoad(frame_, info.urlRequest);

[Nate Chapin, 2017/01/31 22:13:32]

Instead of adding a new path for calling didStartProvisionalLoad(), could we
move it earlier in the navigation start pipeline, so the PlzNavigate and
non-PlzNavigate paths use the same call? I think moving it to the start of
FrameLoader::startLoad() would work.

[ananta, 2017/02/02 01:29:23]

Done.

+
   // Note: At this stage, the goal is to apply all the modifications the
   // renderer wants to make to the request, and then send it to the browser, so
   // that the actual network request can be started. Ideally, all such