[sync] WebContentsObserver based sessions notifications

chrome/browser/sync/sessions/sync_sessions_web_contents_router.h

Index: chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
diff --git a/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h b/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
new file mode 100644
index 0000000000000000000000000000000000000000..69314731523c13694587bd1b92c32e50f62e2c16
--- /dev/null
+++ b/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
@@ -0,0 +1,66 @@
+// Copyright 2017 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_SYNC_SESSIONS_SYNC_SESSIONS_WEB_CONTENTS_ROUTER_H_
+#define CHROME_BROWSER_SYNC_SESSIONS_SYNC_SESSIONS_WEB_CONTENTS_ROUTER_H_
+
+#include "base/callback_list.h"
+
+// Android has no BrowserList or TabStripModel, so we exclude code that refers
+// to those two things. For non-android platforms, this code is used to
+// ensure we are notified of tabs being added and moved between tab strips.
+
+#include "components/keyed_service/core/keyed_service.h"
+#include "components/sync/model/syncable_service.h"
+#include "components/sync_sessions/local_session_event_router.h"
+
+namespace content {
+class WebContents;
+}  // namespace content
+
+class Profile;
+
+namespace sync_sessions {
+
+// WebContentsObserver-based implementation of LocalSessionEventRouter. This
+// class is responsible for notifying Sessions Sync when local tabs are
+// modified. It does this by forwarding the events pushed to it by individual
+// WebContentsObservers, which are scoped to a single WebContents/tab.
+class SyncSessionsWebContentsRouter : public LocalSessionEventRouter,
+                                      public KeyedService {
+ public:
+  explicit SyncSessionsWebContentsRouter(Profile* profile);
+
+  // Notify the router that the tab corresponding to |web_contents| has been
+  // modified in some way.
+  void NotifyTabModified(content::WebContents* web_contents);
+  // Inject a flare that can be used to start sync. See the comment for
+  // StartSyncFlare in syncable_service.h for more.
+  void InjectStartSyncFlare(syncer::SyncableService::StartSyncFlare flare);
+
+  // SessionsSyncManager::LocalEventRouter implementation.
+  void StartRoutingTo(LocalSessionEventHandler* handler) override;
+  void Stop() override;
+
+  // KeyedService implementation.
+  void Shutdown() override;
+
+ protected:
+  ~SyncSessionsWebContentsRouter() override;
+  LocalSessionEventHandler* handler_ = nullptr;

[Nicolas Zea, 2017/03/20 18:14:50]

style: data members should be private
https://google.github.io/styleguide/cppguide.html#Inheritance Consider adding a
getter method instead. Although it's also worth considering if perhaps
composition might be a better fit here (inheritance in general is discouraged
beyond basic interfaces)

[Patrick Noland, 2017/03/21 22:54:31]

Done. I changed the MultiWindowed router to BrowserListRouterHelper, which is a
standalone object owned by SyncSessionsWebContentsRouter.

+
+ private:
+  void OnFaviconsChanged(const std::set<GURL>& page_urls, const GURL& icon_url);
+
+  std::unique_ptr<base::CallbackList<void(const std::set<GURL>&,
+                                          const GURL&)>::Subscription>
+      favicon_changed_subscription_;
+  syncer::SyncableService::StartSyncFlare flare_;
+
+  DISALLOW_COPY_AND_ASSIGN(SyncSessionsWebContentsRouter);
+};
+
+}  // namespace sync_sessions
+
+#endif  // CHROME_BROWSER_SYNC_SESSIONS_SYNC_SESSIONS_WEB_CONTENTS_ROUTER_H_