[Offline pages] Properly informing observers when offline page is deleted

components/offline_pages/offline_page_model_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/offline_pages/offline_page_model.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
@@ skipping 653 matching lines @@
 
   ResetResults();
 
   // Delete one page.
   model()->DeletePageByBookmarkId(
       kTestPageBookmarkId1, base::Bind(&OfflinePageModelTest::OnDeletePageDone,
                                        AsWeakPtr()));
 
   PumpLoop();
 
+  EXPECT_EQ(last_deleted_bookmark_id(), kTestPageBookmarkId1);
   EXPECT_EQ(DeletePageResult::SUCCESS, last_delete_result());
   ASSERT_EQ(1u, store->offline_pages().size());
   EXPECT_EQ(kTestUrl2, store->offline_pages()[0].url);
 
   // Delete another page.
   model()->DeletePageByBookmarkId(
       kTestPageBookmarkId2, base::Bind(&OfflinePageModelTest::OnDeletePageDone,
                                        AsWeakPtr()));
 
   ResetResults();
 
   PumpLoop();
 
+  EXPECT_EQ(last_deleted_bookmark_id(), kTestPageBookmarkId2);
   EXPECT_EQ(DeletePageResult::SUCCESS, last_delete_result());
   EXPECT_EQ(0u, store->offline_pages().size());
 }
 
 TEST_F(OfflinePageModelTest, DeletePageNotFound) {
   model()->DeletePageByBookmarkId(
       kTestPageBookmarkId1, base::Bind(&OfflinePageModelTest::OnDeletePageDone,
                                        AsWeakPtr()));
   EXPECT_EQ(DeletePageResult::NOT_FOUND, last_delete_result());
 }
@@ skipping 245 matching lines @@
           .Pass());
   model()->SavePage(
       kTestUrl2, kTestPageBookmarkId2, archiver3.Pass(),
       base::Bind(&OfflinePageModelTest::OnSavePageDone, AsWeakPtr()));
   PumpLoop();
   EXPECT_EQ(1UL, model()->GetAllPages().size());
   EXPECT_EQ(1UL, GetStore()->offline_pages().size());
 }
 
 }  // namespace offline_pages