diff options
| author | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-18 22:11:01 +0000 |
|---|---|---|
| committer | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-18 22:11:01 +0000 |
| commit | c0c0a30362549e8459a2022cc2676ce4281f388d (patch) | |
| tree | 916b83720a497e701195ff4706c623578f5056c7 /content/ppapi_plugin | |
| parent | facf11efd060d2abcb1d7c296f31d552eab496c8 (diff) | |
| download | chromium_src-c0c0a30362549e8459a2022cc2676ce4281f388d.zip chromium_src-c0c0a30362549e8459a2022cc2676ce4281f388d.tar.gz chromium_src-c0c0a30362549e8459a2022cc2676ce4281f388d.tar.bz2 | |
[ppapi] Add crash key for which plugin is loaded.
In theory, it is easy to look at the modules in the crash dump. In
practice, if the plugin has been unloaded that is hard.
BUG=172319
Review URL: https://chromiumcodereview.appspot.com/12776003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188845 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/ppapi_plugin')
| -rw-r--r-- | content/ppapi_plugin/ppapi_thread.cc | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/content/ppapi_plugin/ppapi_thread.cc b/content/ppapi_plugin/ppapi_thread.cc index 6aa7b80..bed78b8 100644 --- a/content/ppapi_plugin/ppapi_thread.cc +++ b/content/ppapi_plugin/ppapi_thread.cc @@ -7,6 +7,7 @@ #include <limits> #include "base/command_line.h" +#include "base/debug/crash_logging.h" #include "base/logging.h" #include "base/process_util.h" #include "base/rand_util.h" @@ -227,6 +228,10 @@ void PpapiThread::Unregister(uint32 plugin_dispatcher_id) { void PpapiThread::OnLoadPlugin(const base::FilePath& path, const ppapi::PpapiPermissions& permissions) { + // In case of crashes, the crash dump doesn't indicate which plugin + // it came from. + base::debug::SetCrashKeyValue("ppapi_path", path.MaybeAsASCII()); + SavePluginName(path); // This must be set before calling into the plugin so it can get the |