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

Unified Diff: chrome/browser/predictors/resource_prefetch_predictor_tables.h

Issue 2762673002: predictors: Pass manifests from Bork to store in ResourcePrefetchPredictor. (Closed)
Patch Set: Rename PrecacheManifestDelegate to PrecacheManager::Delegate. Created 3 years, 9 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/predictors/resource_prefetch_predictor_tables.h
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_tables.h b/chrome/browser/predictors/resource_prefetch_predictor_tables.h
index ff9c5ea220f9be08f63e4a8f7477c862d644b032..e9d50782414b5a3ca551659d0835f6e75fb18316 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_tables.h
+++ b/chrome/browser/predictors/resource_prefetch_predictor_tables.h
@@ -104,6 +104,9 @@ class ResourcePrefetchPredictorTables : public PredictorTableBase {
// The maximum length of the string that can be stored in the DB.
static constexpr size_t kMaxStringLength = 1024;
+ // The maximum size in bytes of the manifest that can be stored in the DB.
+ static constexpr size_t kMaxManifestByteSize = 16 * 1024;
Benoit L 2017/03/21 13:24:15 nit: Can this be in the implementation file? Up to
alexilin 2017/03/21 15:32:13 I tried to keep it consistent with kMaxStringLengt
+
protected:
// Protected for testing. Use PredictorDatabase::resource_prefetch_tables()
// instead of this constructor.

Powered by Google App Engine
This is Rietveld 408576698