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

chrome/browser/net/net_error_tab_helper.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 "chrome/browser/net/net_error_tab_helper.h"
 
 #include "base/bind.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/io_thread.h"
@@ skipping 58 matching lines @@
 }  // namespace
 
 NetErrorTabHelper::~NetErrorTabHelper() {
 }
 
 // static
 void NetErrorTabHelper::set_state_for_testing(TestingState state) {
   testing_state_ = state;
 }
 
+void NetErrorTabHelper::NotifyBrowserReloading() {
+  // Record that we're doing a browser side reload on the error page.
+  if (is_error_page_)
+    chrome_common_net::RecordEvent(
+        chrome_common_net::BROWSER_RELOAD_BUTTON_CLICKED_EVENT);

[mmenke, 2014/04/11 19:42:32]

+braces.

[mmenke, 2014/04/11 19:42:32]

Does this record F5s?  What about right clicking and selecting reload?  Do we
care?  Not really sure what the purpose is here.

[Randy Smith (Not in Mondays), 2014/04/14 22:02:13]

Whoops :-{.  Done.

[Randy Smith (Not in Mondays), 2014/04/14 22:02:13]

First, the facts: It captures F5, but not right click->selecting reload, I
presume because F5 goes through the browser and right click through the
renderer.

Then, the intent: My purpose is to figure out how often people use the reload
button on the error page versus "other" ways of reloading (loosely, gathering
data to see if we might want to nuke the reload button at some point.  

Ideally, I'd like to break out the different other ways to reload so that we
could understand what different people's "reload" instincts are.  But I don't
want to do more browser plumbing.  I consider it important to catch the right
click case, so I went digging, and it looks like the simplest way to get what I
want is to go with your suggestion of a PS or two back to hook
DidStartNavigationToPendingEntry(), and just dump all browser initiated
navigations in the same bin.  Done :-}.

+}
+
 void NetErrorTabHelper::DidStartProvisionalLoadForFrame(
     int64 frame_id,
     int64 parent_frame_id,
     bool is_main_frame,
     const GURL& validated_url,
     bool is_error_page,
     bool is_iframe_srcdoc,
     RenderViewHost* render_view_host) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
@@ skipping 129 matching lines @@
   DVLOG(1) << "Sending status " << DnsProbeStatusToString(dns_probe_status_);
   content::RenderFrameHost* rfh = web_contents()->GetMainFrame();
   rfh->Send(new ChromeViewMsg_NetErrorInfo(rfh->GetRoutingID(),
                                            dns_probe_status_));
 
   if (!dns_probe_status_snoop_callback_.is_null())
     dns_probe_status_snoop_callback_.Run(dns_probe_status_);
 }
 
 }  // namespace chrome_browser_net