Remove NOTIFICATION_HISTORY_URLS_DELETED

chrome/browser/history/history_backend.h

Index: chrome/browser/history/history_backend.h
diff --git a/chrome/browser/history/history_backend.h b/chrome/browser/history/history_backend.h
index d0987e541af75e6c71a21cf38c88aa38b9cbf99b..c672a6f9440966cb232e97d3deebccb53b7eb447 100644
--- a/chrome/browser/history/history_backend.h
+++ b/chrome/browser/history/history_backend.h
@@ -57,6 +57,7 @@ struct HistoryDetails;
 class HistoryDBTask;
 class InMemoryHistoryBackend;
 class TypedUrlSyncableService;
+struct URLsDeletedDetails;
 class VisitFilter;
 
 // The maximum number of icons URLs per page which can be stored in the
@@ -143,6 +144,12 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
     // be forwarded to the HistoryServiceObservers in the correct thread.
     virtual void NotifyURLsModified(const URLRows& changed_urls) = 0;
 
+    // Notify HistoryService that some or all of the URLs have been deleted.
+    // The event will be forwarded to the HistoryServiceObservers in the correct
+    // thread.
+    virtual void NotifyURLsDeleted(
+        const URLsDeletedDetails& deleted_details) = 0;

[sdefresne, 2014/12/04 17:21:12]

I'd prefer if we also stopped using URLsDeletedDetails by declaring the method
as:

virtual void NotifyURLsDeleted(bool all_history,
                               bool expired,
                               const URLRows& rows) = 0;

[nshaik, 2014/12/07 09:34:49]

Done.

+
     // Broadcasts the specified notification to the notification service.
     // This is implemented here because notifications must only be sent from
     // the main thread. This is the only method that doesn't identify the
@@ -582,7 +589,8 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
   FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, UpdateVisitDuration);
   FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, ExpireHistoryForTimes);
   FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, DeleteFTSIndexDatabases);
-
+  FRIEND_TEST_ALL_PREFIXES(ProfileSyncServiceTypedUrlTest,
+                           ProcessUserChangeRemove);
   friend class ::TestingProfile;
 
   // Computes the name of the specified database on disk.