Fix Ozone build

chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc

Index: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc
diff --git a/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc b/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc
index 519e2cb0df11315f2769c966ab117712c6231db1..ae3278c1a6b093f93bd1368f9363080cc35edda5 100644
--- a/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc
+++ b/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc
@@ -464,13 +464,14 @@ void ChromeBrowserMainExtraPartsMetrics::PreBrowserStart() {
 
 void ChromeBrowserMainExtraPartsMetrics::PostBrowserStart() {
   RecordLinuxGlibcVersion();
-#if defined(USE_X11) && !defined(OS_CHROMEOS)
-  UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager",
-                            GetLinuxWindowManager(),
-                            UMA_LINUX_WINDOW_MANAGER_COUNT);
+#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   base::PostTaskWithTraits(FROM_HERE,
                            {base::MayBlock(), base::TaskPriority::BACKGROUND},
                            base::BindOnce(&RecordLinuxDistro));
+#if defined(USE_X11)

[Tom Anderson, 2017/05/30 22:57:07]

GetLinuxWindowManager above is guarded with "defined(USE_X11) &&
!defined(OS_CHROMEOS)", but here it effectively has "defined(OS_LINUX) &&
!defined(OS_CHROMEOS) && defined(USE_X11)", but this is probably OK because X11
is only used on Linux.  Personally I'd recommend splitting it out because it's
easier to understand.

[timbrown, 2017/05/30 23:03:58]

Sure. Done.

+  UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager", GetLinuxWindowManager(),
+                            UMA_LINUX_WINDOW_MANAGER_COUNT);
+#endif
 #endif
 
 #if defined(USE_OZONE) || defined(USE_X11)