Capture a report on failed browser rendez-vous.

chrome/app/main_dll_loader_win.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 <windows.h>  // NOLINT
 #include <shlwapi.h>  // NOLINT
 #include <stddef.h>
 #include <userenv.h>  // NOLINT
 
 #include "chrome/app/main_dll_loader_win.h"
@@ skipping 206 matching lines @@
 #endif
 };
 
 void ChromeDllLoader::OnBeforeLaunch(const std::string& process_type,
                                      const base::FilePath& dll_path) {
   if (process_type.empty()) {
     RecordDidRun(dll_path);
 
     // Launch the watcher process if stats collection consent has been granted.
 #if defined(GOOGLE_CHROME_BUILD)
-    const bool stats_collection_consent =
-        GoogleUpdateSettings::GetCollectStatsConsent();
+    const bool stats_collection_consent = crash_reporter::GetUploadsEnabled();

[scottmg, 2016/04/08 15:52:25]

Yes, this lgtm. You could collapse out the #if defined(GOOGLE_CHROME_BUILD) to
match what Crashpad does.

[manzagop (departed), 2016/04/08 21:15:56]

I think I understood what you meant, but please validate.

[scottmg, 2016/04/08 22:20:55]

Yeah, that's what I meant, sorry for the terseness.

 #else
     const bool stats_collection_consent = false;
 #endif
     if (stats_collection_consent) {
       base::FilePath exe_path;
       if (PathService::Get(base::FILE_EXE, &exe_path)) {
         chrome_watcher_client_.reset(new ChromeWatcherClient(
             base::Bind(&GenerateChromeWatcherCommandLine, exe_path)));
         if (chrome_watcher_client_->LaunchWatcher()) {
 #if BUILDFLAG(ENABLE_KASKO)
@@ skipping 57 matching lines @@
   }
 };
 
 MainDllLoader* MakeMainDllLoader() {
 #if defined(GOOGLE_CHROME_BUILD)
   return new ChromeDllLoader();
 #else
   return new ChromiumDllLoader();
 #endif
 }