Generate dumps for relevant renderers when users kill a hung pepper plugin.

chrome/app/breakpad_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 "chrome/app/breakpad_win.h"
 
 #include <windows.h>
 #include <shellapi.h>
 #include <tchar.h>
 #include <userenv.h>
@@ skipping 18 matching lines @@
 #include "chrome/app/breakpad_field_trial_win.h"
 #include "chrome/app/hard_error_handler_win.h"
 #include "chrome/common/child_process_logging.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_chrome_sxs_distribution.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "chrome/installer/util/install_util.h"
 #include "policy/policy_constants.h"
+#include "ppapi/shared_impl/ppapi_message_tracker.h"
 
 // userenv.dll is required for GetProfileType().
 #pragma comment(lib, "userenv.lib")
 
 namespace breakpad_win {
 
 // TODO(raymes): Modify the way custom crash info is stored. g_custom_entries
 // is way too too fragile. See
 // https://code.google.com/p/chromium/issues/detail?id=137062.
 std::vector<google_breakpad::CustomInfoEntry>* g_custom_entries = NULL;
@@ skipping 74 matching lines @@
   return 0;
 }
 
 // Injects a thread into a remote process to dump state when there is no crash.
 extern "C" HANDLE __declspec(dllexport) __cdecl
 InjectDumpProcessWithoutCrash(HANDLE process) {
   return CreateRemoteThread(process, NULL, 0, DumpProcessWithoutCrashThread,
                             0, 0, NULL);
 }
 
+DWORD WINAPI DumpProcessHandlingPepper(void*) {

[brettw, 2012/09/14 19:44:37]

It would make more sense to me if this was called something like
DumpIfHandlingPepper or something. From the current name, I originally thought
this would find the process handling Pepper calls and dump it.

[yzshen1, 2012/09/14 20:07:19]

Done.

+  if (ppapi::PpapiMessageTracker::GetInstance()->IsHandlingMessage())
+    DumpProcessWithoutCrash();
+  return 0;
+}
+
+// Injects a thread into a remote process to dump state if the process is
+// handling pepper messages.
+extern "C" HANDLE __declspec(dllexport) __cdecl
+InjectDumpProcessHandlingPepper(HANDLE process) {
+  return CreateRemoteThread(process, NULL, 0, DumpProcessHandlingPepper, 0, 0,
+                            NULL);
+}
+
 // Reduces the size of the string |str| to a max of 64 chars. Required because
 // breakpad's CustomInfoEntry raises an invalid_parameter error if the string
 // we want to set is longer.
 std::wstring TrimToBreakpadMax(const std::wstring& str) {
   std::wstring shorter(str);
   return shorter.substr(0,
       google_breakpad::CustomInfoEntry::kValueMaxLength - 1);
 }
 
 static void SetIntegerValue(size_t offset, int value) {
@@ skipping 771 matching lines @@
   previous_filter = SetUnhandledExceptionFilter(filter);
 }
 
 void StringVectorToCStringVector(const std::vector<std::wstring>& wstrings,
                                  std::vector<const wchar_t*>* cstrings) {
   cstrings->clear();
   cstrings->reserve(wstrings.size());
   for (size_t i = 0; i < wstrings.size(); ++i)
     cstrings->push_back(wstrings[i].c_str());
 }