| Index: content/browser/frame_host/render_frame_host_manager.cc
|
| diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
|
| index 6fcb94122deb087aa949dec2e8299f08e17b58d1..6776ed09e09ce6877268b8470ee05fd64e85ab3b 100644
|
| --- a/content/browser/frame_host/render_frame_host_manager.cc
|
| +++ b/content/browser/frame_host/render_frame_host_manager.cc
|
| @@ -15,7 +15,6 @@
|
| #include "content/browser/frame_host/cross_site_transferring_request.h"
|
| #include "content/browser/frame_host/debug_urls.h"
|
| #include "content/browser/frame_host/interstitial_page_impl.h"
|
| -#include "content/browser/frame_host/navigation_before_commit_info.h"
|
| #include "content/browser/frame_host/navigation_controller_impl.h"
|
| #include "content/browser/frame_host/navigation_entry_impl.h"
|
| #include "content/browser/frame_host/navigation_request.h"
|
| @@ -36,6 +35,7 @@
|
| #include "content/public/browser/notification_types.h"
|
| #include "content/public/browser/render_widget_host_iterator.h"
|
| #include "content/public/browser/render_widget_host_view.h"
|
| +#include "content/public/browser/stream_handle.h"
|
| #include "content/public/browser/user_metrics.h"
|
| #include "content/public/browser/web_ui_controller.h"
|
| #include "content/public/common/content_switches.h"
|
| @@ -106,16 +106,6 @@ RenderFrameHostManager::~RenderFrameHostManager() {
|
|
|
| // Delete any swapped out RenderFrameHosts.
|
| STLDeleteValues(&proxy_hosts_);
|
| -
|
| - // PlzNavigate
|
| - // There is an active navigation request for this RFHM so it needs to be
|
| - // canceled.
|
| - if (CommandLine::ForCurrentProcess()->HasSwitch(
|
| - switches::kEnableBrowserSideNavigation)) {
|
| - if (navigation_request_.get())
|
| - navigation_request_->CancelNavigation();
|
| - }
|
| -
|
| }
|
|
|
| void RenderFrameHostManager::Init(BrowserContext* browser_context,
|
| @@ -596,39 +586,33 @@ void RenderFrameHostManager::OnBeginNavigation(
|
| info.parent_is_main_frame = !frame_tree_node_->parent() ?
|
| false : frame_tree_node_->parent()->IsMainFrame();
|
|
|
| + NavigationControllerImpl& controller =
|
| + delegate_->GetControllerForRenderManager();
|
| + BrowserContext* browser_context = controller.GetBrowserContext();
|
| +
|
| // TODO(clamy): Check if the current RFH should be initialized (in case it has
|
| // crashed) not to display a sad tab while navigating.
|
| // TODO(clamy): Spawn a speculative renderer process if we do not have one to
|
| // use for the navigation.
|
|
|
| - // If there is an ongoing request it must be canceled.
|
| - if (navigation_request_.get())
|
| - navigation_request_->CancelNavigation();
|
| -
|
| navigation_request_.reset(new NavigationRequest(
|
| - info, frame_tree_node_->frame_tree_node_id()));
|
| + info, browser_context, frame_tree_node_));
|
| navigation_request_->BeginNavigation(params.request_body);
|
| }
|
|
|
| // PlzNavigate
|
| -void RenderFrameHostManager::CommitNavigation(
|
| - const NavigationBeforeCommitInfo& info) {
|
| +void RenderFrameHostManager::CommitNavigation(const GURL& url,
|
| + ResourceResponse* response,
|
| + scoped_ptr<StreamHandle> body) {
|
| CHECK(CommandLine::ForCurrentProcess()->HasSwitch(
|
| switches::kEnableBrowserSideNavigation));
|
| - DCHECK(navigation_request_.get());
|
| - // Ignores navigation commits if the request ID doesn't match the current
|
| - // active request.
|
| - if (navigation_request_->navigation_request_id() !=
|
| - info.navigation_request_id) {
|
| - return;
|
| - }
|
|
|
| // Pick the right RenderFrameHost to commit the navigation.
|
| SiteInstance* current_instance = render_frame_host_->GetSiteInstance();
|
| // TODO(clamy): Replace the default values by the right ones. This may require
|
| // some storing in RequestNavigation.
|
| SiteInstance* new_instance = GetSiteInstanceForNavigation(
|
| - info.navigation_url,
|
| + url,
|
| NULL,
|
| navigation_request_->info().navigation_params.transition_type,
|
| false,
|
| @@ -663,7 +647,7 @@ void RenderFrameHostManager::CommitNavigation(
|
| }
|
|
|
| frame_tree_node_->navigator()->CommitNavigation(
|
| - render_frame_host_.get(), info);
|
| + render_frame_host_.get(), url, response, body.Pass());
|
| }
|
|
|
| void RenderFrameHostManager::Observe(
|
|
|