[Sync] Remove static methods on SyncedWindowDelegate.

chrome/browser/sync/glue/synced_tab_delegate.h

Index: chrome/browser/sync/glue/synced_tab_delegate.h
diff --git a/chrome/browser/sync/glue/synced_tab_delegate.h b/chrome/browser/sync/glue/synced_tab_delegate.h
index 0645e4eb949b532d44321ff70b5bbee64fd29ab8..e36d54ba438eed71acd1796931b63bb06312bfef 100644
--- a/chrome/browser/sync/glue/synced_tab_delegate.h
+++ b/chrome/browser/sync/glue/synced_tab_delegate.h
@@ -8,6 +8,8 @@
 #include <string>
 #include <vector>
 
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/sync/sessions/synced_window_delegates_getter.h"
 #include "components/sessions/session_id.h"
 
 class Profile;
@@ -24,7 +26,8 @@ class SyncedWindowDelegate;
 // directly on WebContents, NavigationController, and the extensions TabHelper.
 class SyncedTabDelegate {
  public:
-  virtual ~SyncedTabDelegate() {}
+  SyncedTabDelegate();

[Nicolas Zea, 2015/08/31 22:12:47]

nit: I think it would be better to make this constructor protected to make it
clear that embedders should not be instantiating a SyncedTabDelegate directly,
but instead a derived class.

[maxbogue, 2015/09/01 17:29:27]

Done.

+  virtual ~SyncedTabDelegate();
 
   // Methods from TabContents.
 
@@ -69,6 +72,8 @@ class SyncedTabDelegate {
   // Returns true if this tab should be synchronized.
   bool ShouldSync() const;
 
+  void SetSyncedWindowGetter(scoped_ptr<SyncedWindowDelegatesGetter> getter);

[Nicolas Zea, 2015/08/31 22:12:47]

Add comment? Would probably be good to also explain where/when this is supposed
to be called.

[maxbogue, 2015/09/01 17:29:27]

Done.

+
   // Returns the SyncedTabDelegate associated with WebContents.
   static SyncedTabDelegate* ImplFromWebContents(
       content::WebContents* web_contents);
@@ -76,6 +81,8 @@ class SyncedTabDelegate {
  protected:
   // Overridden by the tests to avoid interaction with static state.
   virtual const SyncedWindowDelegate* GetSyncedWindowDelegate() const;
+
+  scoped_ptr<SyncedWindowDelegatesGetter> synced_window_getter_;

[Nicolas Zea, 2015/08/31 22:12:47]

make private?

[maxbogue, 2015/09/01 17:29:27]

Done.

 };
 
 }  // namespace browser_sync