[Offline Pages] Duplicate pages when save/delete bookmarks.

components/offline_pages/offline_page_model_impl_unittest.cc

Index: components/offline_pages/offline_page_model_impl_unittest.cc
diff --git a/components/offline_pages/offline_page_model_impl_unittest.cc b/components/offline_pages/offline_page_model_impl_unittest.cc
index cb0ee364f6a5416e3d3d9ddf46dbd0aec76feb60..d17536b49f1674f7fa8e6bfb3577478ef63063e4 100644
--- a/components/offline_pages/offline_page_model_impl_unittest.cc
+++ b/components/offline_pages/offline_page_model_impl_unittest.cc
@@ -941,7 +941,7 @@ TEST_F(OfflinePageModelImplTest, SaveRetrieveMultipleClientIds) {
   int64_t offline1 = last_save_offline_id();
   EXPECT_TRUE(HasPages(kTestClientNamespace));
 
-  SavePage(kTestUrl, kTestClientId1);
+  SavePage(kTestUrl2, kTestClientId1);
   int64_t offline2 = last_save_offline_id();
 
   EXPECT_NE(offline1, offline2);
@@ -959,6 +959,29 @@ TEST_F(OfflinePageModelImplTest, SaveRetrieveMultipleClientIds) {
   EXPECT_TRUE(id_set.find(offline2) != id_set.end());
 }
 
+TEST_F(OfflinePageModelImplTest, SaveMultiplePagesWithSameURLBySameClientId) {
+  EXPECT_FALSE(HasPages(kTestClientNamespace));
+  SavePage(kTestUrl, kTestClientId1);
+  int64_t offline1 = last_save_offline_id();
+  EXPECT_TRUE(HasPages(kTestClientNamespace));
+
+  SavePage(kTestUrl, kTestClientId1);
+  int64_t offline2 = last_save_offline_id();
+
+  EXPECT_NE(offline1, offline2);
+
+  const std::vector<int64_t> ids = GetOfflineIdsForClientId(kTestClientId1);
+
+  EXPECT_EQ(1UL, ids.size());
+
+  std::set<int64_t> id_set;
+  for (size_t i = 0; i < ids.size(); i++) {
+    id_set.insert(ids[i]);
+  }
+
+  EXPECT_TRUE(id_set.find(offline2) != id_set.end());
+}
+
 TEST_F(OfflinePageModelImplTest, GetBestPage) {
   // We will save 3 pages - two for the same URL, and one for a different URL.
   // Correct behavior will pick the most recently saved page for the correct