[DO NOT COMMIT] Test implementation of converting same URL navigation into a reload

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 44334dea3167a5799caeb140e8922cae6042e5f5..2ab9c43d48c0f6ef8b69c246623a0f967e511587 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -265,6 +265,9 @@ RenderFrameHostImpl::RenderFrameHostImpl(SiteInstance* site_instance,
       last_navigation_lofi_state_(LOFI_UNSPECIFIED),
       frame_host_binding_(this),
       weak_ptr_factory_(this) {
+  LOG(ERROR) << "RFH[" << this << "]::RFH: "
+    << " routing_id:" << routing_id
+    << " instance:" << site_instance->GetId() << ":" << site_instance->GetSiteURL();
   frame_tree_->AddRenderViewHostRef(render_view_host_);
   GetProcess()->AddRoute(routing_id_, this);
   g_routing_id_frame_map.Get().insert(std::make_pair(
@@ -355,6 +358,8 @@ RenderFrameHostImpl::~RenderFrameHostImpl() {
   // Notify the FrameTree that this RFH is going away, allowing it to shut down
   // the corresponding RenderViewHost if it is no longer needed.
   frame_tree_->ReleaseRenderViewHostRef(render_view_host_);
+
+  LOG(ERROR) << "RFH[" << this << "]::~RFH";
 }
 
 int RenderFrameHostImpl::GetRoutingID() {
@@ -1002,12 +1007,17 @@ void RenderFrameHostImpl::OnDocumentOnLoadCompleted(
 void RenderFrameHostImpl::OnDidStartProvisionalLoad(
     const GURL& url,
     const base::TimeTicks& navigation_start) {
+  LOG(ERROR) << "RFH[" << this << "]::DidStartProvisionalLoad: "
+    << " url:" << url.spec().substr(0, 250);
   frame_tree_node_->navigator()->DidStartProvisionalLoad(this, url,
                                                          navigation_start);
 }
 
 void RenderFrameHostImpl::OnDidFailProvisionalLoadWithError(
     const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) {
+  LOG(ERROR) << "RFH[" << this << "]::DidFailProvisionalLoadWithError: "
+    << " error:" << params.error_code;
+
   // TODO(clamy): Kill the renderer with RFH_FAIL_PROVISIONAL_LOAD_NO_HANDLE and
   // return early if navigation_handle_ is null, once we prevent that case from
   // happening in practice.
@@ -1027,6 +1037,10 @@ void RenderFrameHostImpl::OnDidFailLoadWithError(
     const base::string16& error_description,
     bool was_ignored_by_handler) {
   GURL validated_url(url);
+  LOG(ERROR) << "RFH[" << this << "]::DidFailLoadWithError: "
+    << " url:" << url.spec().substr(0, 250)
+    << " error:" << error_code;
+
   GetProcess()->FilterURL(false, &validated_url);
 
   frame_tree_node_->navigator()->DidFailLoadWithError(
@@ -1054,11 +1068,16 @@ void RenderFrameHostImpl::OnDidCommitProvisionalLoad(const IPC::Message& msg) {
       Read(&msg, &iter, &validated_params)) {
     bad_message::ReceivedBadMessage(
         process, bad_message::RFH_COMMIT_DESERIALIZATION_FAILED);
+    LOG(ERROR) << "RFH[" << this << "]::DidCommitProvisionalLoad: "
+      << " failed to read IPC, killed renderer";
     return;
   }
   TRACE_EVENT1("navigation", "RenderFrameHostImpl::OnDidCommitProvisionalLoad",
                "url", validated_params.url.possibly_invalid_spec());
 
+  LOG(ERROR) << "RFH[" << this << "]::DidCommitProvisionalLoad: "
+    << " url:" << validated_params.url.spec().substr(0, 250);
+
   // Sanity-check the page transition for frame type.
   DCHECK_EQ(ui::PageTransitionIsMainFrame(validated_params.transition),
             !GetParent());
@@ -1297,6 +1316,7 @@ void RenderFrameHostImpl::SwapOut(
   // The trace event may not end properly if the ACK times out.  We expect this
   // to be fixed when RenderViewHostImpl::OnSwapOut moves to RenderFrameHost.
   TRACE_EVENT_ASYNC_BEGIN0("navigation", "RenderFrameHostImpl::SwapOut", this);
+  LOG(ERROR) << "RFH[" << this << "]::SwapOut";
 
   // If this RenderFrameHost is already pending deletion, it must have already
   // gone through this, therefore just return.
@@ -1436,6 +1456,7 @@ bool RenderFrameHostImpl::IsWaitingForUnloadACK() const {
 }
 
 void RenderFrameHostImpl::OnSwapOutACK() {
+  LOG(ERROR) << "RFH[" << this << "]::SwapOutACK";
   OnSwappedOut();
 }
 
@@ -2227,6 +2248,9 @@ void RenderFrameHostImpl::Navigate(
   TRACE_EVENT0("navigation", "RenderFrameHostImpl::Navigate");
   DCHECK(!IsBrowserSideNavigationEnabled());
 
+  LOG(ERROR) << "RFH[" << this << "]::Navigate: "
+    << " url:" << common_params.url.spec().substr(0, 250);
+
   UpdatePermissionsForNavigation(common_params, request_params);
 
   // Only send the message if we aren't suspended at the start of a cross-site
@@ -2276,6 +2300,9 @@ void RenderFrameHostImpl::NavigateToInterstitialURL(const GURL& data_url) {
 void RenderFrameHostImpl::OpenURL(const FrameHostMsg_OpenURL_Params& params,
                                   SiteInstance* source_site_instance) {
   GURL validated_url(params.url);
+
+  LOG(ERROR) << "RFH[" << this << "]::OpenURL: "
+    << " url:" << validated_url.spec().substr(0, 250);
   GetProcess()->FilterURL(false, &validated_url);
 
   TRACE_EVENT1("navigation", "RenderFrameHostImpl::OpenURL", "url",
@@ -2422,6 +2449,9 @@ void RenderFrameHostImpl::CommitNavigation(
     const CommonNavigationParams& common_params,
     const RequestNavigationParams& request_params,
     bool is_view_source) {
+  LOG(ERROR) << "RFH[" << this << "]::CommitNavigation: "
+    << " url:" << common_params.url.spec().substr(0, 250);
+
   DCHECK((response && body.get()) ||
           !ShouldMakeNetworkRequestForURL(common_params.url));
   UpdatePermissionsForNavigation(common_params, request_params);
@@ -2467,6 +2497,8 @@ void RenderFrameHostImpl::FailedNavigation(
     const RequestNavigationParams& request_params,
     bool has_stale_copy_in_cache,
     int error_code) {
+  LOG(ERROR) << "RFH[" << this << "]::FailedNavigation: "
+    << " url:" << common_params.url.spec().substr(0, 250);
   // Get back to a clean state, in case a new navigation started without
   // completing an unload handler.
   ResetWaitingState();