Refactoring recently closed tab filtering

chrome/browser/sessions/session_utils.h

Index: chrome/browser/sessions/session_utils.h
diff --git a/chrome/browser/sessions/session_utils.h b/chrome/browser/sessions/session_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..ff02f531397293d23a071deb85c5ccef8274228c
--- /dev/null
+++ b/chrome/browser/sessions/session_utils.h
@@ -0,0 +1,28 @@
+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_SESSIONS_SESSION_UTILS_H_
+#define CHROME_BROWSER_SESSIONS_SESSION_UTILS_H_
+#pragma once
+
+#include "chrome/browser/sessions/tab_restore_service.h"
+
+class SessionUtils {
+ public:
+  // Fill the passed list from the entries passed as argument filtering out the
+  // ones with the same title and URL as a previous entry. This avoid filling
+  // the list with things that look like duplicates. A user can still access the

[sky, 2011/09/20 22:10:28]

This last sentence doesn't really make sense here.

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

True. It is more related to the user experience, and has nothing to do in the
model. Removed.

+  // filtered out version by restoring the one he sees as it will allow the
+  // older one to show.
+  //
+  // The |filtering_entries| returned are simply references to the one passed in
+  // without copy: There is no transfer of ownership.
+  static void FilteredEntries(const TabRestoreService::Entries& entries,
+                              TabRestoreService::Entries* filtered_entries);
+
+ private:
+  DISALLOW_IMPLICIT_CONSTRUCTORS(SessionUtils);
+};
+
+#endif  // CHROME_BROWSER_SESSIONS_SESSION_UTILS_H_