[OfflinePages] Classifies PRERENDERING_FAILED cases whether to TryNext

components/offline_pages/background/offliner.h

Index: components/offline_pages/background/offliner.h
diff --git a/components/offline_pages/background/offliner.h b/components/offline_pages/background/offliner.h
index cafd068634dde132543e8b599ecf2bd15dd56b02..7238de9d3d3510d9565815f0439c304315b70583 100644
--- a/components/offline_pages/background/offliner.h
+++ b/components/offline_pages/background/offliner.h
@@ -44,6 +44,9 @@ class Offliner {
     PRERENDERING_NOT_STARTED = 9,
     // Prerendering failed with hard error so should not retry the request.
     PRERENDERING_FAILED_NO_RETRY = 10,
+    // Prerendering failed with some error that suggests we should not continue
+    // processing another request at this time.
+    PRERENDERING_FAILED_NO_NEXT = 11,

[chili, 2016/12/02 23:17:43]

Just a thought as I'm working on the background loader...

Should we rename this to "FAILED_NO_NEXT" to make it less prerendering-specific?

[dougarnett, 2016/12/02 23:50:08]

Or LOADING_FAILED_NO_NEXT?

Pete, WDYT - should we rename the Prerendering terminology here to Loading as
part of this CL?

     // NOTE: insert new values above this line and update histogram enum too.
     STATUS_COUNT
   };