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

Unified Diff: chrome/browser/prerender/prerender_config.h

Issue 10802024: Make three simultanious prerenders the default maximum in Canary and Dev only. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remediation to review, rebase on top of field trial change Created 8 years, 5 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
Index: chrome/browser/prerender/prerender_config.h
diff --git a/chrome/browser/prerender/prerender_config.h b/chrome/browser/prerender/prerender_config.h
index 04f244b3e184bb849296e8533576b19d9d421336..6c0136b0f0eb454148dcf6b86b28d7c01a24bd58 100644
--- a/chrome/browser/prerender/prerender_config.h
+++ b/chrome/browser/prerender/prerender_config.h
@@ -5,6 +5,8 @@
#ifndef CHROME_BROWSER_PRERENDER_PRERENDER_CONFIG_H_
#define CHROME_BROWSER_PRERENDER_PRERENDER_CONFIG_H_
+#include <string>
+
#include "base/time.h"
#include "ui/gfx/rect.h"
@@ -17,7 +19,8 @@ struct Config {
size_t max_bytes;
// Number of simultaneous prendered pages allowed.
- unsigned int max_elements;
+ static const size_t kMaximumMaxConcurrency = 3;
dominich 2012/07/23 18:45:56 This isn't referenced. PrerenderManager sets '3' e
gavinp 2012/07/23 18:51:55 See http://codereview.chromium.org/10802024/diff/1
dominich 2012/07/23 18:57:14 I see. maybe instead of an enum you should be usin
gavinp 2012/07/23 19:06:10 Well, that's just giving up and wasting a lot more
+ size_t max_concurrency;
// Is rate limiting enabled?
bool rate_limit_enabled;
@@ -42,3 +45,4 @@ struct Config {
} // namespace prerender
#endif // CHROME_BROWSER_PRERENDER_PRERENDER_CONFIG_H_
+

Powered by Google App Engine
This is Rietveld 408576698