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

Unified Diff: components/reading_list/ios/reading_list_store.h

Issue 2592043002: Add title update merge logic to reading list entry (Closed)
Patch Set: Update function comment Created 4 years 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/reading_list/ios/reading_list_store.h
diff --git a/components/reading_list/ios/reading_list_store.h b/components/reading_list/ios/reading_list_store.h
index ff4b4d880774fdcdb23b4629b659afeab57dde15..a32e39de67b1ba3ee526a6af51417f8c0e74e203 100644
--- a/components/reading_list/ios/reading_list_store.h
+++ b/components/reading_list/ios/reading_list_store.h
@@ -73,7 +73,13 @@ class ReadingListStore : public ReadingListModelStorage,
// submitted to sync in strictly increasing order.
// Entries are in increasing order if all the fields respect increasing order.
// - URL must be the same.
- // - title must verify rhs.title.compare(lhs.title) >= 0
+ // - update_title_time_us:
+ // rhs.update_title_time_us >= lhs.update_title_time_us
+ // - title:
+ // if rhs.update_title_time_us > lhs.update_title_time_us
+ // title can be anything
+ // if rhs.update_title_time_us == lhs.update_title_time_us
+ // title must verify rhs.title.compare(lhs.title) >= 0
// - creation_time_us:
// rhs.creation_time_us >= lhs.creation_time_us
// - rhs.first_read_time_us:

Powered by Google App Engine
This is Rietveld 408576698