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"
 
+#include "content/public/common/service_manager_connection.h"
+#include "services/service_manager/runner/common/client_util.h"

[fwang, 2016/10/26 07:42:58]

It looks like these should be in a
#if defined(USE_AURA) && !defined(OS_MACOSX) too.

[tonikitoo, 2016/10/26 13:41:45]

Done.

+
+#if defined(USE_AURA) && !defined(OS_MACOSX)
+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)
+  content::ServiceManagerConnection* service_manager_connection =
+      content::ServiceManagerConnection::GetForProcess();
+  if (service_manager_connection && service_manager::ServiceManagerIsRemote())
+    return GetLocalProcessWindowAtPointMus(screen_point, ignore);
+  else

[fwang, 2016/10/26 07:42:58]

The else is no longer necessary since we leave the function at the return
statement.

[tonikitoo, 2016/10/26 13:41:46]

Done.

+#endif
+
+  return GetLocalProcessWindowAtPointImpl(screen_point, ignore);
+}