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..06cd399ee28eb375bb164be197e3faf9dbc38014 100644
--- a/chrome/browser/sessions/tab_restore_service.h
+++ b/chrome/browser/sessions/tab_restore_service.h
@@ -173,6 +173,9 @@ class TabRestoreService : public BaseSessionService {
  protected:
   virtual void Save();
 
+  // Set of entries.
+  Entries entries_;

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

I had to move the entries to be protected in order to be able to construct a
mock for testing purpose.

[sky, 2011/09/20 16:24:46]

Use entries() instead of making this field protected.

[noyau (Ping after 24h), 2011/09/20 17:09:35]

Reverted as this class is no longer mocked.

+
  private:
   // Used to indicate what has loaded.
   enum LoadState {
@@ -300,9 +303,6 @@ class TabRestoreService : public BaseSessionService {
   // Gets the current time. This uses the time_factory_ if there is one.
   base::Time TimeNow() const;
 
-  // Set of entries.
-  Entries entries_;
-
   // Whether we've loaded the last session.
   int load_state_;