Build some x11 dependent files only if 'use_x11'

chrome/browser/BUILD.gn

Index: chrome/browser/BUILD.gn
diff --git a/chrome/browser/BUILD.gn b/chrome/browser/BUILD.gn
index 7e5c46b60122f210e0e9dc7e0d50a9e64aaeb442..4a7ea444c409de8fb431a8bf5229f3befb1e53b7 100644
--- a/chrome/browser/BUILD.gn
+++ b/chrome/browser/BUILD.gn
@@ -3493,20 +3493,25 @@ split_static_library("browser") {
       "first_run/upgrade_util.cc",
       "first_run/upgrade_util_linux.cc",
       "first_run/upgrade_util_linux.h",
-      "fullscreen_aurax11.cc",
       "icon_loader_auralinux.cc",
-      "media/webrtc/window_icon_util_x11.cc",
-      "password_manager/native_backend_kwallet_x.cc",
-      "password_manager/native_backend_kwallet_x.h",
       "platform_util_linux.cc",
       "shell_integration_linux.cc",
       "shell_integration_linux.h",
       "speech/tts_linux.cc",
-      "themes/theme_service_aurax11.cc",
-      "themes/theme_service_aurax11.h",
       "web_applications/web_app_linux.cc",
     ]
 
+    if (use_x11) {
+      sources += [
+        "fullscreen_aurax11.cc",
+        "media/webrtc/window_icon_util_x11.cc",
+        "password_manager/native_backend_kwallet_x.cc",

[Lei Zhang, 2016/10/27 17:00:15]

I don't see any x11 usage here. I think the 'x' means unix but not mac.

+        "password_manager/native_backend_kwallet_x.h",
+        "themes/theme_service_aurax11.cc",
+        "themes/theme_service_aurax11.h",
+      ]
+    }
+
     # libsecret hard depends on GLib.
     if (use_glib) {
       sources += [