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

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

Issue 2804543003: [Offline Pages] Remove activation time for background loading. (Closed)
Patch Set: Keep schema. 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/offline_pages/core/background/request_queue_store_sql.h
diff --git a/components/offline_pages/core/background/request_queue_store_sql.h b/components/offline_pages/core/background/request_queue_store_sql.h
index 6bd9b9a4ccdd79d54592fb3a012f4704b7d587dd..7c694f39db754dff923edb247e31935a83241030 100644
--- a/components/offline_pages/core/background/request_queue_store_sql.h
+++ b/components/offline_pages/core/background/request_queue_store_sql.h
@@ -24,6 +24,17 @@ class Connection;
namespace offline_pages {
// SQLite implementation of RequestQueueStore.
+//
+// This store has a history of schema updates.
+// Original schema was delivered in M57. Since then the following changes
+// happened:
+// * In M58 original_url was added.
+//
+// TODO(romax): remove all activation_time related code the next we change the
+// schema.
+//
+// Looking for procesure to update the schema, please refer to
+// offline_page_metadata_store_sql.h
class RequestQueueStoreSQL : public RequestQueueStore {
public:
RequestQueueStoreSQL(

Powered by Google App Engine
This is Rietveld 408576698