Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(562)

Unified Diff: components/offline_pages/core/background/offliner_policy.h

Issue 2650763004: Reduce the "immediate" timeout to 4 min 50 sec to match prerenderer. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/offline_pages/core/background/offliner_policy.h
diff --git a/components/offline_pages/core/background/offliner_policy.h b/components/offline_pages/core/background/offliner_policy.h
index 3a579ec66151e7f1c5a3a882750e6434dea4d1ea..d828d4c6ea6ba5992bad1b020b6c41bc5ddf0509 100644
--- a/components/offline_pages/core/background/offliner_policy.h
+++ b/components/offline_pages/core/background/offliner_policy.h
@@ -24,12 +24,12 @@ const int kDefaultBackgroundProcessingTimeBudgetSeconds =
const int kSinglePageTimeLimitWhenBackgroundScheduledSeconds =
kDozeModeBackgroundServiceWindowSeconds - 10;
-// Immediate processing time limits.
-// Note: experiments on GIN-2g-poor show many page requests took 3 or 4
-// attempts in background scheduled mode with timeout of 2 minutes. So for
-// immediate processing mode, give page requests 4 times that limit (8 min).
-// Then budget up to 5 of those requests in processing window.
-const int kSinglePageTimeLimitForImmediateLoadSeconds = 60 * 8;
+// Immediate processing time limits. Note: experiments on GIN-2g-poor show many
+// page requests took 3 or 4 attempts in background scheduled mode with timeout
+// of 2 minutes. So for immediate processing mode, give page requests just under
+// 5 minutes, which is the timeout limit for the prerender itself. Then budget
+// up to 3 of those requests in processing window.
+const int kSinglePageTimeLimitForImmediateLoadSeconds = 60 * 4 + 50;
const int kImmediateLoadProcessingTimeBudgetSeconds =
kSinglePageTimeLimitForImmediateLoadSeconds * 5;
} // namespace
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698