Revert 93629 - Linux build failure: http://build.chromium.org/p/chromium/builders/Linux%20Builder%20%28dbg%29%28shared%29/builds/4098/steps/compile/logs/stdio#error1

Consolidate access to X Display

Change ui::GetXDisplay to use the MessagePumpForUI class
to get Display. This allows us to have two different
implementation for x and gtk.

 Remove MessageLoop::GetDisplay from message loop API.
Client should use ui::GetXDisplay instead. This
will also be removed in near future, and client should get
display from window it is dealing with.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/7484006

TBR=oshima@google.com
Review URL: http://codereview.chromium.org/7493014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93633 0039d316-1c4b-4281-b951-d872f2087c98

9 files changed, 34 insertions, 34 deletions

diff --git a/base/message_loop.cc b/base/message_loop.cc
index 08985a4..7f1971c 100644
--- a/base/message_loop.cc
+++ b/base/message_loop.cc
@@ -819,6 +819,15 @@ void MessageLoopForUI::DidProcessMessage(const MSG& message) {
 }
 #endif  // defined(OS_WIN)
 
+#if defined(USE_X11)
+Display* MessageLoopForUI::GetDisplay() {
+  GdkDisplay* display = gdk_display_get_default();
+  if (!display)
+    return NULL;
+  return GDK_DISPLAY_XDISPLAY(display);
+}
+#endif  // defined(USE_X11)
+
 #if !defined(OS_MACOSX) && !defined(OS_NACL)
 void MessageLoopForUI::AddObserver(Observer* observer) {
   pump_ui()->AddObserver(observer);
diff --git a/base/message_loop.h b/base/message_loop.h
index 9d500d7..8fd9cbc 100644
--- a/base/message_loop.h
+++ b/base/message_loop.h
@@ -32,6 +32,7 @@
 #else
 #include "base/message_pump_gtk.h"
 #endif
+typedef struct _XDisplay Display;
 #endif
 #endif
 
@@ -583,6 +584,18 @@ class BASE_API MessageLoopForUI : public MessageLoop {
   void DidProcessMessage(const MSG& message);
 #endif  // defined(OS_WIN)
 
+#if defined(USE_X11)
+  // Returns the Xlib Display that backs the MessagePump for this MessageLoop.
+  //
+  // This allows for raw access to the X11 server in situations where our
+  // abstractions do not provide enough power.
+  //
+  // Be careful how this is used. The MessagePump in general expects
+  // exclusive access to the Display. Calling things like XNextEvent() will
+  // likely break things in subtle, hard to detect, ways.
+  Display* GetDisplay();
+#endif  // defined(OS_X11)
+
 #if !defined(OS_MACOSX)
   // Please see message_pump_win/message_pump_glib for definitions of these
   // methods.
diff --git a/base/message_pump_gtk.cc b/base/message_pump_gtk.cc
index e1cb7f6..f5ed042 100644
--- a/base/message_pump_gtk.cc
+++ b/base/message_pump_gtk.cc
@@ -5,7 +5,6 @@
 #include "base/message_pump_gtk.h"
 
 #include <gtk/gtk.h>
-#include <gdk/gdkx.h>
 
 namespace base {
 
@@ -30,12 +29,6 @@ void MessagePumpGtk::DispatchEvents(GdkEvent* event) {
   DidProcessEvent(event);
 }
 
-// static
-Display* MessagePumpGtk::GetDefaultXDisplay() {
-  static GdkDisplay* display = gdk_display_get_default();
-  return display ? GDK_DISPLAY_XDISPLAY(display) : NULL;
-}
-
 bool MessagePumpGtk::RunOnce(GMainContext* context, bool block) {
   // g_main_context_iteration returns true if events have been dispatched.
   return g_main_context_iteration(context, block);
@@ -56,3 +49,4 @@ void MessagePumpGtk::EventDispatcher(GdkEvent* event, gpointer data) {
 }
 
 }  // namespace base
+
diff --git a/base/message_pump_gtk.h b/base/message_pump_gtk.h
index 7e5cf04..72eaafa 100644
--- a/base/message_pump_gtk.h
+++ b/base/message_pump_gtk.h
@@ -9,7 +9,6 @@
 #include "base/message_pump_glib.h"
 
 typedef union _GdkEvent GdkEvent;
-typedef struct _XDisplay Display;
 
 namespace base {
 
@@ -50,9 +49,6 @@ class MessagePumpGtk : public MessagePumpGlib {
   // some task before/after calling the default handler (EventDispatcher).
   void DispatchEvents(GdkEvent* event);
 
-  // Returns default X Display.
-  static Display* GetDefaultXDisplay();
-
  private:
   // Overridden from MessagePumpGlib
   virtual bool RunOnce(GMainContext* context, bool block) OVERRIDE;
diff --git a/base/message_pump_x.cc b/base/message_pump_x.cc
index 1174e4b..20c8b0e 100644
--- a/base/message_pump_x.cc
+++ b/base/message_pump_x.cc
@@ -51,13 +51,6 @@ void MessagePumpX::DisableGtkMessagePump() {
   use_gtk_message_pump = false;
 }
 
-// static
-Display* MessagePumpX::GetDefaultXDisplay() {
-  static GdkDisplay* display = gdk_display_get_default();
-  return display ? GDK_DISPLAY_XDISPLAY(display) : NULL;
-}
-
-
 bool MessagePumpX::ShouldCaptureXEvent(XEvent* xev) {
   return (!use_gtk_message_pump || capture_x_events_[xev->type])
       && (xev->type != GenericEvent || xev->xcookie.extension == xiopcode_)
@@ -93,7 +86,7 @@ bool MessagePumpX::ProcessXEvent(XEvent* xev) {
 }
 
 bool MessagePumpX::RunOnce(GMainContext* context, bool block) {
-  Display* display = GetDefaultXDisplay();
+  Display* display = MessageLoopForUI::current()->GetDisplay();
   if (!display || !GetDispatcher())
     return g_main_context_iteration(context, block);
 
@@ -199,7 +192,7 @@ void MessagePumpX::InitializeEventsToCapture(void) {
 }
 
 void MessagePumpX::InitializeXInput2(void) {
-  Display* display = GetDefaultXDisplay();
+  Display* display = MessageLoopForUI::current()->GetDisplay();
   if (!display)
     return;
 
diff --git a/base/message_pump_x.h b/base/message_pump_x.h
index 8ac9921..ecadc1f 100644
--- a/base/message_pump_x.h
+++ b/base/message_pump_x.h
@@ -14,7 +14,6 @@
 #include <gtk/gtk.h>
 
 typedef union _XEvent XEvent;
-typedef struct _XDisplay Display;
 
 namespace base {
 
@@ -75,9 +74,6 @@ class MessagePumpX : public MessagePumpGlib {
   // NativeWidgetX is enabled.
   static void DisableGtkMessagePump();
 
-  // Returns default X Display.
-  static Display* GetDefaultXDisplay();
-
  private:
   // Some XEvent's can't be directly read from X event queue and will go
   // through GDK's dispatching process and may get discarded. This function
diff --git a/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc b/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
index 7daba16..bf1ab56 100644
--- a/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
+++ b/chrome/browser/chromeos/xinput_hierarchy_changed_event_listener.cc
@@ -9,7 +9,6 @@
 #include <X11/extensions/XInput2.h>
 
 #include "chrome/browser/chromeos/input_method/xkeyboard.h"
-#include "ui/base/x/x11_util.h"
 
 namespace {
 
@@ -20,8 +19,8 @@ int GetXInputOpCode() {
   int event;
   int error;
 
-  if (!XQueryExtension(
-          ui::GetXDisplay(), kExtensionName, &xi_opcode, &event, &error)) {
+  Display* display = MessageLoopForUI::current()->GetDisplay();
+  if (!XQueryExtension(display, kExtensionName, &xi_opcode, &event, &error)) {
     VLOG(1) << "X Input extension not available: error=" << error;
     return -1;
   }
@@ -38,7 +37,7 @@ void SelectXInputEvents() {
   evmask.mask_len = sizeof(mask);
   evmask.mask = mask;
 
-  Display* display = ui::GetXDisplay();
+  Display* display = MessageLoopForUI::current()->GetDisplay();
   XISelectEvents(display, DefaultRootWindow(display), &evmask, 1);
 }
 
diff --git a/ui/base/x/x11_util.cc b/ui/base/x/x11_util.cc
index d7d51cf..51f52e2 100644
--- a/ui/base/x/x11_util.cc
+++ b/ui/base/x/x11_util.cc
@@ -20,7 +20,6 @@
 
 #include "base/command_line.h"
 #include "base/logging.h"
-#include "base/message_loop.h"
 #include "base/stringprintf.h"
 #include "base/string_number_conversions.h"
 #include "base/threading/thread.h"
@@ -97,7 +96,12 @@ bool XDisplayExists() {
 }
 
 Display* GetXDisplay() {
-  return base::MessagePumpForUI::GetDefaultXDisplay();
+  static Display* display = NULL;
+
+  if (!display)
+    display = gdk_x11_get_default_xdisplay();
+
+  return display;
 }
 
 static SharedMemorySupport DoQuerySharedMemorySupport(Display* dpy) {
diff --git a/ui/base/x/x11_util.h b/ui/base/x/x11_util.h
index 6728c1e..ab455a0 100644
--- a/ui/base/x/x11_util.h
+++ b/ui/base/x/x11_util.h
@@ -39,10 +39,6 @@ namespace ui {
 // Check if there's an open connection to an X server.
 UI_API bool XDisplayExists();
 // Return an X11 connection for the current, primary display.
-
-// TODO(oshima|evan): This assume there is one display and dosn't work
-// undef mutiple displays/monitor environment. Remove this and change the
-// chrome codebase to get the display from window.
 UI_API Display* GetXDisplay();
 
 // X shared memory comes in three flavors: