Change how NavigationController reloads transient entries

content/browser/web_contents/navigation_controller_impl.cc

Index: content/browser/web_contents/navigation_controller_impl.cc
===================================================================
--- content/browser/web_contents/navigation_controller_impl.cc	(revision 153076)
+++ content/browser/web_contents/navigation_controller_impl.cc	(working copy)
@@ -248,9 +248,18 @@
 
 void NavigationControllerImpl::ReloadInternal(bool check_for_repost,
                                               ReloadType reload_type) {
-  // Reloading a transient entry does nothing.
-  if (transient_entry_index_ != -1)
+  if (transient_entry_index_ != -1) {
+    // Interstitials are reloaded by navigating back to their URLs.

[Charlie Reis, 2012/08/27 17:25:35]

Maybe clarify this as "If an interstitial is showing, treat a reload as a
navigation to the transient entry's URL."  (I got confused at first, thinking
the interstitial's URL was the data URL for the warning page.)

[mmenke, 2012/08/27 17:37:03]

Done.

+    content::NavigationEntryImpl* active_entry =
+        NavigationEntryImpl::FromNavigationEntry(GetActiveEntry());
+    if (!active_entry)
+      return;
+    LoadURL(active_entry->GetURL(),
+            content::Referrer(),
+            content::PAGE_TRANSITION_RELOAD,
+            active_entry->extra_headers());
     return;
+  }
 
   DiscardNonCommittedEntriesInternal();
   int current_index = GetCurrentEntryIndex();