Refactor WindowFinder definition

chrome/browser/ui/views/tabs/window_finder.cc

 // Copyright 2016 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/views/tabs/window_finder.h"
 
+#if defined(USE_AURA) && !defined(OS_MACOSX)
+#include "content/public/common/service_manager_connection.h"
+#include "services/service_manager/runner/common/client_util.h"
+
+gfx::NativeWindow GetLocalProcessWindowAtPointMus(
+    const gfx::Point& screen_point,
+    const std::set<gfx::NativeWindow>& ignore);
+#endif
+
 WindowFinder::WindowFinder() {}
+
 WindowFinder::~WindowFinder() {}
+
+gfx::NativeWindow WindowFinder::GetLocalProcessWindowAtPoint(
+    const gfx::Point& screen_point,
+    const std::set<gfx::NativeWindow>& ignore) {
+#if defined(USE_AURA) && !defined(OS_MACOSX)

[sky, 2016/10/26 22:01:28]

I dislike the amount of ifdefs needed as a result of this. I would prefer this
exist in the places that know about mash. WindowFinderMus need not be a class.
What is WindowFinderMus::GetLocalProcessWindowAtPoint() can be moved to a
function that is called as appropriate.

+  content::ServiceManagerConnection* service_manager_connection =
+      content::ServiceManagerConnection::GetForProcess();
+  if (service_manager_connection && service_manager::ServiceManagerIsRemote())
+    return GetLocalProcessWindowAtPointMus(screen_point, ignore);
+#endif
+
+  return GetLocalProcessWindowAtPointImpl(screen_point, ignore);
+}