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

Unified Diff: components/offline_pages/offline_page_metadata_store_sql.cc

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Fix trybot 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/offline_pages/offline_page_metadata_store_sql.cc
diff --git a/components/offline_pages/offline_page_metadata_store_sql.cc b/components/offline_pages/offline_page_metadata_store_sql.cc
index e99e267884f3d850179686ad4dfca424fb64fa5a..04ea369e1c8ea7aef2a7121f542e4b9598cf2d65 100644
--- a/components/offline_pages/offline_page_metadata_store_sql.cc
+++ b/components/offline_pages/offline_page_metadata_store_sql.cc
@@ -44,7 +44,7 @@ const char kOfflinePagesColumns[] =
" client_namespace VARCHAR NOT NULL,"
" client_id VARCHAR NOT NULL,"
" online_url VARCHAR NOT NULL,"
- " offline_url VARCHAR NOT NULL DEFAULT '',"
+ " offline_url VARCHAR NOT NULL DEFAULT ''," // Deprecated.
" file_path VARCHAR NOT NULL,"
" title VARCHAR NOT NULL DEFAULT ''"
")";
@@ -73,7 +73,7 @@ enum : int {
OP_CLIENT_NAMESPACE,
OP_CLIENT_ID,
OP_ONLINE_URL,
- OP_OFFLINE_URL,
+ OP_OFFLINE_URL, // Deprecated.
OP_FILE_PATH,
OP_TITLE, // Added in M54.
};

Powered by Google App Engine
This is Rietveld 408576698