PlzNavigate: have renderer-initiated navigations be same-process

content/browser/frame_host/navigator_delegate.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
 
 #include "base/strings/string16.h"
 #include "content/public/browser/invalidate_type.h"
 #include "content/public/browser/navigation_controller.h"
@@ skipping 70 matching lines @@
   virtual void NotifyChangedNavigationState(InvalidateTypes changed_flags) {}
 
   // Notifies the Navigator embedder that it is beginning to navigate a frame.
   virtual void AboutToNavigateRenderFrame(
       RenderFrameHostImpl* old_host,
       RenderFrameHostImpl* new_host) {}
 
   // Notifies the Navigator embedder that a navigation to pending
   // NavigationEntry has started in the browser process.
   virtual void DidStartNavigationToPendingEntry(
-      RenderFrameHostImpl* render_frame_host,

[carlosk, 2015/02/20 19:16:51]

I removed this parameter because a) it was not being used anyway and b) it was
not even available where PlzNavigate needed to make that call.

[clamy, 2015/02/23 10:52:34]

Is this necessary for this patch? If not, I would prefer to have it in a
different CL.

[carlosk, 2015/03/04 19:42:40]

Change was moved to another CL.

       const GURL& url,
       NavigationController::ReloadType reload_type) {}
 
   // Opens a URL with the given parameters. See PageNavigator::OpenURL, which
   // this forwards to.
   virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host,
                               const OpenURLParams& params) {}
 
   // Returns whether URLs for aborted browser-initiated navigations should be
   // preserved in the omnibox.  Defaults to false.
   virtual bool ShouldPreserveAbortedURLs();
 };
 
 }  // namspace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_