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

components/test_runner/web_frame_test_client.cc

Index: components/test_runner/web_frame_test_client.cc
diff --git a/components/test_runner/web_frame_test_client.cc b/components/test_runner/web_frame_test_client.cc
index ca0aa871cf8c7080239684ef9614ce313809b63e..d52efb6ff2097d9484a6b1dc43afc0af42517c62 100644
--- a/components/test_runner/web_frame_test_client.cc
+++ b/components/test_runner/web_frame_test_client.cc
@@ -29,6 +29,7 @@
 #include "third_party/WebKit/public/platform/WebURLRequest.h"
 #include "third_party/WebKit/public/platform/WebURLResponse.h"
 #include "third_party/WebKit/public/web/WebConsoleMessage.h"
+#include "third_party/WebKit/public/web/WebDataSource.h"
 #include "third_party/WebKit/public/web/WebElement.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
@@ -412,7 +413,15 @@ void WebFrameTestClient::loadErrorPage(int reason) {
   }
 }
 
-void WebFrameTestClient::didStartProvisionalLoad(blink::WebLocalFrame* frame) {
+void WebFrameTestClient::didStartProvisionalLoad(
+    blink::WebLocalFrame* frame,

[Nate Chapin, 2017/02/02 19:16:48]

Could we use web_frame_test_proxy_base_->web_frame() below and get rid of the
Frame* parameter? RenderFrameImpl::didStartProvisionalLoad should just be able
to use RenderFrameImpl::frame_

[ananta, 2017/02/02 23:54:02]

Done.

+    blink::WebDataSource* data_source) {
+  // PlzNavigate
+  // A provisional load notification is received when a frame navigation is
+  // sent to the browser. We don't want to log it again during commit.
+  if (delegate_->IsNavigationInitiatedByRenderer(data_source->getRequest()))
+    return;
+
   test_runner()->tryToSetTopLoadingFrame(frame);
 
   if (test_runner()->shouldDumpFrameLoadCallbacks()) {