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

content/browser/tab_contents/navigation_controller.h

Index: content/browser/tab_contents/navigation_controller.h
diff --git a/content/browser/tab_contents/navigation_controller.h b/content/browser/tab_contents/navigation_controller.h
index a01262d5e33548742656605fccb0e597d683e4b4..a37d09a761a724dcbc9c4fab79e5bbaedfba70cb 100644
--- a/content/browser/tab_contents/navigation_controller.h
+++ b/content/browser/tab_contents/navigation_controller.h
@@ -81,6 +81,12 @@ class 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_.