[Sync] Remove static methods on SyncedWindowDelegate.

chrome/browser/ui/sync/browser_synced_window_delegates_getter.cc

+// 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.
+
+#include "chrome/browser/ui/sync/browser_synced_window_delegates_getter.h"
+
+#include "chrome/browser/sync/glue/synced_window_delegate.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/browser_iterator.h"
+#include "chrome/browser/ui/sync/browser_synced_window_delegate.h"
+
+namespace browser_sync {
+
+BrowserSyncedWindowDelegatesGetter::BrowserSyncedWindowDelegatesGetter() {}
+
+std::set<const SyncedWindowDelegate*>
+BrowserSyncedWindowDelegatesGetter::GetSyncedWindowDelegates() {
+  std::set<const SyncedWindowDelegate*> synced_window_delegates;
+  // Add all the browser windows.
+  for (chrome::BrowserIterator it; !it.done(); it.Next())
+    synced_window_delegates.insert(it->synced_window_delegate());
+  return synced_window_delegates;
+}
+
+const SyncedWindowDelegate* BrowserSyncedWindowDelegatesGetter::FindById(
+    SessionID::id_type id) {
+  Browser* browser = chrome::FindBrowserWithID(id);
+  return (browser != NULL) ? browser->synced_window_delegate() : NULL;

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

nit: use nullptr (here and elsewhere in the CL).
(browser != nullptr ? browser->synced_window_delegate() : nullptr)

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

I changed most, but not ones in files where NULL is already prevalent.

+}
+
+}  // namespace browser_sync