Surface button for loading stale cache copy on net error page.

chrome/browser/ui/browser_commands.cc

Index: chrome/browser/ui/browser_commands.cc
diff --git a/chrome/browser/ui/browser_commands.cc b/chrome/browser/ui/browser_commands.cc
index 94d132ee620cffe22210800a049917dba702d94c..814fdf6d2c40fca3cf3bae25e10c3a4891bc5c4e 100644
--- a/chrome/browser/ui/browser_commands.cc
+++ b/chrome/browser/ui/browser_commands.cc
@@ -24,6 +24,7 @@
 #include "chrome/browser/favicon/favicon_tab_helper.h"
 #include "chrome/browser/google/google_util.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
+#include "chrome/browser/net/net_error_tab_helper.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/prefs/incognito_mode_prefs.h"
 #include "chrome/browser/profiles/profile.h"
@@ -240,6 +241,13 @@ void ReloadInternal(Browser* browser,
   new_tab->UserGestureDone();
   if (!new_tab->FocusLocationBarByDefault())
     new_tab->GetView()->Focus();
+
+  // Notify error system for statistics.
+  chrome_browser_net::NetErrorTabHelper* net_error_tab_helper =
+      chrome_browser_net::NetErrorTabHelper::FromWebContents(new_tab);
+  if (net_error_tab_helper)
+    net_error_tab_helper->NotifyBrowserReloading();

[mmenke, 2014/04/09 15:59:26]

We can't deduce this from the information passed to
WebContentsObserver::DidStartNavigationToPendingEntry?

[Randy Smith (Not in Mondays), 2014/04/10 21:51:01]

So I stopped here going down the stack because this was the last place where I
could be sure that we really were coming from the browser reload button. 
Codesearch says that NavigationController::Reload* is called all over the place.
 So I didn't think I could with certainty tell that the browser reload button
was being hit if I went below this point in the stack.

+
   if (ignore_cache)
     new_tab->GetController().ReloadIgnoringCache(true);
   else