Store NavigationEntryImpl data into NavigationRequest for later usage.

content/browser/frame_host/navigation_request.cc

 // Copyright 2014 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.
 
 #include "content/browser/frame_host/navigation_request.h"
 
 #include "content/browser/frame_host/frame_tree_node.h"
+#include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/browser/frame_host/navigation_request_info.h"
 #include "content/browser/frame_host/navigator.h"
 #include "content/browser/loader/navigation_url_loader.h"
+#include "content/browser/site_instance_impl.h"
 #include "content/common/resource_request_body.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/stream_handle.h"
 #include "net/url_request/redirect_info.h"
 
 namespace content {
 
 NavigationRequest::NavigationRequest(
     FrameTreeNode* frame_tree_node,
     const CommonNavigationParams& common_params,
     const CommitNavigationParams& commit_params)
     : frame_tree_node_(frame_tree_node),
       common_params_(common_params),
       commit_params_(commit_params),
-      state_(NOT_STARTED) {
+      state_(NOT_STARTED),
+      is_restore_(false),
+      is_view_source_(false),
+      bindings_(NavigationEntryImpl::kInvalidBindings) {
 }
 
 NavigationRequest::~NavigationRequest() {
 }
 
 void NavigationRequest::BeginNavigation(
     scoped_ptr<NavigationRequestInfo> info,
     scoped_refptr<ResourceRequestBody> request_body) {
   DCHECK(!loader_);
   DCHECK(state_ == NOT_STARTED || state_ == WAITING_FOR_RENDERER_RESPONSE);
   state_ = STARTED;
   loader_ = NavigationURLLoader::Create(
       frame_tree_node_->navigator()->GetController()->GetBrowserContext(),
       frame_tree_node_->frame_tree_node_id(), common_params_, info.Pass(),
       request_body.get(), this);
 
   // TODO(davidben): Fire (and add as necessary) observer methods such as
   // DidStartProvisionalLoadForFrame for the navigation.
 }
 
+void NavigationRequest::CopyDataFrom(const NavigationEntryImpl& nav_entry) {
+  source_site_instance_ = nav_entry.source_site_instance();
+  dest_site_instance_ = nav_entry.site_instance();
+  is_restore_ = nav_entry.restore_type() != NavigationEntryImpl::RESTORE_NONE;

[nasko, 2015/01/16 15:03:24]

Why not store the restore type directly?

[carlosk, 2015/01/19 15:02:52]

The reason was that we only need to know if it is a restore or not, and not the
kind of restore.

But I changed to storing the actual type after discussing with clamy@ that it
might be needed in the future.

+  is_view_source_ = nav_entry.IsViewSourceMode();
+  bindings_ = nav_entry.bindings();
+}
+
 void NavigationRequest::OnRequestRedirected(
     const net::RedirectInfo& redirect_info,
     const scoped_refptr<ResourceResponse>& response) {
   // TODO(davidben): Track other changes from redirects. These are important
   // for, e.g., reloads.
   common_params_.url = redirect_info.new_url;
 
   // TODO(davidben): This where prerender and navigation_interceptor should be
   // integrated. For now, just always follow all redirects.
   loader_->FollowRedirect();
 }
 
 void NavigationRequest::OnResponseStarted(
     const scoped_refptr<ResourceResponse>& response,
     scoped_ptr<StreamHandle> body) {
   DCHECK(state_ == STARTED);
   state_ = RESPONSE_STARTED;
   frame_tree_node_->navigator()->CommitNavigation(frame_tree_node_,
                                                   response.get(), body.Pass());
 }
 
 void NavigationRequest::OnRequestFailed(int net_error) {
   DCHECK(state_ == STARTED);
   state_ = FAILED;
   // TODO(davidben): Network failures should display a network error page.
   NOTIMPLEMENTED();
 }
 
 }  // namespace content