Add methods to NavigationHandle to allow refactoring webNavigation to use it.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index c231724a78d2e4f3352f2f9342a67ed34a5a6370..d293131c59f4ee4f3a43b86f6e223fd9147e8bcf 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -1012,8 +1012,11 @@ void RenderFrameHostImpl::OnDidCommitProvisionalLoad(const IPC::Message& msg) {
   // DidCommitProvisionalLoad IPC without a prior DidStartProvisionalLoad
   // message.
   if (!navigation_handle_) {
-    navigation_handle_ = NavigationHandleImpl::Create(
-        validated_params.url, frame_tree_node_, base::TimeTicks::Now());
+    navigation_handle_ =
+        NavigationHandleImpl::Create(validated_params.url, frame_tree_node_,
+                                     true,   // is_synchronous

[Charlie Reis, 2016/02/04 20:46:04]

Is it safe to conclude this?  Above, we clear the NavHandle if the URL doesn't
match.  What if we're in that case for an async navigation?

[nasko, 2016/02/04 23:36:59]

Based on the comments of the two blocks, it seems that we can only get here for
synchronous navigations. The block resetting the navigation_handle_ is explicit
that it is for cases of sync navigation interrupting an ongoing one. It must be
the case, otherwise we would've gotten DidStartProvisionalLoad, which will
replace the NavigationHandle with a new one and the URLs would match.

[Charlie Reis, 2016/02/05 18:09:40]

Acknowledged.

+                                     false,  // is_srcdoc

[Charlie Reis, 2016/02/04 20:46:04]

Is it possible for srcdoc to come through this path, or is it necessarily not
synchronous?

[nasko, 2016/02/04 23:36:59]

I don't think srcdoc is synchronous. We do get DidStartProvisionalLoad for
those.

[Charlie Reis, 2016/02/05 18:09:41]

Ok, thanks.

+                                     base::TimeTicks::Now());
   }
 
   accessibility_reset_count_ = 0;