[sync] WebContentsObserver based sessions notifications

chrome/browser/sync/sessions/sync_sessions_router_tab_helper.h

Index: chrome/browser/sync/sessions/sync_sessions_router_tab_helper.h
diff --git a/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.h b/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.h
new file mode 100644
index 0000000000000000000000000000000000000000..5efb31d224259b4332d2d8e4eec3db5347396f44
--- /dev/null
+++ b/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.h
@@ -0,0 +1,74 @@
+// 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_ROUTER_TAB_HELPER_H_
+#define CHROME_BROWSER_SYNC_SESSIONS_SYNC_SESSIONS_ROUTER_TAB_HELPER_H_
+
+#include "components/sessions/core/session_id.h"
+#include "content/public/browser/web_contents_observer.h"
+#include "content/public/browser/web_contents_user_data.h"
+
+namespace sync_sessions {
+
+class SyncSessionsWebContentsRouter;
+
+// TabHelper class that forwards tab-level WebContentsObserver events to a
+// (per-profile) sessions router. The router is responsible for forwarding
+// these events to sessions sync. This class also tracks the source tab id
+// of its corresponding tab, if available.
+// A TabHelper is a WebContentsObserver tied to the top level WebContents for a
+// browser tab.
+// https://chromium.googlesource.com/chromium/src/+/master/docs/tab_helpers.md
+class SyncSessionsRouterTabHelper
+    : public content::WebContentsUserData<SyncSessionsRouterTabHelper>,
+      public content::WebContentsObserver {
+ public:
+  ~SyncSessionsRouterTabHelper() override;
+
+  static void CreateForWebContents(
+      content::WebContents* web_contents,
+      SyncSessionsWebContentsRouter* session_router);
+
+  // Get the tab id of the tab responsible for creating the tab this helper
+  // corresponds to. Returns -1 if there is no such tab.
+  SessionID::id_type source_tab_id() const { return source_tab_id_; }
+
+ private:
+  friend class content::WebContentsUserData<SyncSessionsRouterTabHelper>;
+
+  explicit SyncSessionsRouterTabHelper(content::WebContents* web_contents,
+                                       SyncSessionsWebContentsRouter* router);
+
+  // Set the tab id of the tab reponsible for creating the tab this helper
+  // corresponds to.
+  void set_source_tab_id(const SessionID::id_type id) { source_tab_id_ = id; }
+
+  // WebContentsObserver implementation.
+  void DidFinishNavigation(
+      content::NavigationHandle* navigation_handle) override;
+  void TitleWasSet(content::NavigationEntry* entry, bool explicit_set) override;
+  void WebContentsDestroyed() override;
+  void DidFinishLoad(content::RenderFrameHost* render_frame_host,
+                     const GURL& validated_url) override;
+  void DidOpenRequestedURL(content::WebContents* new_contents,
+                           content::RenderFrameHost* source_render_frame_host,
+                           const GURL& url,
+                           const content::Referrer& referrer,
+                           WindowOpenDisposition disposition,
+                           ui::PageTransition transition,
+                           bool started_from_context_menu,
+                           bool renderer_initiated) override;
+
+  void NotifyRouter();
+
+  // router_ is a KeyedService and is guaranteed to outlive |this|.

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

nit: "router_" -> "|router_|"

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

Done.

+  SyncSessionsWebContentsRouter* router_;
+  SessionID::id_type source_tab_id_;

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

Might be good to have a comment explaining this a bit more (e.g. what kind of
events might set this)

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

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(SyncSessionsRouterTabHelper);
+};
+
+}  // namespace sync_sessions
+
+#endif  // CHROME_BROWSER_SYNC_SESSIONS_SYNC_SESSIONS_ROUTER_TAB_HELPER_H_