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

Unified Diff: ios/chrome/browser/reading_list/reading_list_entry_unittest.cc

Issue 2350663003: Make reading list entry movable only (Closed)
Patch Set: indentation Created 4 years, 3 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: ios/chrome/browser/reading_list/reading_list_entry_unittest.cc
diff --git a/ios/chrome/browser/reading_list/reading_list_entry_unittest.cc b/ios/chrome/browser/reading_list/reading_list_entry_unittest.cc
index 215c3fdd32df3c9a0d0ff76facae81c995ee8fd4..6d517ee6d5e8067f8b16673bcabca25cdbfe5ffc 100644
--- a/ios/chrome/browser/reading_list/reading_list_entry_unittest.cc
+++ b/ios/chrome/browser/reading_list/reading_list_entry_unittest.cc
@@ -20,12 +20,16 @@ TEST(ReadingListEntry, CompareFailureIgnoreTitle) {
EXPECT_FALSE(e1 == e2);
}
-TEST(ReadingListEntry, CopyAreEquals) {
- const ReadingListEntry e1(GURL("http://example.com"), "bar");
- const ReadingListEntry e2(e1);
+TEST(ReadingListEntry, MovesAreEquals) {
+ ReadingListEntry e1(GURL("http://example.com"), "bar");
+ ReadingListEntry e2(GURL("http://example.com"), "bar");
+ ASSERT_EQ(e1, e2);
+ ASSERT_EQ(e1.Title(), e2.Title());
- EXPECT_EQ(e1, e2);
- EXPECT_EQ(e1.Title(), e2.Title());
+ ReadingListEntry e3(std::move(e1));
+
+ EXPECT_EQ(e3, e2);
+ EXPECT_EQ(e3.Title(), e2.Title());
}
TEST(ReadingListEntry, DistilledURL) {
« no previous file with comments | « ios/chrome/browser/reading_list/reading_list_entry.cc ('k') | ios/chrome/browser/reading_list/reading_list_model_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698