Remove dependency of Blimp on X11/Xlib.h header to avoid namespace polution

blimp/client/app/linux/blimp_main.cc

Index: blimp/client/app/linux/blimp_main.cc
diff --git a/blimp/client/app/linux/blimp_main.cc b/blimp/client/app/linux/blimp_main.cc
index 3eedffe64b0f68d0b7c12ebdaf9627d969d1d9c9..bc0b30e62dbb1af30d4a762fd0be476e09644b50 100644
--- a/blimp/client/app/linux/blimp_main.cc
+++ b/blimp/client/app/linux/blimp_main.cc
@@ -2,8 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include <X11/Xlib.h>
-
 #include <string>
 
 #include "base/at_exit.h"
@@ -16,6 +14,7 @@
 #include "blimp/client/feature/navigation_feature.h"
 #include "blimp/client/feature/tab_control_feature.h"
 #include "blimp/client/session/assignment_source.h"
+#include "ui/gfx/x/x11_connection.h"
 
 namespace {
 const char kDummyLoginToken[] = "";
@@ -27,7 +26,7 @@ int main(int argc, const char**argv) {
   base::AtExitManager at_exit;
   base::CommandLine::Init(argc, argv);
 
-  XInitThreads();
+  gfx::InitializeThreadedX11();

[Wez, 2016/03/23 23:33:21]

nit: CHECK(...) so that we'll crash here if X11 fails to initialize, or connect?

 
   blimp::client::InitializeLogging();
   blimp::client::InitializeMainMessageLoop();