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

content/browser/frame_host/navigation_request.cc

Index: content/browser/frame_host/navigation_request.cc
diff --git a/content/browser/frame_host/navigation_request.cc b/content/browser/frame_host/navigation_request.cc
index df0f7a1e7b7128331b36debd5882b31dd4f37bd5..55f936ab92e8c0dc04a1e92faebbcf33c3324d84 100644
--- a/content/browser/frame_host/navigation_request.cc
+++ b/content/browser/frame_host/navigation_request.cc
@@ -227,8 +227,13 @@ void NavigationRequest::BeginNavigation() {
 }
 
 void NavigationRequest::CreateNavigationHandle() {
-  navigation_handle_ = NavigationHandleImpl::Create(
-      common_params_.url, frame_tree_node_, common_params_.navigation_start);
+  // TODO(nasko): Update the NavigationHandle creation to ensure that the
+  // proper values are specified for is_synchronous and is_srcdoc.

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

Will the new tests fail for PlzNavigate in the meantime?  I'm wondering if they
need to be disabled or how hard it would be to fix them here.

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

The one failing is srcdoc, which I'm not sure is working properly in PlzNavigate
currently. The sync one should work, as the NavigationHandle isn't created
through this method, but rather at the DidCommitProvisionalLoad call, as there
is no resource to be loaded.

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

Great.  Do we need to add the srcdoc one to their test filter so we don't turn
them red?  (Just trying to be proactive, since we've been in those shoes.)

[nasko, 2016/02/05 21:40:21]

Done.

+  navigation_handle_ =
+      NavigationHandleImpl::Create(common_params_.url, frame_tree_node_,
+                                   false,  // is_synchronous
+                                   false,  // is_srcdoc
+                                   common_params_.navigation_start);
 }
 
 void NavigationRequest::TransferNavigationHandleOwnership(