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

Unified Diff: chrome/browser/chrome_browser_main_win.cc

Issue 2514483002: Add a crash key to record whether a windows machine is domain joined. (Closed)
Patch Set: Merge ToT. Created 4 years 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/app/chrome_crash_reporter_client_win.cc ('k') | chrome/common/crash_keys.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_main_win.cc
diff --git a/chrome/browser/chrome_browser_main_win.cc b/chrome/browser/chrome_browser_main_win.cc
index 465239d949bcbe953c907dbdb4fb615980daac28..877024a2be04460bbedc77af411168a8f81a86f2 100644
--- a/chrome/browser/chrome_browser_main_win.cc
+++ b/chrome/browser/chrome_browser_main_win.cc
@@ -43,6 +43,7 @@
#include "chrome/common/chrome_result_codes.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/chrome_utility_messages.h"
+#include "chrome/common/crash_keys.h"
#include "chrome/common/env_vars.h"
#include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h"
@@ -289,6 +290,13 @@ void ChromeBrowserMainPartsWin::PreMainMessageLoopStart() {
}
int ChromeBrowserMainPartsWin::PreCreateThreads() {
+// Record whether the machine is domain joined in a crash key. This will be used
+// to better identify whether crashes are from enterprise users.
+#if defined(OS_WIN)
sky 2016/12/06 23:23:43 These ifdefs should no longer be necessary (this f
Georges Khalil 2016/12/07 15:00:22 Oops, removed.
+ base::debug::SetCrashKeyValue(crash_keys::kEnrolledToDomain,
+ base::win::IsEnrolledToDomain() ? "yes" : "no");
+#endif
+
int rv = ChromeBrowserMainParts::PreCreateThreads();
// TODO(viettrungluu): why don't we run this earlier?
« no previous file with comments | « chrome/app/chrome_crash_reporter_client_win.cc ('k') | chrome/common/crash_keys.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698