Avoid stale navigation requests without excessive page id knowledge in the renderer process.

content/public/test/render_view_test.cc

 // Copyright (c) 2012 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/public/test/render_view_test.h"
 
 #include "base/run_loop.h"
 #include "content/common/dom_storage/dom_storage_types.h"
 #include "content/common/frame_messages.h"
 #include "content/common/input_messages.h"
@@ skipping 319 matching lines @@
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
   impl->OnMessageReceived(*input_message);
   return true;
 }
 
 void RenderViewTest::SetFocused(const blink::WebNode& node) {
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
   impl->focusedNodeChanged(node);
 }
 
-void RenderViewTest::ClearHistory() {
-  RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
-  impl->page_id_ = -1;
-  impl->history_list_offset_ = -1;
-  impl->history_list_length_ = 0;
-  impl->history_page_ids_.clear();
-}
-
 void RenderViewTest::Reload(const GURL& url) {
   FrameMsg_Navigate_Params params;
   params.common_params.url = url;
   params.common_params.navigation_type = FrameMsg_Navigate_Type::RELOAD;
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
   impl->GetMainRenderFrame()->OnNavigate(params);
   FrameLoadWaiter(impl->GetMainRenderFrame()).Wait();
 }
 
 uint32 RenderViewTest::GetNavigationIPCType() {
@@ skipping 57 matching lines @@
 
 scoped_ptr<ViewMsg_Resize_Params> RenderViewTest::InitialSizeParams() {
   return make_scoped_ptr(new ViewMsg_Resize_Params());
 }
 
 void RenderViewTest::GoToOffset(int offset, const PageState& state) {
   RenderViewImpl* impl = static_cast<RenderViewImpl*>(view_);
 
   int history_list_length = impl->historyBackListCount() +
                             impl->historyForwardListCount() + 1;
-  int pending_offset = offset + impl->history_list_offset();
+  int pending_offset = offset + impl->history_list_offset_;
 
   FrameMsg_Navigate_Params navigate_params;
   navigate_params.common_params.navigation_type =
       FrameMsg_Navigate_Type::NORMAL;
   navigate_params.common_params.transition = ui::PAGE_TRANSITION_FORWARD_BACK;
   navigate_params.current_history_list_length = history_list_length;
-  navigate_params.current_history_list_offset = impl->history_list_offset();
+  navigate_params.current_history_list_offset = impl->history_list_offset_;
   navigate_params.pending_history_list_offset = pending_offset;
   navigate_params.page_id = impl->page_id_ + offset;
   navigate_params.commit_params.page_state = state;
   navigate_params.request_time = base::Time::Now();
 
   FrameMsg_Navigate navigate_message(impl->GetMainRenderFrame()->GetRoutingID(),
                                      navigate_params);
   impl->GetMainRenderFrame()->OnMessageReceived(navigate_message);
 
   // The load actually happens asynchronously, so we pump messages to process
   // the pending continuation.
   FrameLoadWaiter(view_->GetMainRenderFrame()).Wait();
 }
 
 }  // namespace content