Better remove HistoryNodes

content/renderer/history_controller.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.
 
 /*
  * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  *     (http://www.torchmobile.com/)
  *
@@ skipping 161 matching lines @@
       break;
     case blink::WebInitialCommitInChildFrame:
       UpdateForInitialLoadInChildFrame(frame, item);
       break;
     case blink::WebHistoryInertCommit:
       // Even for inert commits (e.g., location.replace, client redirects), make
       // sure the current entry gets updated, if there is one.
       if (current_entry_) {
         if (HistoryEntry::HistoryNode* node =
                 current_entry_->GetHistoryNodeForFrame(frame)) {
+          if (!navigation_within_page)
+            node->RemoveChildren();
           node->set_item(item);
         }
       }
       break;
     default:
       NOTREACHED() << "Invalid commit type: " << commit_type;
   }
 }
 
 HistoryEntry* HistoryController::GetCurrentEntry() {
@@ skipping 27 matching lines @@
   if (!current_entry_) {
     current_entry_.reset(
         new HistoryEntry(new_item, target_frame->GetRoutingID()));
   } else {
     current_entry_.reset(current_entry_->CloneAndReplace(
         new_item, clone_children_of_target, target_frame, render_view_));
   }
 }
 
 }  // namespace content