Crashpad Windows: Use the Crashpad client instead of Breakpad on Windows

chrome/app/main_dll_loader_win.cc

 // Copyright (c) 2012 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 <windows.h>  // NOLINT
 #include <shlwapi.h>  // NOLINT
 
 #include "chrome/app/main_dll_loader_win.h"
 
 #include "base/base_paths.h"
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/environment.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/path_service.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/trace_event/trace_event.h"
+#include "base/win/metro.h"
 #include "base/win/scoped_handle.h"
 #include "base/win/windows_version.h"
 #include "chrome/app/chrome_crash_reporter_client.h"
 #include "chrome/app/chrome_watcher_client_win.h"
 #include "chrome/app/chrome_watcher_command_line_win.h"
 #include "chrome/app/image_pre_reader_win.h"
 #include "chrome/app/kasko_client.h"
 #include "chrome/chrome_watcher/chrome_watcher_main_api.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_result_codes.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/env_vars.h"
 #include "chrome/installer/util/google_update_constants.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "chrome/installer/util/install_util.h"
 #include "chrome/installer/util/module_util_win.h"
 #include "chrome/installer/util/util_constants.h"
-#include "components/crash/content/app/breakpad_win.h"
 #include "components/crash/content/app/crash_reporter_client.h"
+#include "components/crash/content/app/crashpad.h"
 #include "content/public/app/sandbox_helper_win.h"
 #include "content/public/common/content_switches.h"
 #include "sandbox/win/src/sandbox.h"
 
 namespace {
 // The entry point signature of chrome.dll.
 typedef int (*DLL_MAIN)(HINSTANCE, sandbox::SandboxInterfaceInfo*);
 
 typedef void (*RelaunchChromeBrowserWithNewCommandLineIfNeededFunc)();
 
@@ skipping 21 matching lines @@
 void RecordDidRun(const base::FilePath& dll_path) {
   bool system_level = !InstallUtil::IsPerUserInstall(dll_path);
   GoogleUpdateSettings::UpdateDidRunState(true, system_level);
 }
 
 void ClearDidRun(const base::FilePath& dll_path) {
   bool system_level = !InstallUtil::IsPerUserInstall(dll_path);
   GoogleUpdateSettings::UpdateDidRunState(false, system_level);
 }
 
-bool InMetroMode() {
-  return (wcsstr(
-      ::GetCommandLineW(), L" -ServerName:DefaultBrowserServer") != nullptr);
+typedef int (*InitMetro)();
+
+// Returns the directory in which the currently running executable resides.
+base::FilePath GetExecutableDir() {
+  base::char16 path[MAX_PATH];
+  ::GetModuleFileNameW(nullptr, path, MAX_PATH);
+  return base::FilePath(path).DirName();
 }
 
-typedef int (*InitMetro)();
+bool WrapMessageBoxWithSEH(const wchar_t* text,
+                           const wchar_t* caption,
+                           UINT flags,
+                           bool* exit_now) {
+  // We wrap the call to MessageBoxW with a SEH handler because it some

[Mark Mentovai, 2015/11/10 17:21:17]

it→on

[scottmg, 2015/11/16 21:48:39]

Done.

+  // machines with CursorXP, PeaDict or with FontExplorer installed it crashes
+  // uncontrollably here. Being this a best effort deal we better go away.
+  __try {
+    *exit_now = (IDOK != ::MessageBoxW(NULL, text, caption, flags));

[Mark Mentovai, 2015/11/10 17:21:17]

and maybe restyle this copied stuff to modern standards, nullptr, IDOK on the
right side.

[scottmg, 2015/11/16 21:48:39]

Done.

+  } __except(EXCEPTION_EXECUTE_HANDLER) {
+    // Its not safe to continue executing, exit silently here.
+    ::TerminateProcess(
+        ::GetCurrentProcess(),
+        g_chrome_crash_client.Pointer()->GetResultCodeRespawnFailed());
+  }
+
+  return true;
+}
+
+// This function is executed by the child process that DumpDoneCallback()
+// spawned and basically just shows the 'chrome has crashed' dialog if
+// the CHROME_CRASHED environment variable is present.
+bool ShowRestartDialogIfCrashed(bool* exit_now) {
+  // If we are being launched in metro mode don't try to show the dialog.
+  if (base::win::IsMetroProcess())
+    return false;
+
+  base::string16 message;
+  base::string16 title;
+  bool is_rtl_locale;
+  if (!g_chrome_crash_client.Pointer()->ShouldShowRestartDialog(
+          &title, &message, &is_rtl_locale)) {
+    return false;
+  }
+
+  // If the UI layout is right-to-left, we need to pass the appropriate MB_XXX
+  // flags so that an RTL message box is displayed.
+  UINT flags = MB_OKCANCEL | MB_ICONWARNING;
+  if (is_rtl_locale)
+    flags |= MB_RIGHT | MB_RTLREADING;
+
+  return WrapMessageBoxWithSEH(message.c_str(), title.c_str(), flags, exit_now);
+}
 
 }  // namespace
 
 //=============================================================================
 
 MainDllLoader::MainDllLoader()
-    : dll_(nullptr), metro_mode_(InMetroMode()) {
+    : dll_(nullptr), metro_mode_(base::win::IsMetroProcess()) {

[cpu_(ooo_6.6-7.5), 2015/11/10 03:06:50]

note that IsProcessImmersive() does not exist in win7 or earlier.

[scottmg, 2015/11/16 21:48:39]

Hmm, is that OK? IsMetroProcess
https://code.google.com/p/chromium/codesearch#chromium/src/base/win/metro.cc&...

-> IsProcessImmersive (ours) -> IsImmersiveProcess() (Windows') and otherwise
returns false. So on <= 7 we'll get false, which I think matches what this
wants?

 }
 
 MainDllLoader::~MainDllLoader() {
 }
 
 // Loading chrome is an interesting affair. First we try loading from the
 // current directory to support run-what-you-compile and other development
 // scenarios.
 // If that fails then we look at the version resource in the current
 // module. This is the expected path for chrome.exe browser instances in an
@@ skipping 43 matching lines @@
   if (metro_mode_) {
     HMODULE metro_dll = Load(&version, &file);
     if (!metro_dll)
       return chrome::RESULT_CODE_MISSING_DATA;
 
     InitMetro chrome_metro_main =
         reinterpret_cast<InitMetro>(::GetProcAddress(metro_dll, "InitMetro"));
     return chrome_metro_main();
   }
 
+  chrome::RegisterPathProvider();
+
   if (process_type_ == "watcher") {
-    chrome::RegisterPathProvider();
 
     base::win::ScopedHandle parent_process;
     base::win::ScopedHandle on_initialized_event;
     DWORD main_thread_id = 0;
     if (!InterpretChromeWatcherCommandLine(cmd_line, &parent_process,
                                            &main_thread_id,
                                            &on_initialized_event)) {
       return chrome::RESULT_CODE_UNSUPPORTED_PARAM;
     }
 
@@ skipping 29 matching lines @@
                         message_window_name.c_str(), channel_name.c_str());
   }
 
   // Initialize the sandbox services.
   sandbox::SandboxInterfaceInfo sandbox_info = {0};
   content::InitializeSandboxInfo(&sandbox_info);
 
   crash_reporter::SetCrashReporterClient(g_chrome_crash_client.Pointer());
   bool exit_now = true;
   if (process_type_.empty()) {
-    if (breakpad::ShowRestartDialogIfCrashed(&exit_now)) {
+    if (ShowRestartDialogIfCrashed(&exit_now)) {
       // We restarted because of a previous crash. Ask user if we should
       // Relaunch. Only for the browser process. See crbug.com/132119.
       if (exit_now)
         return content::RESULT_CODE_NORMAL_EXIT;
     }
   }
-  breakpad::InitCrashReporter(process_type_);
+
+  crash_reporter::InitializeCrashpad(process_type_ == "", process_type_);
 
   dll_ = Load(&version, &file);
   if (!dll_)
     return chrome::RESULT_CODE_MISSING_DATA;
 
   scoped_ptr<base::Environment> env(base::Environment::Create());
   env->SetVar(chrome::kChromeVersionEnvVar, base::WideToUTF8(version));
 
   OnBeforeLaunch(process_type_, file);
   DLL_MAIN chrome_main =
       reinterpret_cast<DLL_MAIN>(::GetProcAddress(dll_, "ChromeMain"));
   int rc = chrome_main(instance, &sandbox_info);
   rc = OnBeforeExit(rc, file);
-  // Sandboxed processes close some system DLL handles after lockdown so ignore
-  // EXCEPTION_INVALID_HANDLE generated on Windows 10 during shutdown of these
-  // processes.
-  // TODO(wfh): Check whether MS have fixed this in Win10 RTM. crbug.com/456193
-  if (base::win::GetVersion() >= base::win::VERSION_WIN10)

[cpu_(ooo_6.6-7.5), 2015/11/10 03:06:50]

scary ..

[scottmg, 2015/11/16 21:48:39]

Yeah, I asked Will about this one, and he said it was OK to try removing. I'd
rather not have to hack exception-swallowing into Crashpad if we can avoid it.

In one of the linked bugs, we there's a log of the specific exception type we
should see in crash/ so we'll know pretty quickly if we're OK.

-    breakpad::ConsumeInvalidHandleExceptions();
   return rc;
 }
 
 void MainDllLoader::RelaunchChromeBrowserWithNewCommandLineIfNeeded() {
   if (!dll_)
     return;
 
   RelaunchChromeBrowserWithNewCommandLineIfNeededFunc relaunch_function =
       reinterpret_cast<RelaunchChromeBrowserWithNewCommandLineIfNeededFunc>(
           ::GetProcAddress(dll_,
@@ skipping 93 matching lines @@
   }
 };
 
 MainDllLoader* MakeMainDllLoader() {
 #if defined(GOOGLE_CHROME_BUILD)
   return new ChromeDllLoader();
 #else
   return new ChromiumDllLoader();
 #endif
 }