| Index: content/renderer/history_controller.cc
|
| diff --git a/content/renderer/history_controller.cc b/content/renderer/history_controller.cc
|
| index 5de3172d79d57301de45d093d4d6f45d28865a78..ddd187853be623ef5d15238c56eb3b139108b7ce 100644
|
| --- a/content/renderer/history_controller.cc
|
| +++ b/content/renderer/history_controller.cc
|
| @@ -146,6 +146,11 @@ void HistoryController::RecursiveGoToEntry(
|
| new_item.documentSequenceNumber() ==
|
| old_item.documentSequenceNumber()) {
|
| same_document_loads.push_back(std::make_pair(frame, new_item));
|
| +
|
| + // Returning here (and omitting child frames which have also changed) is
|
| + // wrong, but not returning here is worse. See the discussion in
|
| + // NavigationControllerImpl::FindFramesToNavigate for more information.
|
| + return;
|
| } else {
|
| different_document_loads.push_back(std::make_pair(frame, new_item));
|
| // For a different document, the subframes will be destroyed, so there's
|
|
|