Don't update URL bar or SSL icon for pending history navs until they commit.

content/browser/tab_contents/navigation_controller.h

 // Copyright (c) 2011 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.
 
 #ifndef CONTENT_BROWSER_TAB_CONTENTS_NAVIGATION_CONTROLLER_H_
 #define CONTENT_BROWSER_TAB_CONTENTS_NAVIGATION_CONTROLLER_H_
 #pragma once
 
 #include "build/build_config.h"
 
@@ skipping 60 matching lines @@
                bool from_last_session,
                std::vector<NavigationEntry*>* entries);
 
   // Active entry --------------------------------------------------------------
 
   // Returns the active entry, which is the transient entry if any, the pending
   // entry if a navigation is in progress or the last committed entry otherwise.
   // NOTE: This can be NULL!!
   //
   // If you are trying to get the current state of the NavigationController,
   // this is the method you will typically want to call.

[brettw, 2011/08/30 16:15:18]

Can you update this comment to say when you should call GetVisibleEntry instead?
A similar comment on GetVisibleEntry might also be nice.

[Charlie Reis, 2011/08/30 17:04:58]

Sure-- fixed.

   NavigationEntry* GetActiveEntry() const;
 
+  // Returns the same entry as GetActiveEntry, except that it ignores pending
+  // history navigation entries.  When the user clicks back or forward, the
+  // location bar and other indicators should not update until the pending entry
+  // commits.
+  NavigationEntry* GetVisibleEntry() const;
+
   // Returns the index from which we would go back/forward or reload.  This is
   // the last_committed_entry_index_ if pending_entry_index_ is -1.  Otherwise,
   // it is the pending_entry_index_.
   int GetCurrentEntryIndex() const;
 
   // Returns the last committed entry, which may be null if there are no
   // committed entries.
   NavigationEntry* GetLastCommittedEntry() const;
 
   // Returns true if the source for the current entry can be viewed.
@@ skipping 376 matching lines @@
   static size_t max_entry_count_;
 
   // If a repost is pending, its type (RELOAD or RELOAD_IGNORING_CACHE),
   // NO_RELOAD otherwise.
   ReloadType pending_reload_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationController);
 };
 
 #endif  // CONTENT_BROWSER_TAB_CONTENTS_NAVIGATION_CONTROLLER_H_