Remove X11/Xlib.h include from selection_requestor.h

ui/base/x/selection_requestor.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/base/x/selection_requestor.h"
 
+#include <X11/Xlib.h>
+
 #include "base/run_loop.h"
 #include "ui/base/x/selection_utils.h"
 #include "ui/base/x/x11_util.h"
 #include "ui/events/platform/platform_event_dispatcher.h"
 #include "ui/events/platform/platform_event_source.h"
 #include "ui/gfx/x/x11_types.h"
 
 namespace ui {
 
 namespace {
 
 const char kChromeSelection[] = "CHROME_SELECTION";
 
 const char* kAtomsToCache[] = {
   kChromeSelection,
   NULL
 };
 
 }  // namespace
 
-SelectionRequestor::SelectionRequestor(Display* x_display,
-                                       Window x_window,
+SelectionRequestor::SelectionRequestor(XDisplay* x_display,
+                                       XID x_window,
                                        Atom selection_name,
                                        PlatformEventDispatcher* dispatcher)
     : x_display_(x_display),
       x_window_(x_window),
       selection_name_(selection_name),
       dispatcher_(dispatcher),
       atom_cache_(x_display_, kAtomsToCache) {
 }
 
 SelectionRequestor::~SelectionRequestor() {}
@@ skipping 27 matching lines @@
                                          pending_request.returned_property,
                                          out_data, out_data_items, out_type);
   }
   if (pending_request.returned_property != None)
     XDeleteProperty(x_display_, x_window_, pending_request.returned_property);
   return success;
 }
 
 void SelectionRequestor::PerformBlockingConvertSelectionWithParameter(
     Atom target,
-    const std::vector< ::Atom>& parameter) {
+    const std::vector<Atom>& parameter) {
   SetAtomArrayProperty(x_window_, kChromeSelection, "ATOM", parameter);
   PerformBlockingConvertSelection(target, NULL, NULL, NULL);
 }
 
 SelectionData SelectionRequestor::RequestAndWaitForTypes(
-    const std::vector< ::Atom>& types) {
-  for (std::vector< ::Atom>::const_iterator it = types.begin();
+    const std::vector<Atom>& types) {
+  for (std::vector<Atom>::const_iterator it = types.begin();
        it != types.end(); ++it) {
     scoped_refptr<base::RefCountedMemory> data;
-    ::Atom type = None;
+    Atom type = None;
     if (PerformBlockingConvertSelection(*it,
                                         &data,
                                         NULL,
                                         &type) &&
         type == *it) {
       return SelectionData(type, data);
     }
   }
 
   return SelectionData();
 }
 
-void SelectionRequestor::OnSelectionNotify(const XSelectionEvent& event) {
+void SelectionRequestor::OnSelectionNotify(const XEvent& event) {
   // Find the PendingRequest for the corresponding XConvertSelection call. If
   // there are multiple pending requests on the same target, satisfy them in
   // FIFO order.
   PendingRequest* request_notified = NULL;
-  if (selection_name_ == event.selection) {
+  if (selection_name_ == event.xselection.selection) {
     for (std::list<PendingRequest*>::iterator iter = pending_requests_.begin();
          iter != pending_requests_.end(); ++iter) {
       PendingRequest* request = *iter;
       if (request->returned)
         continue;
-      if (request->target != event.target)
+      if (request->target != event.xselection.target)
         continue;
       request_notified = request;
       break;
     }
   }
 
   // This event doesn't correspond to any XConvertSelection calls that we
   // issued in PerformBlockingConvertSelection. This shouldn't happen, but any
   // client can send any message, so it can happen.
+  Atom returned_property = event.xselection.property;
   if (!request_notified) {
     // ICCCM requires us to delete the property passed into SelectionNotify. If
     // |request_notified| is true, the property will be deleted when the run
     // loop has quit.
-    if (event.property != None)
-      XDeleteProperty(x_display_, x_window_, event.property);
+    if (returned_property != None)
+      XDeleteProperty(x_display_, x_window_, returned_property);
     return;
   }
 
-  request_notified->returned_property = event.property;
+  request_notified->returned_property = returned_property;
   request_notified->returned = true;
 
   if (!request_notified->quit_closure.is_null())
     request_notified->quit_closure.Run();
 }
 
 void SelectionRequestor::BlockTillSelectionNotifyForRequest(
     PendingRequest* request) {
   pending_requests_.push_back(request);
 
@@ skipping 34 matching lines @@
 SelectionRequestor::PendingRequest::PendingRequest(Atom target)
     : target(target),
       returned_property(None),
       returned(false) {
 }
 
 SelectionRequestor::PendingRequest::~PendingRequest() {
 }
 
 }  // namespace ui