Eliminate BaseSessionServiceDelegateImpl

chrome/browser/sessions/session_common_utils.h

Index: chrome/browser/sessions/session_common_utils.h
diff --git a/chrome/browser/sessions/session_common_utils.h b/chrome/browser/sessions/session_common_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..7d1281b2dfbc6004a0bee08cdbbf4a151654b871
--- /dev/null
+++ b/chrome/browser/sessions/session_common_utils.h
@@ -0,0 +1,15 @@
+// Copyright 2014 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_COMMON_UTILS_H_
+#define CHROME_BROWSER_SESSIONS_SESSION_COMMON_UTILS_H_
+
+class GURL;
+
+// Returns whether sessions code should track an entry in the context of
+// //chrome. In particular, blacklists chrome://quit and chrome://restart to
+// avoid quit or restart loops.
+bool ShouldTrackEntry(const GURL& url);

[sky, 2015/09/10 18:03:51]

This function name, and description, are a bit cryptic when viewed in isolation.
Maybe ShouldRestoreUrl?

[blundell, 2015/09/11 08:07:53]

Thanks for pointing this out. I changed to ShouldTrackURLForRestore(), including
changing the client method: I think it's more clear even in the context of the
client.

+
+#endif  // CHROME_BROWSER_SESSIONS_SESSION_COMMON_UTILS_H_