Support auto-reload on errors.

chrome/renderer/net/net_error_helper.cc

Index: chrome/renderer/net/net_error_helper.cc
diff --git a/chrome/renderer/net/net_error_helper.cc b/chrome/renderer/net/net_error_helper.cc
index af615f7243d3527f17163657c1a88c78c17352ff..90ca25986c2bb3b9bf245ae023adf91e89a52916 100644
--- a/chrome/renderer/net/net_error_helper.cc
+++ b/chrome/renderer/net/net_error_helper.cc
@@ -13,6 +13,7 @@
 #include "chrome/common/localized_error.h"
 #include "chrome/common/net/net_error_info.h"
 #include "chrome/common/render_messages.h"
+#include "chrome/renderer/net/mockable_one_shot_timer.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/renderer/content_renderer_client.h"
@@ -64,10 +65,12 @@ NetErrorHelperCore::FrameType GetFrameType(const blink::WebFrame* frame) {
 
 }  // namespace
 
-NetErrorHelper::NetErrorHelper(RenderFrame* render_view)
+NetErrorHelper::NetErrorHelper(RenderFrame* render_view,
+                               bool auto_reload_enabled)
     : RenderFrameObserver(render_view),
       content::RenderFrameObserverTracker<NetErrorHelper>(render_view),
       core_(this) {
+  core_.set_auto_reload_enabled(auto_reload_enabled);

[Randy Smith (Not in Mondays), 2014/02/06 16:47:16]

Why not as part of the constructor?

 }
 
 NetErrorHelper::~NetErrorHelper() {
@@ -219,3 +222,15 @@ void NetErrorHelper::OnAlternateErrorPageRetrieved(
     core_.OnAlternateErrorPageFetched("");
   }
 }
+
+void NetErrorHelper::ReloadPage() {
+  render_frame()->GetWebFrame()->reload(false);
+}
+
+MockableOneShotTimer* NetErrorHelper::NewMockableOneShotTimer() {
+  return new MockableOneShotTimer();
+}
+
+void NetErrorHelper::NetworkStateChanged(bool enabled) {
+  core_.NetworkStateChanged(enabled);

[Randy Smith (Not in Mondays), 2014/02/06 16:47:16]

Could you make NetErrorHelperCore a RenderProcessObserver directly?  I think
that wouldn't negatively affect testing any, and would mean you could avoid this
delegation.  (If Matt disagrees as to the cleanliness of this suggestion, he
wins :-}.)

+}