Support auto-reload on errors.

chrome/renderer/chrome_content_renderer_client.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/renderer/chrome_content_renderer_client.h"
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
@@ skipping 970 matching lines @@
     return false;
   }
 
   *error_domain = LocalizedError::kHttpErrorDomain;
   return true;
 }
 
 bool ChromeContentRendererClient::ShouldSuppressErrorPage(
     content::RenderFrame* render_frame,
     const GURL& url) {
+  if (render_frame->GetRenderView()->GetMainRenderFrame() == render_frame
+      && NetErrorHelper::Get(render_frame)->ShouldSuppressErrorPage(url))

[mmenke, 2014/03/04 17:26:25]

Optional:  I think it's cleaner not to have the main frame check here.  I don't
think it's the content renderer client's responsibility to know that the
NetErrorHelper only suppresses reloads for the main frame.  Think that
responsibility actually belongs to the NetErrorHelperCore, which is where all
the other main frame checks live.

[Elly Fong-Jones, 2014/03/06 21:48:37]

Done.

+    return true;

[mmenke, 2014/03/04 17:26:25]

nit:  Use braces when either part of an if takes more than one line.

[Elly Fong-Jones, 2014/03/06 21:48:37]

Done.

   // Do not flash an error page if the Instant new tab page fails to load.
   return search_bouncer_.get() && search_bouncer_->IsNewTabPage(url);
 }
 
 void ChromeContentRendererClient::GetNavigationErrorStrings(
     content::RenderView* render_view,
     blink::WebFrame* frame,
     const blink::WebURLRequest& failed_request,
     const blink::WebURLError& error,
     std::string* error_html,
@@ skipping 436 matching lines @@
   CommandLine* command_line = CommandLine::ForCurrentProcess();
   return !command_line->HasSwitch(extensions::switches::kExtensionProcess);
 }
 
 blink::WebWorkerPermissionClientProxy*
 ChromeContentRendererClient::CreateWorkerPermissionClientProxy(
     content::RenderFrame* render_frame,
     blink::WebFrame* frame) {
   return new WorkerPermissionClientProxy(render_frame, frame);
 }