PlzNavigate: preserve SourceLocation when navigating

third_party/WebKit/Source/web/LocalFrameClientImpl.cpp

Index: third_party/WebKit/Source/web/LocalFrameClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/LocalFrameClientImpl.cpp b/third_party/WebKit/Source/web/LocalFrameClientImpl.cpp
index 73c1b65eae7e909e3c67d43e77b7aad96d22c5d4..348f30c675909d4dcad1716f70fcb89cfa298474 100644
--- a/third_party/WebKit/Source/web/LocalFrameClientImpl.cpp
+++ b/third_party/WebKit/Source/web/LocalFrameClientImpl.cpp
@@ -571,6 +571,17 @@ NavigationPolicy LocalFrameClientImpl::decidePolicyForNavigation(
   if (form)
     navigationInfo.form = WebFormElement(form);
 
+  std::unique_ptr<SourceLocation> sourceLocation =
+      SourceLocation::capture(m_webFrame->frame()->document());
+  if (sourceLocation && !sourceLocation->isUnknown()) {
+    navigationInfo.hasSourceLocation = true;

[Nate Chapin, 2017/03/01 23:22:19]

Rather than have hasSourceLocation, could we either have
WebSourceLocation::isUnknown(), or just check WebSourceLocation.url.isNull()?

[clamy, 2017/03/02 13:34:32]

Done.

+    navigationInfo.sourceLocation.url = sourceLocation->url();
+    navigationInfo.sourceLocation.lineNumber = sourceLocation->lineNumber();
+    navigationInfo.sourceLocation.columnNumber = sourceLocation->columnNumber();
+  } else {
+    navigationInfo.hasSourceLocation = false;
+  }
+
   WebNavigationPolicy webPolicy =
       m_webFrame->client()->decidePolicyForNavigation(navigationInfo);
   return static_cast<NavigationPolicy>(webPolicy);