Refactoring recently closed tab filtering

chrome/browser/sessions/tab_restore_service.h

Index: chrome/browser/sessions/tab_restore_service.h
diff --git a/chrome/browser/sessions/tab_restore_service.h b/chrome/browser/sessions/tab_restore_service.h
index 714e83eeeaf656ff2c1c9aa5189c1dc5ffdc66e5..fedd03d778725fa75d7f8be7306161502385db6f 100644
--- a/chrome/browser/sessions/tab_restore_service.h
+++ b/chrome/browser/sessions/tab_restore_service.h
@@ -146,6 +146,12 @@ class TabRestoreService : public BaseSessionService {
   // front.
   virtual const Entries& entries() const;
 
+  // Fill the passed list with the entries filtering out the ones that are on
+  // the same URL as a previous entry. This avoid filling the list with things
+  // that look like duplicates. A user can still access the filtered out version
+  // by restoring the one he sees as it will allow the older one to show.
+  void FilteredEntries(TabRestoreService::Entries* filteredEntries) const;

[sky, 2011/09/19 16:23:15]

If this is only going to be used by RecentlyClosedTabHandler I would rather see
the code there. If that isn't the case pull this into a static method on a class
name like SessionUtil in this directory.

[sky, 2011/09/19 20:58:17]

When you move this to a separate class make the method take a const Entries&.
That'll be easier to write tests for.

[noyau (Ping after 24h), 2011/09/20 15:46:58]

This will be used in some other context. SessionUtils created.

+
   // Restores the most recently closed entry. Does nothing if there are no
   // entries to restore. If the most recently restored entry is a tab, it is
   // added to |delegate|.