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

content/renderer/render_process_impl.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 "content/renderer/render_process_impl.h"
 
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <objidl.h>
 #include <mlang.h>
 #endif
 
 #include <stddef.h>
 
 #include <vector>
 
 #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"
 #include "base/task_scheduler/scheduler_worker_pool_params.h"
 #include "base/task_scheduler/task_scheduler.h"
 #include "base/task_scheduler/task_traits.h"
 #include "base/threading/platform_thread.h"
 #include "base/time/time.h"
 #include "content/child/site_isolation_stats_gatherer.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/content_features.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/renderer/content_renderer_client.h"
 #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 {
   BACKGROUND = 0,
   BACKGROUND_FILE_IO,
   FOREGROUND,
   FOREGROUND_FILE_IO,
   WORKER_POOL_COUNT  // Always last.
 };
 
@@ skipping 100 matching lines @@
       params_vector, index_to_traits_callback);
 }
 
 }  // namespace
 
 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
     // when buffering into a EMF buffer for printing.
     typedef BOOL (__stdcall *GdiInitializeLanguagePack)(int LoadedShapingDLLs);
     GdiInitializeLanguagePack gdi_init_lpk =
         reinterpret_cast<GdiInitializeLanguagePack>(GetProcAddress(
             GetModuleHandle(L"GDI32.DLL"),
             "GdiInitializeLanguagePack"));
     DCHECK(gdi_init_lpk);
@@ skipping 55 matching lines @@
 
 void RenderProcessImpl::AddBindings(int bindings) {
   enabled_bindings_ |= bindings;
 }
 
 int RenderProcessImpl::GetEnabledBindings() const {
   return enabled_bindings_;
 }
 
 }  // namespace content