Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(142)

Unified Diff: chrome/common/crash_keys.h

Issue 23471007: Set active extension IDs for crash reports using the crash key logging system. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/child_process_logging_win.cc ('k') | chrome/common/crash_keys.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/crash_keys.h
diff --git a/chrome/common/crash_keys.h b/chrome/common/crash_keys.h
index 45bb6779e935d38e8f5178d2a57de64806c83fe4..0054011fb818143f63e5c9798eb7c2a30013c82b 100644
--- a/chrome/common/crash_keys.h
+++ b/chrome/common/crash_keys.h
@@ -5,6 +5,9 @@
#ifndef CHROME_COMMON_CRASH_KEYS_H_
#define CHROME_COMMON_CRASH_KEYS_H_
+#include <set>
+#include <string>
+
#include "base/debug/crash_logging.h"
namespace crash_keys {
@@ -13,11 +16,28 @@ namespace crash_keys {
// reporting server. Returns the size of the union of all keys.
size_t RegisterChromeCrashKeys();
+// Sets the list of "active" extensions in this process. We overload "active" to
+// mean different things depending on the process type:
+// - browser: all enabled extensions
+// - renderer: the unique set of extension ids from all content scripts
+// - extension: the id of each extension running in this process (there can be
+// multiple because of process collapsing).
+void SetActiveExtensions(const std::set<std::string>& extensions);
+
// Crash Key Name Constants ////////////////////////////////////////////////////
// The URL of the active tab.
extern const char kActiveURL[];
+// Installed extensions. |kExtensionID| should be formatted with an integer,
+// in the range [0, kExtensionIDMaxCount).
+const size_t kExtensionIDMaxCount = 10;
+extern const char kExtensionID[];
+// The total number of installed extensions, recorded in case it exceeds
+// kExtensionIDMaxCount. Also used in chrome/app, but defined here to avoid
+// a common->app dependency.
+extern const char kNumExtensionsCount[];
+
// GPU information.
#if !defined(OS_ANDROID)
extern const char kGPUVendorID[];
« no previous file with comments | « chrome/common/child_process_logging_win.cc ('k') | chrome/common/crash_keys.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698