Record field-trial information in stability file for crash analysis.

chrome/browser/chrome_browser_field_trials_desktop.cc

Index: chrome/browser/chrome_browser_field_trials_desktop.cc
diff --git a/chrome/browser/chrome_browser_field_trials_desktop.cc b/chrome/browser/chrome_browser_field_trials_desktop.cc
index a8c81507e2ea79070d69d10d6271cc7e0c6a1e66..1383d200001ec37ad9e9fbcd20bf7df80359da53 100644
--- a/chrome/browser/chrome_browser_field_trials_desktop.cc
+++ b/chrome/browser/chrome_browser_field_trials_desktop.cc
@@ -112,6 +112,7 @@ void RecordChromeModuleInfo(
 void SetupStabilityDebugging() {
   if (!base::FeatureList::IsEnabled(
           browser_watcher::kStabilityDebuggingFeature)) {
+    base::debug::GlobalActivityTracker::DisableFieldTrialRecording();

[Alexei Svitkine (slow), 2017/02/07 16:14:06]

I notice SetupStabilityDebugging() is only called on Windows.

So on other platforms we'll never hit this Disable call.

What will happen then - since the field_trial.cc code still runs on all the
platforms?

[bcwhite, 2017/02/07 17:39:31]

Good catch.  Added call to disable on non-Win platforms.

     return;
   }
 
@@ -161,7 +162,7 @@ void SetupStabilityDebugging() {
                                                 &version_number, &special_build,
                                                 &channel_name);
 
-    base::debug::ActivityUserData& global_data = global_tracker->user_data();
+    base::debug::ActivityUserData& global_data = global_tracker->global_data();
     global_data.SetString(browser_watcher::kStabilityProduct, product_name);
     global_data.SetString(browser_watcher::kStabilityVersion, version_number);
     global_data.SetString(browser_watcher::kStabilityChannel, channel_name);