Add enrolled to domain bit to crash keys in renderer process.

content/renderer/render_process_impl.cc

Index: content/renderer/render_process_impl.cc
diff --git a/content/renderer/render_process_impl.cc b/content/renderer/render_process_impl.cc
index 20f71e1ed9fcf3b8e0b04ca5ed85f6041d56bb8c..e837e4f5d92f152545a699bc36dcbd80bf26c35c 100644
--- a/content/renderer/render_process_impl.cc
+++ b/content/renderer/render_process_impl.cc
@@ -19,6 +19,7 @@
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
+#include "base/debug/crash_logging.h"
 #include "base/feature_list.h"
 #include "base/sys_info.h"
 #include "base/task_scheduler/initialization_util.h"
@@ -35,6 +36,10 @@
 #include "third_party/WebKit/public/web/WebFrame.h"
 #include "v8/include/v8.h"
 
+#if defined(OS_WIN)
+#include "base/win/win_util.h"
+#endif
+
 namespace {
 
 enum WorkerPoolType : size_t {
@@ -155,6 +160,13 @@ namespace content {
 RenderProcessImpl::RenderProcessImpl()
     : enabled_bindings_(0) {
 #if defined(OS_WIN)
+  // Record whether the machine is domain joined in a crash key. This will be
+  // used to better identify whether crashes are from enterprise users.
+  // Note that this is done very early on so that crashes have the highest
+  // chance of getting tagged.
+  base::debug::SetCrashKeyValue("enrolled-to-domain",
+                                base::win::IsEnrolledToDomain() ? "yes" : "no");

[nasko, 2017/01/09 23:12:35]

Shouldn't this be logged only once per Chrome instance? I don't think it makes
sense to log it for each renderer process, as domain membership changes on
Windows require reboot, therefore the probability of having two renderer process
be started across change in domain members is very low.

Also, don't we want to annotate browser process crashes with this info as well?
If so, this is much later than we'd like it to be. Why not look at the browser
process initialization and capture it there?

+
   // HACK:  See http://b/issue?id=1024307 for rationale.
   if (GetModuleHandle(L"LPK.DLL") == NULL) {
     // Makes sure lpk.dll is loaded by gdi32 to make sure ExtTextOut() works