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

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

Issue 2229983002: Send the list of used and unused resources for precache (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments, removed timestamp from proto Created 4 years, 4 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/unfinished_work.proto
diff --git a/components/precache/core/proto/unfinished_work.proto b/components/precache/core/proto/unfinished_work.proto
index deb6329bbf6c627820cbd379e62312b5da9ecd18..5651bbb8d5e3068b559c5c015c53c6f65fa6d678 100644
--- a/components/precache/core/proto/unfinished_work.proto
+++ b/components/precache/core/proto/unfinished_work.proto
@@ -16,7 +16,7 @@ message TopHost {
optional string hostname = 1;
};
-message PrecacheManifestURL {
+message DeprecatedPrecacheManifestURL {
// The URL that provides a manifest.
optional string url = 1;
};
@@ -28,8 +28,9 @@ message PrecacheUnfinishedWork {
optional PrecacheConfigurationSettings config_settings = 2;
- // Manifest URLs remaining to be fetched.
- repeated PrecacheManifestURL manifest = 3;
+ // DEPRECATED: Manifest URLs remaining to be fetched.
sclittle 2016/08/16 21:05:06 nit: While this is the normal way to deprecate thi
Raj 2016/08/18 00:23:24 Yes. Makes sense.
+ repeated DeprecatedPrecacheManifestURL deprecated_manifest = 3
+ [deprecated = true];
// Resource URLs remaining to be fetched.
repeated PrecacheResource resource = 4;

Powered by Google App Engine
This is Rietveld 408576698