linux_aura: Bail early if we can't connect to the X11 server.

ui/gfx/x/x11_types.cc

 // Copyright (c) 2013 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 "ui/gfx/x/x11_types.h"
 
 #include <X11/Xlib.h>
 
+#include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
+#include "ui/gfx/x/x11_switches.h"
 
 namespace gfx {
 
 XDisplay* GetXDisplay() {
   static XDisplay* display = NULL;
   if (!display)
-    display = XOpenDisplay(NULL);
+    display = OpenNewXDisplay();
   return display;
 }
 
+XDisplay* OpenNewXDisplay() {
+#if defined(OS_CHROMEOS)

[Ami GONE FROM CHROMIUM, 2014/05/29 00:28:43]

CHECK cmd line value is not present, since you ignore it?

[Elliot Glaysher, 2014/05/29 00:35:22]

I don't think I can do this because switches::kX11Display isn't compiled in on
chromoes.

+  return XOpenDisplay(NULL);
+#else
+  std::string display_str = base::CommandLine::ForCurrentProcess()->
+                            GetSwitchValueASCII(switches::kX11Display);
+  return XOpenDisplay(display_str.empty() ? NULL : display_str.c_str());
+#endif
+}
+
 void PutARGBImage(XDisplay* display,
                   void* visual, int depth,
                   XID pixmap, void* pixmap_gc,
                   const uint8* data,
                   int width, int height) {
   PutARGBImage(display,
                visual, depth,
                pixmap, pixmap_gc,
                data, width, height,
                0, 0, // src_x, src_y
@@ skipping 119 matching lines @@
     free(orig_bitmap16);
   } else {
     LOG(FATAL) << "Sorry, we don't support your visual depth without "
                   "Xrender support (depth:" << depth
                << " bpp:" << pixmap_bpp << ")";
   }
 }
 
 }  // namespace gfx