AU: Retry up to 3 times 30 seconds apart on HTTP response code 0.

libcurl_http_fetcher.h

Index: libcurl_http_fetcher.h
diff --git a/libcurl_http_fetcher.h b/libcurl_http_fetcher.h
index c41804e38703744bd3b0a594c1c77541ed972897..253ba749982c891947efe9c90d21c02171726424 100644
--- a/libcurl_http_fetcher.h
+++ b/libcurl_http_fetcher.h
@@ -33,6 +33,8 @@ class LibcurlHttpFetcher : public HttpFetcher {
         resume_offset_(0),
         retry_count_(0),
         retry_seconds_(60),
+        no_network_retry_count_(0),
+        no_network_max_retries_(0),
         idle_seconds_(1),
         force_connection_type_(false),
         forced_expensive_connection_(false),
@@ -74,6 +76,10 @@ class LibcurlHttpFetcher : public HttpFetcher {
   // Sets the retry timeout. Useful for testing.
   void set_retry_seconds(int seconds) { retry_seconds_ = seconds; }
 
+  void set_no_network_max_retries(int retries) {
+    no_network_max_retries_ = retries;
+  }
+
   void SetConnectionAsExpensive(bool is_expensive) {
     force_connection_type_ = true;
     forced_expensive_connection_ = is_expensive;
@@ -185,6 +191,10 @@ class LibcurlHttpFetcher : public HttpFetcher {
   // Seconds to wait before retrying a resume.
   int retry_seconds_;
 
+  // Number of resumes due to HTTP response code 0.

[adlr, 2010/11/29 21:23:28]

... due to no network (i.e., HTTP response code 0).

This comment helps make it clear that "no" in the var names is the word "no" and
not the abbreviation for "number."

[petkov, 2010/11/29 21:28:19]

Done.

+  int no_network_retry_count_;
+  int no_network_max_retries_;
+
   // Seconds to wait before asking libcurl to "perform".
   int idle_seconds_;
 
@@ -200,7 +210,7 @@ class LibcurlHttpFetcher : public HttpFetcher {
 
   // If true, we are currently performing a write callback on the delegate.
   bool in_write_callback_;
-  
+
   // If true, we have returned at least one byte in the write callback
   // to the delegate.
   bool sent_byte_;