[Sync] Componentize synced_tab_delegate

components/sync_sessions/sync_sessions_client.h

Index: components/sync_sessions/sync_sessions_client.h
diff --git a/components/sync_sessions/sync_sessions_client.h b/components/sync_sessions/sync_sessions_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..d86b4ef87d82c9f75ea30eb2997fbac5d3528b10
--- /dev/null
+++ b/components/sync_sessions/sync_sessions_client.h
@@ -0,0 +1,38 @@
+// Copyright 2015 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 COMPONENTS_SYNC_SESSIONS_SYNC_SESSIONS_CLIENT_H_
+#define COMPONENTS_SYNC_SESSIONS_SYNC_SESSIONS_CLIENT_H_
+
+class GURL;
+
+namespace browser_sync {
+class SyncedTabDelegate;

[skym, 2015/10/16 16:55:47]

Are these forward declarations in preparation for something? Or did you change
your mind somewhere and never remove them?

[Nicolas Zea, 2015/10/20 23:14:42]

Woops, these were leftovers from previous iterations. Removed.

+}
+
+namespace sessions {
+struct SessionTab;

[skym, 2015/10/16 16:55:47]

Same.

[Nicolas Zea, 2015/10/20 23:14:42]

Done.

+struct SessionWindow;
+}
+
+namespace sync_sessions {
+
+// Interface for clients of a sync sessions datatype. Should be used as a getter
+// for services and data the Sync Sessions datatype depends on.
+class SyncSessionsClient {
+ public:
+  SyncSessionsClient();
+  virtual ~SyncSessionsClient();
+
+  // Checks if the given url is considered interesting enough to sync. Most urls
+  // are considered interesting. Examples of ones that are not are invalid urls,
+  // files, and chrome internal pages.
+  // TODO(zea): make this a standalone function if the url constants are
+  // componentized.
+  virtual bool ShouldSyncURL(const GURL& url) const = 0;
+};
+
+}  // namespace sync_sessions
+
+#endif  // COMPONENTS_SYNC_SESSIONS_SYNC_SESSIONS_CLIENT_H_