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

Unified Diff: components/sync/protocol/search_engine_specifics.proto

Issue 2498053002: Add field to monitor last visited time for each search engine (Closed)
Patch Set: Add unit test for last_visited field. Created 4 years, 1 month 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/sync/protocol/search_engine_specifics.proto
diff --git a/components/sync/protocol/search_engine_specifics.proto b/components/sync/protocol/search_engine_specifics.proto
index 03a3e07f8066bfa93068c68453600909a4cab7db..790b87b5a019964544c5965c3354e29899a50769 100644
--- a/components/sync/protocol/search_engine_specifics.proto
+++ b/components/sync/protocol/search_engine_specifics.proto
@@ -78,4 +78,9 @@ message SearchEngineSpecifics {
// The parameterized URL for a search provider specified new tab page.
optional string new_tab_url = 26;
+
+ // The date this search engine entry was last visited. A UTC timestamp with
+ // units
Peter Kasting 2016/11/21 03:35:08 Nit: Broken line wrapping This usually happens if
ltian 2016/11/28 22:08:02 I know Java's maximum column size is 100 and C/C++
Peter Kasting 2016/11/28 22:20:24 I don't know. I would assume 80 columns in the ab
+ // in microseconds.
+ optional int64 last_visited = 27;
}

Powered by Google App Engine
This is Rietveld 408576698