x11: Use scoped_ptr<> for X11 objects where it makes sense.

ui/base/x/selection_owner.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_owner.h"
 
 #include <algorithm>
 #include <X11/Xlib.h>
 #include <X11/Xatom.h>
 
@@ skipping 59 matching lines @@
                                   0,          // offset into property data to
                                               // read
                                   (~0L),      // entire array
                                   False,      // deleted
                                   AnyPropertyType,
                                   &type,
                                   &format,
                                   &num_items,
                                   &remaining_bytes,
                                   &properties);
+  gfx::XScopedPtr<unsigned char> scoped_properties(properties);
 
   if (result != Success)
     return false;
 
   // GTK does not require |type| to be kAtomPair.
-  if (format != 32 || num_items % 2 != 0) {
-    XFree(properties);
+  if (format != 32 || num_items % 2 != 0)
     return false;
-  }
 
   XAtom* atom_properties = reinterpret_cast<XAtom*>(properties);
   value->clear();
   for (size_t i = 0; i < num_items; i+=2)
     value->push_back(std::make_pair(atom_properties[i], atom_properties[i+1]));
-  XFree(properties);
   return true;
 }
 
 }  // namespace
 
 SelectionOwner::SelectionOwner(XDisplay* x_display,
                                XID x_window,
                                XAtom selection_name)
     : x_display_(x_display),
       x_window_(x_window),
@@ skipping 270 matching lines @@
       data(data),
       offset(offset),
       timeout(timeout),
       foreign_window_manager_id(foreign_window_manager_id) {
 }
 
 SelectionOwner::IncrementalTransfer::~IncrementalTransfer() {
 }
 
 }  // namespace ui