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

Unified Diff: components/precache/core/proto/precache.proto

Issue 2831233004: predictors: Add resource type to manifest. (Closed)
Patch Set: Use the same resource type ordering. Created 3 years, 8 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: components/precache/core/proto/precache.proto
diff --git a/components/precache/core/proto/precache.proto b/components/precache/core/proto/precache.proto
index 2e9aec257eba888885ead0edb2ea113a006b963f..f8a10c0e729b813255b1a7d01a8859e26e7a2826 100644
--- a/components/precache/core/proto/precache.proto
+++ b/components/precache/core/proto/precache.proto
@@ -26,6 +26,20 @@ message PrecacheResource {
// This is a non-negative number, with higher being more important. Its value
// depends on PrecacheConfigurationSettings.resource_weight_function.
optional double weight = 4;
+
+ enum Type {
+ RESOURCE_TYPE_UNKNOWN = 0;
+
+ RESOURCE_TYPE_IMAGE = 1;
+ RESOURCE_TYPE_FONT = 2;
+ RESOURCE_TYPE_STYLESHEET = 3;
+ RESOURCE_TYPE_SCRIPT = 4;
+
+ RESOURCE_TYPE_OTHER = 7;
+ }
+
+ // The type of resource.
+ optional Type type = 5;
Benoit L 2017/04/27 08:54:15 Does the precache code need to be updated in any w
alexilin 2017/04/27 15:32:23 I don't think so. This field was in real manifests
};
message PrecacheManifestId {

Powered by Google App Engine
This is Rietveld 408576698