If the renderer drops a navigation, make sure the spinner doesn't spin endlessly.

content/renderer/render_view_browsertest.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 "base/basictypes.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/memory/shared_memory.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 846 matching lines @@
   params_B.commit_params.page_state =
       state_A;  // Doesn't matter, just has to be present.
   params_B.commit_params.browser_navigation_start =
       base::TimeTicks::FromInternalValue(1);
   frame()->OnNavigate(params_B);
 
   // State should be unchanged.
   EXPECT_EQ(2, view()->history_list_length_);
   EXPECT_EQ(1, view()->history_list_offset_);
   EXPECT_EQ(3, view()->history_page_ids_[1]);
+
+  // Check for a valid DidDropNavigation message.
+  ProcessPendingMessages();
+  const IPC::Message* msg = render_thread_->sink().GetUniqueMessageMatching(
+      FrameHostMsg_DidDropNavigation::ID);
+  ASSERT_TRUE(msg);
+  render_thread_->sink().ClearMessages();
 }
 
 // Test that we do not ignore navigations after the entry limit is reached,
 // in which case the browser starts dropping entries from the front.  In this
 // case, we'll see a page_id mismatch but the RenderView's id will be older,
 // not newer, than params.page_id.  Use this as a cue that we should update the
 // state and not treat it like a navigation to a cropped forward history item.
 // See http://crbug.com/89798.
 TEST_F(RenderViewImplTest, DontIgnoreBackAfterNavEntryLimit) {
   // Load page A.
@@ skipping 1659 matching lines @@
 
   gfx::Size initial_size_;
 };
 
 TEST_F(RenderViewImplInitialSizeTest, InitialSize) {
   ASSERT_EQ(initial_size_, view_->GetSize());
   ASSERT_EQ(initial_size_, gfx::Size(view_->GetWebView()->size()));
 }
 
 }  // namespace content