Make chrome.exe rendezvous with existing chrome process earlier.

chrome/browser/chrome_process_finder_win.cc

Index: chrome/browser/chrome_process_finder_win.cc
diff --git a/chrome/browser/chrome_process_finder_win.cc b/chrome/browser/chrome_process_finder_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3b753a981dacf45e61c9f2f8e3c8c639d0feb3ab
--- /dev/null
+++ b/chrome/browser/chrome_process_finder_win.cc
@@ -0,0 +1,104 @@
+// Copyright 2013 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/chrome_process_finder_win.h"
+
+#include "base/file_util.h"
+#include "base/process_util.h"
+#include "base/stringprintf.h"
+#include "base/utf_string_conversions.h"
+#include "base/win/metro.h"
+#include "base/win/scoped_handle.h"
+#include "base/win/win_util.h"
+#include "base/win/windows_version.h"
+#include "chrome/browser/ui/metro_chrome_win.h"
+#include "chrome/common/chrome_constants.h"
+#include "net/base/escape.h"

[cpu_(ooo_6.6-7.5), 2013/05/09 18:15:47]

grt, can we make chrome exe not depend on net. Or is this a lost cause. At least
in the gypi I don't see net.

[grt (UTC plus 2), 2013/05/09 18:29:24]

That's probably no good. chrome.exe doesn't currently depend on net.

+
+
+namespace {
+const int kTimeoutInSeconds = 20;
+}
+
+namespace chrome {
+
+HWND FindRunningChromeWindow(const base::FilePath& user_data_dir) {
+  return FindWindowEx(HWND_MESSAGE, NULL, chrome::kMessageWindowClass,
+                      user_data_dir.value().c_str());
+}
+
+bool AttemptToNotifyRunningChrome(HWND remote_window) {
+  const char kSearchUrl[] =

[grt (UTC plus 2), 2013/05/09 16:10:44]

  static const char

[koz (OOO until 15th September), 2013/05/10 01:46:48]

Done.

+    "http://www.google.com/search?q=%s&sourceid=chrome&ie=UTF-8";
+  DWORD process_id = 0;

[grt (UTC plus 2), 2013/05/09 16:10:44]

how about:
  DCHECK(remote_window);

[koz (OOO until 15th September), 2013/05/10 01:46:48]

Done.

+  DWORD thread_id = GetWindowThreadProcessId(remote_window, &process_id);
+  if (!thread_id || !process_id)
+    return false;
+
+  if (base::win::IsMetroProcess()) {
+    // Interesting corner case. We are launched as a metro process but we
+    // found another chrome running. Since metro enforces single instance then
+    // the other chrome must be desktop chrome and this must be a search charm
+    // activation. This scenario is unique; other cases should be properly
+    // handled by the delegate_execute which will not activate a second chrome.
+    string16 terms;
+    base::win::MetroLaunchType launch = base::win::GetMetroLaunchParams(&terms);
+    if (launch != base::win::METRO_SEARCH) {
+      LOG(WARNING) << "In metro mode, but and launch is " << launch;
+    } else {
+      std::string query = net::EscapeQueryParamValue(UTF16ToUTF8(terms), true);
+      std::string url = base::StringPrintf(kSearchUrl, query.c_str());
+      SHELLEXECUTEINFOA sei = { sizeof(sei) };
+      sei.fMask = SEE_MASK_FLAG_LOG_USAGE;
+      sei.nShow = SW_SHOWNORMAL;
+      sei.lpFile = url.c_str();
+      OutputDebugStringA(sei.lpFile);
+      sei.lpDirectory = "";
+      ::ShellExecuteExA(&sei);

[cpu_(ooo_6.6-7.5), 2013/05/09 18:15:47]

Mix of :: For windows api calls see for example line 58 vs line 92. My
preference is for making them all have ::

[koz (OOO until 15th September), 2013/05/10 01:46:48]

Done.

+    }
+    return true;
+  }
+
+  base::win::ScopedHandle process_handle;
+  if (base::win::GetVersion() >= base::win::VERSION_WIN8 &&
+      base::OpenProcessHandleWithAccess(
+          process_id, PROCESS_QUERY_INFORMATION,
+          process_handle.Receive()) &&
+      base::win::IsProcessImmersive(process_handle.Get())) {
+    chrome::ActivateMetroChrome();
+    return true;
+  }
+
+  // Send the command line to the remote chrome window.
+  // Format is "START\0<<<current directory>>>\0<<<commandline>>>".
+  std::wstring to_send(L"START\0", 6);  // want the NULL in the string.
+  base::FilePath cur_dir;
+  file_util::GetCurrentDirectory(&cur_dir);
+  to_send.append(cur_dir.value());
+  to_send.append(L"\0", 1);  // Null separator.
+  to_send.append(GetCommandLineW());
+  to_send.append(L"\0", 1);  // Null separator.
+
+  // Allow the current running browser window making itself the foreground
+  // window (otherwise it will just flash in the taskbar).
+  AllowSetForegroundWindow(process_id);
+
+  COPYDATASTRUCT cds;
+  cds.dwData = 0;
+  cds.cbData = static_cast<DWORD>((to_send.length() + 1) * sizeof(wchar_t));
+  cds.lpData = const_cast<wchar_t*>(to_send.c_str());
+  DWORD_PTR result = 0;
+  if (SendMessageTimeout(remote_window,
+                         WM_COPYDATA,
+                         NULL,
+                         reinterpret_cast<LPARAM>(&cds),
+                         SMTO_ABORTIFHUNG,
+                         kTimeoutInSeconds * 1000,
+                         &result)) {
+    return !!result;
+  }
+  return false;
+}
+
+}  // namespace chrome