Add merge rules for first read time

Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041
Committed: https://crrev.com/27a90aeec4c3930f8d68f8d7dd1d6d2f7d8a64a6
Cr-Commit-Position: refs/heads/master@{#437506}

[Olivier, 2016-12-08 16:28:11]

Description was changed from

==========
Add merge rules for first read time

BUG=651041
==========

to

==========
Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041
==========

[Olivier, 2016-12-08 16:28:59]

olivierrobin@chromium.org changed reviewers:
+ gambard@chromium.org

[gambard, 2016-12-09 08:06:55]

lgtm with comments

https://codereview.chromium.org/2558183004/diff/10001/components/reading_list...
File components/reading_list/ios/reading_list_store.h (right):

https://codereview.chromium.org/2558183004/diff/10001/components/reading_list...
components/reading_list/ios/reading_list_store.h:85: // - if
rhs.creation_time_us == lhs.creation_time_us, t
remove "t"

https://codereview.chromium.org/2558183004/diff/10001/components/reading_list...
components/reading_list/ios/reading_list_store.h:86: //         
rhs.first_read_time_us <= lhs.first_read_time_us
What happens if lhs.first_read_time_us == 0?

[Olivier, 2016-12-09 09:37:53]

Done, thanks

[Olivier, 2016-12-09 09:37:56]

The CQ bit was checked by olivierrobin@chromium.org

[Olivier, 2016-12-09 09:37:57]

The patchset sent to the CQ was uploaded after l-g-t-m from gambard@chromium.org
Link to the patchset: https://codereview.chromium.org/2558183004/#ps50001
(title: "=0")

[commit-bot: I haz the power, 2016-12-09 09:38:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 09:46:48]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1481276276574760,
"parent_rev": "d596a10b9056addf0c44a48e800fa7f01a749ec6", "commit_rev":
"ac2a79937bd64ffcc42735c4bdbdadb636c35dee"}

[commit-bot: I haz the power, 2016-12-09 09:47:13]

Description was changed from

==========
Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041
==========

to

==========
Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041
==========

[commit-bot: I haz the power, 2016-12-09 09:47:13]

Committed patchset #4 (id:50001)

[commit-bot: I haz the power, 2016-12-09 09:51:04]

Description was changed from

==========
Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041
==========

to

==========
Add merge rules for first read time

Also fix the seen merge logic.

BUG=651041

Committed: https://crrev.com/27a90aeec4c3930f8d68f8d7dd1d6d2f7d8a64a6
Cr-Commit-Position: refs/heads/master@{#437506}
==========

[commit-bot: I haz the power, 2016-12-09 09:51:05]

Patchset 4 (id:??) landed as
https://crrev.com/27a90aeec4c3930f8d68f8d7dd1d6d2f7d8a64a6
Cr-Commit-Position: refs/heads/master@{#437506}